-
Notifications
You must be signed in to change notification settings - Fork 2.1k
ci(workflow): upload integration test resuls from main branch, next.j… …s release #3502
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
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
8 Ignored Deployments
|
5d0761b
to
fb02e8b
Compare
🟢 CI successful 🟢Thanks |
686509c
to
dadd44f
Compare
Benchmark for 81c2f87Click to view benchmark
|
6c841a1
to
21a114b
Compare
Benchmark for fea5d51
Click to view full benchmark
|
5ab5028
to
7e1e495
Compare
e5ed684
to
669a23a
Compare
Failing next.js integration test suitesCommit: d3b4459
Expand output● Component Stack in error overlay › should show a component stack on hydration error
Expand output● i18n: Event with stale state - static route previously was dynamic › Ignore event without query param
● i18n: Event with stale state - static route previously was dynamic › Ignore event with query param
● i18n: Event with stale state - static route previously was dynamic › Don't ignore event with different locale
Expand output● 404 Page Support with _app › dev mode › should not show pages/404 GIP error if _app has GIP
Expand output● ReactRefreshRegression › styled-components hydration mismatch
● ReactRefreshRegression › can fast refresh a page with getStaticProps
● ReactRefreshRegression › can fast refresh a page with getServerSideProps
● ReactRefreshRegression › can fast refresh a page with config
● ReactRefreshRegression › shows an overlay for a server-side error
● ReactRefreshRegression › custom loader (mdx) should have Fast Refresh enabled
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● i18n: Event with stale state - static route previously was dynamic › Ignore event without query param
● i18n: Event with stale state - static route previously was dynamic › Ignore event with query param
● i18n: Event with stale state - static route previously was dynamic › Don't ignore event with different locale
Expand output● 404 Page Support with _app › dev mode › should not show pages/404 GIP error if _app has GIP
Expand output● ReactRefreshRegression › styled-components hydration mismatch
● ReactRefreshRegression › can fast refresh a page with getStaticProps
● ReactRefreshRegression › can fast refresh a page with getServerSideProps
● ReactRefreshRegression › can fast refresh a page with config
● ReactRefreshRegression › shows an overlay for a server-side error
● ReactRefreshRegression › custom loader (mdx) should have Fast Refresh enabled
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
|
669a23a
to
226021d
Compare
226021d
to
be4f16e
Compare
Benchmark for 94a40faClick to view benchmark
|
be4f16e
to
3317b60
Compare
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.
In general, I'm a little worried about this breaking (maybe silently) because of its reliance on log formats, marker strings, etc.
.map((line) => line.substr("2020-03-02T19:39:16.8832288Z ".length)); | ||
|
||
const nextjsVersion = dateTimeStripped | ||
.find((x) => x.includes("RUNNING NEXTJS VERSION:") && !x.includes("$(")) |
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.
nit: extract "RUNNING NEXTJS VERSION:"
into a const?
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.
yup, will do it following pr for the formatting changes.
Benchmark for d3b4459
Click to view full benchmark
|
Closes WEB-487.
This PR amends existing workflow for next-integration test to store its aggregated test results into github artifacts, then corresponding events for the running test (main branch update / new next.js release) to upload it into specific github branch
nextjs-integration-test-data
.