-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
test(nextjs): Add route handler tests for turbopack #17515
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
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.
we can make this test.fail so it will expect to fail and we will be "notified" when this is fixed! (+ add a comment for this)
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.
test.fail seems to not work with test timeouts – I added a workaround so the test fails once the issue is resolved upstream
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.
Nevermind, this is already fixed in canary. leaving as is
node-overhead report 🧳Note: This is a synthetic benchmark with a minimal express app and does not necessarily reflect the real-world performance impact in an application.
|
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
Adds tests for tracing and exception handling in turbopack route handlers.
Note: this will fail until vercel/next.js#82612 is resolved