-
Notifications
You must be signed in to change notification settings - Fork 2
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
Add GitHub Action for Vercel preview deployment e2e testing #446
Add GitHub Action for Vercel preview deployment e2e testing #446
Conversation
Co-Authored-By: Nathan Clevenger <[email protected]>
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
Co-Authored-By: Nathan Clevenger <[email protected]>
Co-Authored-By: Nathan Clevenger <[email protected]>
Co-Authored-By: Nathan Clevenger <[email protected]>
Co-Authored-By: Nathan Clevenger <[email protected]>
we should add e2e tests for APIs that just use fetch Also, there is a new secret in Github Actions for VERCEL_AUTOMATION_BYPASS_SECRET ... you can bypass vercel protection by adding that to a header called x-vercel-protection-bypass for playwright, we can: const config: PlaywrightTestConfig = {
use: {
extraHTTPHeaders: {
'x-vercel-protection-bypass': process.env.VERCEL_AUTOMATION_BYPASS_SECRET,
'x-vercel-set-bypass-cookie': true | 'samesitenone' (optional)
}
}
} finally, let's shorten the action name to |
…n name Co-Authored-By: Nathan Clevenger <[email protected]>
try { | ||
const baseUrl = process.env.BASE_URL || 'http://localhost:3000' | ||
|
||
const apiEndpoints = [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You should also add /
and /api
Co-Authored-By: Nathan Clevenger <[email protected]>
Co-Authored-By: Nathan Clevenger <[email protected]>
Co-Authored-By: Nathan Clevenger <[email protected]>
…ests Co-Authored-By: Nathan Clevenger <[email protected]>
Co-Authored-By: Nathan Clevenger <[email protected]>
…loyments Co-Authored-By: Nathan Clevenger <[email protected]>
…review URL Co-Authored-By: Nathan Clevenger <[email protected]>
…ct Vercel preview URL Co-Authored-By: Nathan Clevenger <[email protected]>
Co-Authored-By: Nathan Clevenger <[email protected]>
Co-Authored-By: Nathan Clevenger <[email protected]>
…view URL Co-Authored-By: Nathan Clevenger <[email protected]>
Co-Authored-By: Nathan Clevenger <[email protected]>
Co-Authored-By: Nathan Clevenger <[email protected]>
Co-Authored-By: Nathan Clevenger <[email protected]>
…s workflow Co-Authored-By: Nathan Clevenger <[email protected]>
Co-Authored-By: Nathan Clevenger <[email protected]>
Co-Authored-By: Nathan Clevenger <[email protected]>
…us URL formats Co-Authored-By: Nathan Clevenger <[email protected]>
Co-Authored-By: Nathan Clevenger <[email protected]>
Co-Authored-By: Nathan Clevenger <[email protected]>
Co-Authored-By: Nathan Clevenger <[email protected]>
Co-Authored-By: Nathan Clevenger <[email protected]>
should you pull latest from main on this branch? a lot has changed on main |
…el-preview-e2e-tests
…nly BASE_URL Co-Authored-By: Nathan Clevenger <[email protected]>
Closing due to inactivity for more than 7 days. |
Resolves #445
This PR adds a GitHub Action workflow that waits for Vercel preview deployments to complete and then runs e2e tests against the preview URL. The workflow:
Also adds a new e2e test file that tests:
Link to Devin run: https://app.devin.ai/sessions/fc3bf7f6915448d39a00af6ba6368f1b
Requested by: Nathan Clevenger ([email protected])