Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: Run Playwright against alpha.cartokit.dev in production. #218

Merged
merged 2 commits into from
May 16, 2024

Conversation

parkerziegler
Copy link
Owner

This PR attempts to address an issue unintentionally missed in #211.

Currently, we run our Playwright e2e tests against Vercel's deployment urls, even in production. However, our production tile server (Stadia Maps) will reject requests from arbitrary Vercel deployments owing to its strict domain-based authentication scheme. We get around this in preview environments by using MapTiler with the cartokit API key instead. In production, however, we want to run using Stadia's basemaps, which means running e2e tests against alpha.cartokit.dev directly.

This PR conditionally sets our PLAYWRIGHT_TEST_BASE_URL to alpha.cartokit.dev when we're in a production environment (e.g., on merge to main).

Copy link

vercel bot commented May 16, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
cartokit-zetq ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 16, 2024 10:24pm

@parkerziegler parkerziegler force-pushed the ci/run-against-alpha-cartokit-dev branch from 0741ec4 to edcd062 Compare May 16, 2024 22:22
@parkerziegler parkerziegler merged commit 6659827 into main May 16, 2024
5 checks passed
@parkerziegler parkerziegler deleted the ci/run-against-alpha-cartokit-dev branch May 16, 2024 22:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant