-
Notifications
You must be signed in to change notification settings - Fork 27.1k
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
test: replace check
to retry
in hmr.test.ts
#73590
test: replace check
to retry
in hmr.test.ts
#73590
Conversation
Failing test suitesCommit: 05602e7
Expand output● AMP Validation on Export › production mode › should have shown errors during build
Read more about building and testing Next.js in contributing.md.
Expand output● Build warnings › production mode › should warn about missing cache in CI
Read more about building and testing Next.js in contributing.md.
Expand output● Production Usage › should navigate through history after query update
● Production Usage › should handle query/hash correctly during query updating #hello? $search
● Production Usage › should handle query/hash correctly during query updating #? $search
● Production Usage › should handle query/hash correctly during query updating ## $search
● Production Usage › should handle query/hash correctly during query updating ##? $search
● Production Usage › should handle query/hash correctly during query updating ##hello? $search
● Production Usage › should handle query/hash correctly during query updating ##hello $search
● Production Usage › should handle query/hash correctly during query updating #hello?world $search
● Production Usage › should handle query/hash correctly during query updating #a ?hello=world
● Production Usage › should handle query/hash correctly during query updating #a ?hello
● Production Usage › should handle query/hash correctly during query updating #a ?hello=
● Production Usage › should not show target deprecation warning
● Production Usage › should respond with 405 for POST to static page
● Production Usage › should contain generated page count in output
● Production Usage › should output traces
● Production Usage › should not contain currentScript usage for publicPath
● Production Usage › should not contain amp, rsc APIs in main chunk
● Production Usage › With basic usage › should render the page
● Production Usage › With basic usage › should polyfill Node.js modules
● Production Usage › With basic usage › should allow etag header support
● Production Usage › With basic usage › should allow etag header support with getStaticProps
● Production Usage › With basic usage › should allow etag header support with getServerSideProps
● Production Usage › With basic usage › should have X-Powered-By header support
● Production Usage › With basic usage › should render 404 for routes that do not exist
● Production Usage › With basic usage › should render 404 for /_next/static route
● Production Usage › With basic usage › should render 200 for POST on page
● Production Usage › With basic usage › should render 404 for POST on missing page
● Production Usage › With basic usage › should render 404 for _next routes that do not exist
● Production Usage › With basic usage › should render 404 even if the HTTP method is not GET or HEAD
● Production Usage › With basic usage › should render 404 for dotfiles in /static
● Production Usage › With basic usage › should return 405 method on static then GET and HEAD
● Production Usage › With basic usage › should return 412 on static file when If-Unmodified-Since is provided and file is modified
● Production Usage › With basic usage › should return 200 on static file if If-Unmodified-Since is invalid date
● Production Usage › With basic usage › should set Content-Length header
● Production Usage › With basic usage › should set Cache-Control header
● Production Usage › With basic usage › should set correct Cache-Control header for static 404s
● Production Usage › With basic usage › should block special pages
● Production Usage › With basic usage › should not contain customServer in NEXT_DATA
● Production Usage › API routes › should work with pages/api/index.js
● Production Usage › API routes › should work with pages/api/hello.js
● Production Usage › API routes › does not work with pages/api/readfile-dirname.js
● Production Usage › API routes › should work with pages/api/readfile-processcwd.js
● Production Usage › API routes › should work with dynamic params and search string
● Production Usage › With navigation › should navigate via client side
● Production Usage › With navigation › should navigate to nested index via client side
● Production Usage › With navigation › should reload page successfully (on bad link)
● Production Usage › With navigation › should reload page successfully (on bad data fetch)
● Production Usage › should navigate to external site and back
● Production Usage › should navigate to page with CSS and back
● Production Usage › should navigate to external site and back (with query)
● Production Usage › should change query correctly
● Production Usage › Runtime errors › should render a server side error on the client side
● Production Usage › Runtime errors › should render a client side component error
● Production Usage › Runtime errors › should call getInitialProps on _error page during a client side component error
● Production Usage › Misc › should handle already finished responses
● Production Usage › Misc › should allow to access /static/ and /_next/
● Production Usage › Misc › Should allow access to public files
● Production Usage › Misc › should reload the page on page script error
● Production Usage › Misc › should have default runtime values when not defined
● Production Usage › Misc › should not have runtimeConfig in NEXT_DATA
● Production Usage › Misc › should add autoExport for auto pre-rendered pages
● Production Usage › Misc › should not add autoExport for non pre-rendered pages
● Production Usage › Misc › should add prefetch tags when Link prefetch prop is used
● Production Usage › Misc › It does not add a timestamp to link tags with prefetch attribute
● Production Usage › Misc › should reload the page on page script error with prefetch
● Production Usage › should not expose the compiled page file in development
● Production Usage › should not put backslashes in pages-manifest.json
● Production Usage › should handle failed param decoding
● Production Usage › should replace static pages with HTML files
● Production Usage › should not replace non-static pages with HTML files
● Production Usage › should handle AMP correctly in IE
● Production Usage › should warn when prefetch is true
● Production Usage › should not emit stats
● Production Usage › should contain the Next.js version in window export
● Production Usage › should clear all core performance marks
● Production Usage › should not clear custom performance marks
● Production Usage › should have defer on all script tags
● Production Usage › should only have one DOCTYPE
● Production Usage › should preserve query when hard navigating from page 404
● Production Usage › should remove placeholder for next/image correctly
● Production Usage › Dynamic import › default behavior › should render dynamic import components
● Production Usage › Dynamic import › default behavior › should render one dynamically imported component and load its css files
● Production Usage › Dynamic import › default behavior › should render three dynamically imported components and load their css files
● Production Usage › Dynamic import › default behavior › should bundle two css modules for one dynamically imported component into one css file
● Production Usage › Dynamic import › default behavior › should bundle two css modules for nested components into one css file
● Production Usage › Dynamic import › default behavior › should not remove css styles for same css file between page transitions
● Production Usage › Dynamic import › default behavior › should output two css files even in case of three css module files while one is shared across files
● Production Usage › Dynamic import › default behavior › should render one dynamically imported component without any css files
● Production Usage › Dynamic import › default behavior › should render even there are no physical chunk exists
● Production Usage › Dynamic import › ssr:false option › should not render loading on the server side
● Production Usage › Dynamic import › ssr:false option › should render the component on client side
● Production Usage › Dynamic import › ssr:true option › should render the component on the server side
● Production Usage › Dynamic import › ssr:true option › should render the component on client side
● Production Usage › Dynamic import › custom loading › should render custom loading on the server side when
● Production Usage › Dynamic import › custom loading › should render the component on client side
● Production Usage › process.env › should set process.env.NODE_ENV in production
● Production Usage › process.browser › should eliminate server only code on the client
● Production Usage › process.browser › should eliminate client only code on the server
● Production Usage › With Security Related Issues › should handle invalid URL properly
● Production Usage › With Security Related Issues › should only access files inside .next directory
● Production Usage › With Security Related Issues › should not allow accessing files outside .next/static directory
● Production Usage › With Security Related Issues › should not leak the user's home directory into the build
● Production Usage › With Security Related Issues › should prevent URI based XSS attacks
● Production Usage › With Security Related Issues › should prevent URI based XSS attacks using single quotes
● Production Usage › With Security Related Issues › should prevent URI based XSS attacks using double quotes
● Production Usage › With Security Related Issues › should prevent URI based XSS attacks using semicolons and double quotes
● Production Usage › With Security Related Issues › should prevent URI based XSS attacks using semicolons and single quotes
● Production Usage › With Security Related Issues › should prevent URI based XSS attacks using src
● Production Usage › With Security Related Issues › should prevent URI based XSS attacks using querystring
● Production Usage › With Security Related Issues › should prevent URI based XSS attacks using querystring and quotes
● Production Usage › With Security Related Issues › should handle encoded value in the pathname correctly \
● Production Usage › With Security Related Issues › should handle encoded value in the pathname correctly %
● Production Usage › With Security Related Issues › should handle encoded value in the query correctly
● Production Usage › With Security Related Issues › should handle encoded value in the pathname correctly /
● Production Usage › With Security Related Issues › should handle encoded value in the pathname to query correctly (/)
● Production Usage › With Security Related Issues › should handle encoded / value for trailing slash correctly
● Production Usage › With Security Related Issues › should not execute script embedded inside svg image, even if dangerouslyAllowSVG=true
Read more about building and testing Next.js in contributing.md.
Expand output● ReactRefreshLogBox app default › server component can recover from error thrown in the module
Read more about building and testing Next.js in contributing.md. |
Stats from current PRDefault BuildGeneral
Client Bundles (main, webpack)
Legacy Client Bundles (polyfills)
Client Pages
Client Build Manifests
Rendered Page Sizes
Edge SSR bundle Size
Middleware size
Next Runtimes
build cache
Diff detailsDiff for main-HASH.jsDiff too large to display |
Note
The "Test new tests for flakes" may fail due to the number of tests in a single
hmr.test.ts
file.The follow-up stacks are to resolve the issue.
Why?
The test util
check
is deprecated internally, recommended to replace toretry
.