Skip to content

Commit

Permalink
Skip Math.random test with route handler and 'use cache' (#71025)
Browse files Browse the repository at this point in the history
Using `'use cache'` in route handlers is not yet fully supported, and
fails in isolation, e.g. in a deployment test, because it implicitly
relies on the fact that a page must have been compiled before the route.
Otherwise rendering the route errors with "Missing manifest for Server
Actions.".

This deploy test failure slipped through because #70837 was created from
a fork.
  • Loading branch information
unstubbable authored Oct 9, 2024
1 parent b772ab4 commit 9497f17
Showing 1 changed file with 2 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ describe('Node Extensions', () => {
describe('Dynamic IO', () => {
const { next, skipped } = nextTestSetup({
files: __dirname + '/fixtures/random/dynamic-io',
skipDeployment: true,
})

if (skipped) {
Expand Down Expand Up @@ -64,7 +65,7 @@ describe('Node Extensions', () => {
expect($('li').length).toBe(2)
})

it('should not error when accessing routes that use Math.random() in App Router', async () => {
it.skip('should not error when accessing routes that use Math.random() in App Router', async () => {
let res, body

res = await next.fetch('/app/prerendered/uncached/api')
Expand Down

0 comments on commit 9497f17

Please sign in to comment.