-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Move E2E tests to main test.yml workflow #3442
Move E2E tests to main test.yml workflow #3442
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 10 Ignored Deployments
|
Failing next.js integration test suitesCommit: f7222aa
Expand output● Component Stack in error overlay › should show a component stack on hydration error
Expand output● Edge API endpoints can receive body › reads the body as text
● Edge API endpoints can receive body › reads the body from index
Expand output● bigint API route support › dev mode › should return 200
● bigint API route support › dev mode › should return the BigInt result text
Expand output● ReactRefresh › can edit a component without losing state
● ReactRefresh › cyclic dependencies
Expand output● app-dir alias handling › should handle typescript paths alias correctly
● app-dir alias handling › should resolve css imports from outside with src folder presented
Expand output● Default 404 Page with custom _error › dev mode › should respond to 404 correctly
● Default 404 Page with custom _error › dev mode › should render error correctly
● Default 404 Page with custom _error › dev mode › should render index page normal
Expand output● Component Stack in error overlay › should show a component stack on hydration error
Expand output● Edge API endpoints can receive body › reads the body as text
● Edge API endpoints can receive body › reads the body from index
Expand output● bigint API route support › dev mode › should return 200
● bigint API route support › dev mode › should return the BigInt result text
Expand output● ReactRefresh › can edit a component without losing state
● ReactRefresh › cyclic dependencies
Expand output● app-dir alias handling › should handle typescript paths alias correctly
● app-dir alias handling › should resolve css imports from outside with src folder presented
Expand output● Default 404 Page with custom _error › dev mode › should respond to 404 correctly
● Default 404 Page with custom _error › dev mode › should render error correctly
● Default 404 Page with custom _error › dev mode › should render index page normal
|
Benchmark for ff4b13a
Click to view full benchmark
|
|
Benchmark for 78b8c3a
Click to view full benchmark
|
name: Go E2E Tests | ||
needs: determine_jobs | ||
if: needs.determine_jobs.outputs.go == 'true' | ||
timeout-minutes: 30 |
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.
It looks like this might not be enough time to build and run the tests on Windows 😢
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.
2b6dc14
to
6771812
Compare
Benchmark for 9394c56
Click to view full benchmark
|
4dfef5b
to
602c98e
Compare
Benchmark for 0f96e1d
Click to view full benchmark
|
You'll need to mark these as required as well (like here): #3519 |
@tknickman done |
3831040
to
2066e5f
Compare
The base branch was changed.
2066e5f
to
4979f8d
Compare
.github/workflows/test.yml
Outdated
uses: technote-space/get-diff-action@v6 | ||
with: | ||
PATTERNS: | | ||
shim/** |
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.
shim is no longer needed. We should probably add turbo-updater
too
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.
removing shim, but can we rename turbo-updater to turborepo-updater instead? Why the naming inconsistency?
Benchmark for f7222aa
Click to view full benchmark
|
Merging this. Rust tests failed, but @jridgewell confirmed that they're failing at the moment. All other checks are green |
No description provided.