ci: add preview smoke test for frontend PRs#7192
Open
talissoncosta wants to merge 5 commits intomainfrom
Open
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
1 Skipped Deployment
|
github-actions
bot
added
ci-cd
github-actions
bot
added
ci-cd
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #7192 +/- ##
=======================================
Coverage 98.35% 98.35%
=======================================
Files 1348 1348
Lines 50691 50691
=======================================
Hits 49859 49859
Misses 832 832 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
github-actions
bot
added
ci-cd
github-actions
bot
added
ci-cd
github-actions
bot
added
ci-cd
github-actions
bot
added
ci-cd
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
talissoncosta
changed the title
Adds a CI job that waits for Vercel preview deployments to complete, then curls /health, /config/project-overrides, and / to verify they return 200. This catches server-side failures (like the missing index.handlebars in #7178) that the existing E2E and unit tests don't cover because they don't exercise the Vercel serverless path. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Trivial comment to ensure Vercel deploys a preview so the new smoke test job can run against it. Will be removed before merging. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Temporary change to validate the workflow works. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
talissoncosta
force-pushed
the
ci/frontend-smoke-test-on-vercel-preview
branch
from
8281d0d to
d10f1e5
Compare
github-actions
bot
added
ci-cd
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
talissoncosta
changed the title
github-actions
bot
added
ci-cd
![claude[bot]](https://avatars.githubusercontent.com/in/1236702?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code review is billed via overage credits. To resume reviews, an organization admin can raise the monthly limit at claude.ai/admin-settings/claude-code.
Once credits are available, reopen this pull request to trigger a review.
github-actions
bot
added
ci-cd
Contributor
Docker builds report
|
talissoncosta
changed the title
Contributor
Playwright Test Results (oss - depot-ubuntu-latest-16)
Details
Playwright Test Results (oss - depot-ubuntu-latest-arm-16)
Details
Playwright Test Results (private-cloud - depot-ubuntu-latest-arm-16)
Details
Playwright Test Results (private-cloud - depot-ubuntu-latest-16)
Details
|
Contributor
Visual Regression16 screenshots compared. See report for details. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
docs/if required so people know about the feature.Changes
Contributes to #7178
After the Vercel preview returned 500 on a recent PR while all CI checks passed, this adds a simple smoke test to the PR workflow that validates Vercel preview deployments are healthy before merging.
What it does
Adds a
smoke-testjob tofrontend-pull-request.ymlthat:/health/config/project-overrides/(index page)Checks both preview environments (
flagsmith-frontend-previewandflagsmith-frontend-staging). Times out gracefully if a deployment doesn't appear within 10 minutes.Why this approach
This is intentionally simple — just HTTP status checks, no E2E or browser tests. The goal is to catch deployment failures (500s, build errors, broken Express server) that unit tests can't detect. If we want more coverage later, we can build on this.
What changed
.github/workflows/frontend-pull-request.yml— Addeddeployments: readpermission andsmoke-testjobHow did you test this code?
Triggered the workflow via
workflow_dispatchon this branch and verified the smoke test job runs against the Vercel preview URLs. Will validate on this PR once Vercel deploys the preview.