Skip to content

Conversation

@lukesandberg
Copy link
Contributor

@lukesandberg lukesandberg commented Nov 18, 2025

possibly broken by #85835 ?

@ijjk ijjk added created-by: Turbopack team PRs by the Turbopack team. tests labels Nov 18, 2025
Copy link
Contributor Author

lukesandberg commented Nov 18, 2025

@lukesandberg lukesandberg changed the title Fix a stale snapshot [turbopack] Fix a stale nft snapshot Nov 18, 2025
@lukesandberg lukesandberg marked this pull request as ready for review November 18, 2025 22:27
@lukesandberg lukesandberg requested a review from a team November 18, 2025 22:27
@ijjk
Copy link
Member

ijjk commented Nov 19, 2025

Failing test suites

Commit: cbdffd0 | About building and testing Next.js

pnpm test-start test/e2e/app-dir/app-prefetch/prefetching.stale-times.test.ts (job)

  • app dir - prefetching (custom staleTime) > should not re-fetch cached data when navigating back to a route group (DD)
Expand output

● app dir - prefetching (custom staleTime) › should not re-fetch cached data when navigating back to a route group

thrown: "Exceeded timeout of 120000 ms for a test.
Add a timeout value to this test to increase the timeout, if this is a long-running test. See https://jestjs.io/docs/api#testname-fn-timeout."

  128 |   })
  129 |
> 130 |   it('should not re-fetch cached data when navigating back to a route group', async () => {
      |   ^
  131 |     let act: ReturnType<typeof createRouterAct>
  132 |     // Just installing so that the page doesn't automatically move past dynamic stale time
  133 |     createTimeController()

  at it (e2e/app-dir/app-prefetch/prefetching.stale-times.test.ts:130:3)
  at Object.describe (e2e/app-dir/app-prefetch/prefetching.stale-times.test.ts:6:1)

@mischnic
Copy link
Member

#86288 is merged already

@mischnic mischnic closed this Nov 19, 2025
@github-actions github-actions bot added the locked label Dec 4, 2025
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 4, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants