-
Notifications
You must be signed in to change notification settings - Fork 27.5k
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
Update Turbopack development test manifest #66023
Closed
+133
−27
Closed
Changes from all commits
Commits
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -428,6 +428,22 @@ | |
"flakey": [], | ||
"runtimeError": false | ||
}, | ||
"packages/next/src/server/after/after-context.test.ts": { | ||
"passed": [ | ||
"createAfterContext does not hang forever if onClose failed", | ||
"createAfterContext runs after() callbacks added within an after()", | ||
"createAfterContext runs after() callbacks from a run() callback that resolves", | ||
"createAfterContext runs after() callbacks from a run() callback that streams", | ||
"createAfterContext runs after() callbacks from a run() callback that throws", | ||
"createAfterContext runs all after() callbacks even if some of them threw", | ||
"createAfterContext throws from after() if onClose is not provided", | ||
"createAfterContext throws from after() if waitUntil is not provided" | ||
], | ||
"failed": [], | ||
"pending": [], | ||
"flakey": [], | ||
"runtimeError": false | ||
}, | ||
"packages/next/src/server/app-render/csrf-protection.test.ts": { | ||
"passed": [ | ||
"isCsrfOriginAllowed should return false when allowedOrigins contains originDomain with non-matching pattern", | ||
|
@@ -453,6 +469,17 @@ | |
"flakey": [], | ||
"runtimeError": false | ||
}, | ||
"packages/next/src/server/base-http/web.test.ts": { | ||
"passed": [ | ||
"WebNextResponse onClose doesn't track onClose unless enabled", | ||
"WebNextResponse onClose stream body", | ||
"WebNextResponse onClose string body" | ||
], | ||
"failed": [], | ||
"pending": [], | ||
"flakey": [], | ||
"runtimeError": false | ||
}, | ||
"packages/next/src/server/config.test.ts": { | ||
"passed": [ | ||
"loadConfig nextConfig.images defaults should assign a `images.remotePatterns` when using assetPrefix", | ||
|
@@ -1471,7 +1498,8 @@ | |
"test/development/acceptance/server-component-compiler-errors-in-pages.test.ts": { | ||
"passed": [ | ||
"Error Overlay for server components compiler errors in pages importing 'next/headers' in pages", | ||
"Error Overlay for server components compiler errors in pages importing 'server-only' in pages" | ||
"Error Overlay for server components compiler errors in pages importing 'server-only' in pages", | ||
"Error Overlay for server components compiler errors in pages importing unstable_after from 'next/server' in pages" | ||
], | ||
"failed": [], | ||
"pending": [], | ||
|
@@ -4282,6 +4310,7 @@ | |
"app dir - navigation hash should scroll to the specified hash", | ||
"app dir - navigation hash-link-back-to-same-page should scroll to the specified hash", | ||
"app dir - navigation hash-with-scroll-offset should scroll to the specified hash", | ||
"app dir - navigation middleware redirect should change browser location when router.refresh() gets a redirect response", | ||
"app dir - navigation navigating to a page with async metadata should render the final state of the page with correct metadata", | ||
"app dir - navigation navigating to dynamic params & changing the casing should load the page correctly", | ||
"app dir - navigation navigation between pages and app should not contain _rsc query while navigating from app to pages", | ||
|
@@ -4325,6 +4354,54 @@ | |
"flakey": [], | ||
"runtimeError": false | ||
}, | ||
"test/e2e/app-dir/next-after-app/index.test.ts": { | ||
"passed": [ | ||
"unstable_after() in edge runtime does not allow modifying cookies in a callback", | ||
"unstable_after() in edge runtime interrupted RSC renders runs callbacks if a user error was thrown in the RSC render", | ||
"unstable_after() in edge runtime interrupted RSC renders runs callbacks if notFound() was called", | ||
"unstable_after() in edge runtime interrupted RSC renders runs callbacks if redirect() was called", | ||
"unstable_after() in edge runtime invalid usages errors at compile time when used in a client module", | ||
"unstable_after() in edge runtime invalid usages errors at compile time with dynamic = \"error\"", | ||
"unstable_after() in edge runtime invalid usages errors at compile time with dynamic = \"force-static\"", | ||
"unstable_after() in edge runtime only runs callbacks after the response is fully sent", | ||
"unstable_after() in edge runtime runs callbacks from nested unstable_after calls", | ||
"unstable_after() in edge runtime runs in dynamic pages", | ||
"unstable_after() in edge runtime runs in dynamic route handlers", | ||
"unstable_after() in edge runtime runs in generateMetadata()", | ||
"unstable_after() in edge runtime runs in middleware", | ||
"unstable_after() in edge runtime runs in server actions", | ||
"unstable_after() in nodejs runtime does not allow modifying cookies in a callback", | ||
"unstable_after() in nodejs runtime interrupted RSC renders runs callbacks if a user error was thrown in the RSC render", | ||
"unstable_after() in nodejs runtime interrupted RSC renders runs callbacks if notFound() was called", | ||
"unstable_after() in nodejs runtime interrupted RSC renders runs callbacks if redirect() was called", | ||
"unstable_after() in nodejs runtime invalid usages errors at compile time when used in a client module", | ||
"unstable_after() in nodejs runtime invalid usages errors at compile time with dynamic = \"error\"", | ||
"unstable_after() in nodejs runtime invalid usages errors at compile time with dynamic = \"force-static\"", | ||
"unstable_after() in nodejs runtime only runs callbacks after the response is fully sent", | ||
"unstable_after() in nodejs runtime runs callbacks from nested unstable_after calls", | ||
"unstable_after() in nodejs runtime runs in dynamic pages", | ||
"unstable_after() in nodejs runtime runs in dynamic route handlers", | ||
"unstable_after() in nodejs runtime runs in generateMetadata()", | ||
"unstable_after() in nodejs runtime runs in middleware", | ||
"unstable_after() in nodejs runtime runs in server actions" | ||
], | ||
"failed": [], | ||
"pending": [], | ||
"flakey": [], | ||
"runtimeError": false | ||
}, | ||
"test/e2e/app-dir/next-after-pages/index.test.ts": { | ||
"passed": [ | ||
"unstable_after() - pages invalid usages errors at compile time when used in pages dir errors when used in getServerSideProps", | ||
"unstable_after() - pages invalid usages errors at compile time when used in pages dir errors when used in getStaticProps", | ||
"unstable_after() - pages invalid usages errors at compile time when used in pages dir errors when used in within a page component", | ||
"unstable_after() - pages runs in middleware" | ||
], | ||
"failed": [], | ||
"pending": [], | ||
"flakey": [], | ||
"runtimeError": false | ||
}, | ||
"test/e2e/app-dir/next-config/index.test.ts": { | ||
"passed": [ | ||
"app dir - next config should support importing webpack in next.config" | ||
|
@@ -4462,6 +4539,19 @@ | |
"flakey": [], | ||
"runtimeError": false | ||
}, | ||
"test/e2e/app-dir/options-request/options-request.test.ts": { | ||
"passed": [ | ||
"options-request should 404 for an OPTIONS request to a non-existent route", | ||
"options-request should respond with a 200 + response body when invoking a pages API route with an OPTIONS request", | ||
"options-request should respond with a 204 No Content when invoking an app route handler with an OPTIONS request", | ||
"options-request should return a 400 status code when invoking /app-page with an OPTIONS request", | ||
"options-request should return a 400 status code when invoking /pages-page with an OPTIONS request" | ||
], | ||
"failed": [], | ||
"pending": [], | ||
"flakey": [], | ||
"runtimeError": false | ||
}, | ||
"test/e2e/app-dir/pages-to-app-routing/pages-to-app-routing.test.ts": { | ||
"passed": ["pages-to-app-routing should work using browser"], | ||
"failed": [], | ||
|
@@ -4871,6 +4961,15 @@ | |
"flakey": [], | ||
"runtimeError": false | ||
}, | ||
"test/e2e/app-dir/prefetch-searchparam/prefetch-searchparam.test.ts": { | ||
"passed": [ | ||
"prefetch-searchparam should set prefetch cache properly on different search params" | ||
], | ||
"failed": [], | ||
"pending": [], | ||
"flakey": [], | ||
"runtimeError": false | ||
}, | ||
"test/e2e/app-dir/prefetching-not-found/prefetching-not-found.test.ts": { | ||
"passed": [ | ||
"prefetching-not-found should correctly navigate to/from a global 404 page when following links with prefetch=auto" | ||
|
@@ -8708,53 +8807,49 @@ | |
"runtimeError": false | ||
}, | ||
"test/integration/create-next-app/package-manager/bun.test.ts": { | ||
"passed": [ | ||
"should use bun for --use-bun flag with example", | ||
"should use bun when user-agent is bun", | ||
"should use bun when user-agent is bun with example" | ||
], | ||
"passed": [], | ||
"failed": [], | ||
"pending": [ | ||
"create-next-app with package manager bun should use bun for --use-bun flag" | ||
"create-next-app with package manager bun should use bun for --use-bun flag", | ||
"create-next-app with package manager bun should use bun for --use-bun flag with example", | ||
"create-next-app with package manager bun should use bun when user-agent is bun", | ||
"create-next-app with package manager bun should use bun when user-agent is bun with example" | ||
], | ||
"flakey": [], | ||
"runtimeError": false | ||
}, | ||
"test/integration/create-next-app/package-manager/npm.test.ts": { | ||
"passed": [ | ||
"should use npm for --use-npm flag with example", | ||
"should use npm when user-agent is npm", | ||
"should use npm when user-agent is npm with example" | ||
], | ||
"passed": [], | ||
"failed": [], | ||
"pending": [ | ||
"create-next-app with package manager npm should use npm for --use-npm flag" | ||
"create-next-app with package manager npm should use npm for --use-npm flag", | ||
"create-next-app with package manager npm should use npm for --use-npm flag with example", | ||
"create-next-app with package manager npm should use npm when user-agent is npm", | ||
"create-next-app with package manager npm should use npm when user-agent is npm with example" | ||
], | ||
"flakey": [], | ||
"runtimeError": false | ||
}, | ||
"test/integration/create-next-app/package-manager/pnpm.test.ts": { | ||
"passed": [ | ||
"should use pnpm for --use-pnpm flag with example", | ||
"should use pnpm when user-agent is pnpm", | ||
"should use pnpm when user-agent is pnpm with example" | ||
], | ||
"passed": [], | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. One more regression |
||
"failed": [], | ||
"pending": [ | ||
"create-next-app with package manager pnpm should use pnpm for --use-pnpm flag" | ||
"create-next-app with package manager pnpm should use pnpm for --use-pnpm flag", | ||
"create-next-app with package manager pnpm should use pnpm for --use-pnpm flag with example", | ||
"create-next-app with package manager pnpm should use pnpm when user-agent is pnpm", | ||
"create-next-app with package manager pnpm should use pnpm when user-agent is pnpm with example" | ||
], | ||
"flakey": [], | ||
"runtimeError": false | ||
}, | ||
"test/integration/create-next-app/package-manager/yarn.test.ts": { | ||
"passed": [ | ||
"should use yarn for --use-yarn flag with example", | ||
"should use yarn when user-agent is yarn", | ||
"should use yarn when user-agent is yarn with example" | ||
], | ||
"passed": [], | ||
"failed": [], | ||
"pending": [ | ||
"create-next-app with package manager yarn should use yarn for --use-yarn flag" | ||
"create-next-app with package manager yarn should use yarn for --use-yarn flag", | ||
"create-next-app with package manager yarn should use yarn for --use-yarn flag with example", | ||
"create-next-app with package manager yarn should use yarn when user-agent is yarn", | ||
"create-next-app with package manager yarn should use yarn when user-agent is yarn with example" | ||
], | ||
"flakey": [], | ||
"runtimeError": false | ||
|
@@ -8780,7 +8875,8 @@ | |
"create-next-app --app (App Router) should create TypeScript project with --ts flag", | ||
"create-next-app --app (App Router) should create an empty TailwindCSS project with --empty flag", | ||
"create-next-app --app (App Router) should create an empty project with --empty flag", | ||
"create-next-app --app (App Router) should create project inside \"src\" directory with --src-dir flag" | ||
"create-next-app --app (App Router) should create project inside \"src\" directory with --src-dir flag", | ||
"create-next-app --app (App Router) should enable turbopack dev with --turbo flag" | ||
], | ||
"flakey": [], | ||
"runtimeError": false | ||
|
@@ -8794,7 +8890,8 @@ | |
"create-next-app --no-app (Pages Router) should create TypeScript project with --ts flag", | ||
"create-next-app --no-app (Pages Router) should create an empty TailwindCSS project with --empty flag", | ||
"create-next-app --no-app (Pages Router) should create an empty project with --empty flag", | ||
"create-next-app --no-app (Pages Router) should create project inside \"src\" directory with --src-dir flag" | ||
"create-next-app --no-app (Pages Router) should create project inside \"src\" directory with --src-dir flag", | ||
"create-next-app --no-app (Pages Router) should enable turbopack dev with --turbo flag" | ||
], | ||
"flakey": [], | ||
"runtimeError": false | ||
|
@@ -15645,6 +15742,15 @@ | |
"flakey": [], | ||
"runtimeError": false | ||
}, | ||
"test/integration/root-catchall-cache/test/index.test.js": { | ||
"passed": [], | ||
"failed": [], | ||
"pending": [ | ||
"Root Catch-all Cache production mode should cache / correctly" | ||
], | ||
"flakey": [], | ||
"runtimeError": false | ||
}, | ||
"test/integration/root-optional-revalidate/test/index.test.js": { | ||
"passed": [], | ||
"failed": [], | ||
|
Oops, something went wrong.
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.
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.
Is this correct?
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.
describe
above them was skipped in #65845https://github.com/vercel/next.js/blame/af49a5b54c06fc2b5c505fc4bce2dd64657b8551/test/integration/create-next-app/package-manager/bun.test.ts#L29
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.
Those are
create-next-app
tests, and at #65926, I moved someit
tests that were outside thedescribe
function without intended (I know it because I'm the one that made a mistake 😅).As mentioned above, the whole tests were
describe.skip
, and should've moved topending
before!