Skip to content
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

failing test for infinite loading bug #75726

Closed
wants to merge 1 commit into from

Conversation

ztanner
Copy link
Member

@ztanner ztanner commented Feb 6, 2025

Failing test for #72104

@ijjk ijjk added created-by: Next.js team PRs by the Next.js team. tests labels Feb 6, 2025
Copy link
Member Author

ztanner commented Feb 6, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@ijjk
Copy link
Member

ijjk commented Feb 6, 2025

Failing test suites

Commit: b604173

pnpm test-dev test/e2e/app-dir/client-component-deduping/client-component-deduping.test.ts

  • client-component-deduping > should load the client component
Expand output

● client-component-deduping › should load the client component

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."

   6 |   })
   7 |
>  8 |   it('should load the client component', async () => {
     |   ^
   9 |     const html = await next.render('/')
  10 |     expect(html).toContain('hello world')
  11 |   })

  at it (e2e/app-dir/client-component-deduping/client-component-deduping.test.ts:8:3)
  at Object.describe (e2e/app-dir/client-component-deduping/client-component-deduping.test.ts:3:1)

Read more about building and testing Next.js in contributing.md.

@ztanner
Copy link
Member Author

ztanner commented Feb 10, 2025

Closing in favor of the fix & test added to React in facebook/react#32316

@ztanner ztanner closed this Feb 10, 2025
@ztanner ztanner deleted the 02-05-failing_test_for_infinite_loading_bug branch February 10, 2025 18:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
created-by: Next.js team PRs by the Next.js team. tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants