Skip to content

Upgrade React from 040f8286-20250402 to 540cd652-20250403 #77836

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

Closed

Conversation

@ijjk
Copy link
Member

ijjk commented Apr 4, 2025

Failing test suites

Commit: 42b0577

pnpm test test/integration/amp-export-validation/test/index.test.js

  • AMP Validation on Export > production mode > should have shown errors during build
Expand output

● AMP Validation on Export › production mode › should have shown errors during build

expect(received).toMatch(expected)

Expected pattern: /error.*The mandatory attribute 'height' is missing in tag 'amp-video'\./
Received string:  "   Loading config from /root/actions-runner/_work/next.js/next.js/test/integration/amp-export-validation/next.config.js
   Loading config from /root/actions-runner/_work/next.js/next.js/test/integration/amp-export-validation/next.config.js
   ▲ Next.js 15.3.0-canary.36·
   Checking validity of types ...
   Creating an optimized production build ...
   Loading config from /root/actions-runner/_work/next.js/next.js/test/integration/amp-export-validation/next.config.js
   Loading config from /root/actions-runner/_work/next.js/next.js/test/integration/amp-export-validation/next.config.js
   Loading config from /root/actions-runner/_work/next.js/next.js/test/integration/amp-export-validation/next.config.js
 ✓ Compiled successfully in 2000ms
   Collecting page data ...
   Generating static pages (0/8) ...
   Generating static pages (2/8)··
 ⚠ Linting is disabled.
Error occurred prerendering page \"/dog-cat\". Read more: https://nextjs.org/docs/messages/prerender-error
AssertionError: Assertion failed: WebAssembly is uninitialized

  at new module$contents$goog$asserts_AssertionError (../evalmachine.<anonymous>:106:1695)
  at module$contents$goog$asserts_doAssertFailure (../evalmachine.<anonymous>:107:354)
  at goog.asserts.assertExists (../evalmachine.<anonymous>:109:142)
  at Object.module$contents$amp$validator_validateString [as validateString] (../evalmachine.<anonymous>:2663:108)
  at Validator.validateString (../packages/next/dist/compiled/amphtml-validator/index.js:1:20650)
  at validateAmp (../packages/next/dist/export/routes/pages.js:89:34)
  at async exportPagesPage (../packages/next/dist/export/routes/pages.js:106:13)
  at async Span.traceAsyncFn (../packages/next/dist/trace/trace.js:157:20)
  at async exportPage (../packages/next/dist/export/worker.js:357:18)
  Export encountered an error on /dog-cat, exiting the build.
   ⨯ Next.js build worker exited with code: 1 and signal: null
  "
  at Object.toMatch (integration/amp-export-validation/test/index.test.js:29:29)

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

__NEXT_EXPERIMENTAL_PPR=true pnpm test-dev test/development/app-dir/owner-stack-invalid-element-type/owner-stack-invalid-element-type.test.ts (PPR)

  • app-dir - owner-stack-invalid-element-type > should catch invalid element from a browser only component
Expand output

● app-dir - owner-stack-invalid-element-type › should catch invalid element from a browser only component

expect(received).toMatchInlineSnapshot(snapshot)

Snapshot name: `app-dir - owner-stack-invalid-element-type should catch invalid element from a browser only component 1`

- Snapshot  - 16
+ Received  +  1

- {
-   "count": 1,
-   "description": "Error: Element type is invalid: expected a string (for built-in components) or a class/function (for composite components) but got: object. You likely forgot to export your component from the file it's defined in, or you might have mixed up default and named imports.
-
- Check the render method of `BrowserOnly`.",
-   "environmentLabel": null,
-   "label": "Runtime Error",
-   "source": "app/browser/browser-only.js (8:7) @ BrowserOnly
- >  8 |       <Foo />
-      |       ^",
-   "stack": [
-     "BrowserOnly app/browser/browser-only.js (8:7)",
-     "Inner app/browser/page.js (11:10)",
-     "Page app/browser/page.js (15:10)",
-   ],
- }
+ "Expected Redbox but found no visible one."

   9 |     const browser = await next.browser('/browser')
  10 |
> 11 |     await expect(browser).toDisplayRedbox(`
     |                           ^
  12 |      {
  13 |        "count": 1,
  14 |        "description": "Error: Element type is invalid: expected a string (for built-in components) or a class/function (for composite components) but got: object. You likely forgot to export your component from the file it's defined in, or you might have mixed up default and named imports.

  at Object.toDisplayRedbox (development/app-dir/owner-stack-invalid-element-type/owner-stack-invalid-element-type.test.ts:11:27)

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

__NEXT_EXPERIMENTAL_PPR=true pnpm test-dev test/e2e/app-dir/app-external/app-external.test.ts (PPR)

  • app dir - external dependency > mixed syntax external modules > should handle mixed module with next/dynamic
Expand output

● app dir - external dependency › mixed syntax external modules › should handle mixed module with next/dynamic

page.waitForSelector: Timeout 60000ms exceeded.
Call log:
  - waiting for locator('#component')

  431 |     return this.chain(() => {
  432 |       return page
> 433 |         .waitForSelector(selector, { timeout, state: 'attached' })
      |          ^
  434 |         .then(async (el) => {
  435 |           // it seems selenium waits longer and tests rely on this behavior
  436 |           // so we wait for the load event fire before returning

  at waitForSelector (lib/browsers/playwright.ts:433:10)
  at BrowserInterface.chain (lib/browsers/base.ts:17:23)
  at BrowserInterface.chain [as waitForElementByCss] (lib/browsers/playwright.ts:431:17)
  at BrowserInterface.waitForElementByCss [as elementByCss] (lib/browsers/playwright.ts:351:17)
  at Object.elementByCss (e2e/app-dir/app-external/app-external.test.ts:238:28)
  at Proxy.chain (lib/browsers/base.ts:17:23)
  at Proxy.chain (lib/browsers/playwright.ts:363:17)
  at Object.text (e2e/app-dir/app-external/app-external.test.ts:238:55)

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

pnpm test test/integration/next-image-legacy/basic/test/index.test.ts

  • Image Component Tests > production mode > SSR Lazy Loading Tests > should have loaded the first image immediately
  • Image Component Tests > production mode > SSR Lazy Loading Tests > should not have loaded the second image immediately
  • Image Component Tests > production mode > SSR Lazy Loading Tests > should pass through classes on a lazy loaded image
  • Image Component Tests > production mode > SSR Lazy Loading Tests > should load the second image after scrolling down
  • Image Component Tests > production mode > SSR Lazy Loading Tests > should not have loaded the third image after scrolling down
  • Image Component Tests > production mode > SSR Lazy Loading Tests > should load the third image, which is unoptimized, after scrolling further down
  • Image Component Tests > production mode > SSR Lazy Loading Tests > should load the fourth image lazily after scrolling down
  • Image Component Tests > production mode > SSR Lazy Loading Tests > should load the fifth image eagerly, without scrolling
  • Image Component Tests > production mode > SSR Lazy Loading Tests > should load the sixth image, which has lazyBoundary property after scrolling down
Expand output

● Image Component Tests › production mode › SSR Lazy Loading Tests › should have loaded the first image immediately

thrown: "Exceeded timeout of 60000 ms for a hook.
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."

  383 |       })
  384 |       describe('SSR Lazy Loading Tests', () => {
> 385 |         beforeAll(async () => {
      |         ^
  386 |           browser = await webdriver(appPort, '/lazy')
  387 |         })
  388 |         afterAll(async () => {

  at beforeAll (integration/next-image-legacy/basic/test/index.test.ts:385:9)
  at describe (integration/next-image-legacy/basic/test/index.test.ts:384:7)
  at integration/next-image-legacy/basic/test/index.test.ts:267:56
  at Object.describe (integration/next-image-legacy/basic/test/index.test.ts:266:1)

● Image Component Tests › production mode › SSR Lazy Loading Tests › should not have loaded the second image immediately

thrown: "Exceeded timeout of 60000 ms for a hook.
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."

  383 |       })
  384 |       describe('SSR Lazy Loading Tests', () => {
> 385 |         beforeAll(async () => {
      |         ^
  386 |           browser = await webdriver(appPort, '/lazy')
  387 |         })
  388 |         afterAll(async () => {

  at beforeAll (integration/next-image-legacy/basic/test/index.test.ts:385:9)
  at describe (integration/next-image-legacy/basic/test/index.test.ts:384:7)
  at integration/next-image-legacy/basic/test/index.test.ts:267:56
  at Object.describe (integration/next-image-legacy/basic/test/index.test.ts:266:1)

● Image Component Tests › production mode › SSR Lazy Loading Tests › should pass through classes on a lazy loaded image

thrown: "Exceeded timeout of 60000 ms for a hook.
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."

  383 |       })
  384 |       describe('SSR Lazy Loading Tests', () => {
> 385 |         beforeAll(async () => {
      |         ^
  386 |           browser = await webdriver(appPort, '/lazy')
  387 |         })
  388 |         afterAll(async () => {

  at beforeAll (integration/next-image-legacy/basic/test/index.test.ts:385:9)
  at describe (integration/next-image-legacy/basic/test/index.test.ts:384:7)
  at integration/next-image-legacy/basic/test/index.test.ts:267:56
  at Object.describe (integration/next-image-legacy/basic/test/index.test.ts:266:1)

● Image Component Tests › production mode › SSR Lazy Loading Tests › should load the second image after scrolling down

thrown: "Exceeded timeout of 60000 ms for a hook.
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."

  383 |       })
  384 |       describe('SSR Lazy Loading Tests', () => {
> 385 |         beforeAll(async () => {
      |         ^
  386 |           browser = await webdriver(appPort, '/lazy')
  387 |         })
  388 |         afterAll(async () => {

  at beforeAll (integration/next-image-legacy/basic/test/index.test.ts:385:9)
  at describe (integration/next-image-legacy/basic/test/index.test.ts:384:7)
  at integration/next-image-legacy/basic/test/index.test.ts:267:56
  at Object.describe (integration/next-image-legacy/basic/test/index.test.ts:266:1)

● Image Component Tests › production mode › SSR Lazy Loading Tests › should not have loaded the third image after scrolling down

thrown: "Exceeded timeout of 60000 ms for a hook.
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."

  383 |       })
  384 |       describe('SSR Lazy Loading Tests', () => {
> 385 |         beforeAll(async () => {
      |         ^
  386 |           browser = await webdriver(appPort, '/lazy')
  387 |         })
  388 |         afterAll(async () => {

  at beforeAll (integration/next-image-legacy/basic/test/index.test.ts:385:9)
  at describe (integration/next-image-legacy/basic/test/index.test.ts:384:7)
  at integration/next-image-legacy/basic/test/index.test.ts:267:56
  at Object.describe (integration/next-image-legacy/basic/test/index.test.ts:266:1)

● Image Component Tests › production mode › SSR Lazy Loading Tests › should load the third image, which is unoptimized, after scrolling further down

thrown: "Exceeded timeout of 60000 ms for a hook.
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."

  383 |       })
  384 |       describe('SSR Lazy Loading Tests', () => {
> 385 |         beforeAll(async () => {
      |         ^
  386 |           browser = await webdriver(appPort, '/lazy')
  387 |         })
  388 |         afterAll(async () => {

  at beforeAll (integration/next-image-legacy/basic/test/index.test.ts:385:9)
  at describe (integration/next-image-legacy/basic/test/index.test.ts:384:7)
  at integration/next-image-legacy/basic/test/index.test.ts:267:56
  at Object.describe (integration/next-image-legacy/basic/test/index.test.ts:266:1)

● Image Component Tests › production mode › SSR Lazy Loading Tests › should load the fourth image lazily after scrolling down

thrown: "Exceeded timeout of 60000 ms for a hook.
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."

  383 |       })
  384 |       describe('SSR Lazy Loading Tests', () => {
> 385 |         beforeAll(async () => {
      |         ^
  386 |           browser = await webdriver(appPort, '/lazy')
  387 |         })
  388 |         afterAll(async () => {

  at beforeAll (integration/next-image-legacy/basic/test/index.test.ts:385:9)
  at describe (integration/next-image-legacy/basic/test/index.test.ts:384:7)
  at integration/next-image-legacy/basic/test/index.test.ts:267:56
  at Object.describe (integration/next-image-legacy/basic/test/index.test.ts:266:1)

● Image Component Tests › production mode › SSR Lazy Loading Tests › should load the fifth image eagerly, without scrolling

thrown: "Exceeded timeout of 60000 ms for a hook.
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."

  383 |       })
  384 |       describe('SSR Lazy Loading Tests', () => {
> 385 |         beforeAll(async () => {
      |         ^
  386 |           browser = await webdriver(appPort, '/lazy')
  387 |         })
  388 |         afterAll(async () => {

  at beforeAll (integration/next-image-legacy/basic/test/index.test.ts:385:9)
  at describe (integration/next-image-legacy/basic/test/index.test.ts:384:7)
  at integration/next-image-legacy/basic/test/index.test.ts:267:56
  at Object.describe (integration/next-image-legacy/basic/test/index.test.ts:266:1)

● Image Component Tests › production mode › SSR Lazy Loading Tests › should load the sixth image, which has lazyBoundary property after scrolling down

thrown: "Exceeded timeout of 60000 ms for a hook.
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."

  383 |       })
  384 |       describe('SSR Lazy Loading Tests', () => {
> 385 |         beforeAll(async () => {
      |         ^
  386 |           browser = await webdriver(appPort, '/lazy')
  387 |         })
  388 |         afterAll(async () => {

  at beforeAll (integration/next-image-legacy/basic/test/index.test.ts:385:9)
  at describe (integration/next-image-legacy/basic/test/index.test.ts:384:7)
  at integration/next-image-legacy/basic/test/index.test.ts:267:56
  at Object.describe (integration/next-image-legacy/basic/test/index.test.ts:266:1)

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

__NEXT_EXPERIMENTAL_PPR=true pnpm test-dev test/development/acceptance-app/rsc-build-errors.test.ts (PPR)

  • Error overlay - RSC build errors > should error for invalid undefined module retuning from next dynamic
Expand output

● Error overlay - RSC build errors › should error for invalid undefined module retuning from next dynamic

Expected Redbox but found no visible one.

  323 |     )
  324 |
> 325 |     await session.assertHasRedbox()
      |     ^
  326 |     expect(await session.getRedboxDescription()).toInclude(
  327 |       `Element type is invalid. Received a promise that resolves to: undefined. Lazy element type must resolve to a class or function.`
  328 |     )

  at Object.<anonymous> (development/acceptance-app/rsc-build-errors.test.ts:325:5)

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

__NEXT_EXPERIMENTAL_PPR=true pnpm test-dev test/e2e/app-dir/actions/app-action.test.ts (PPR)

  • app-dir action handling > should support next/dynamic with ssr: false
  • app-dir action handling > should support next/dynamic with ssr: false (edge)
Expand output

● app-dir action handling › should support next/dynamic with ssr: false

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

  554 |   })
  555 |
> 556 |   it('should support next/dynamic with ssr: false', async () => {
      |   ^
  557 |     const browser = await next.browser('/dynamic-csr')
  558 |
  559 |     await retry(async () => {

  at it (e2e/app-dir/actions/app-action.test.ts:556:3)
  at Object.describe (e2e/app-dir/actions/app-action.test.ts:19:1)

● app-dir action handling › should support next/dynamic with ssr: false (edge)

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

  568 |   })
  569 |
> 570 |   it('should support next/dynamic with ssr: false (edge)', async () => {
      |   ^
  571 |     const browser = await next.browser('/dynamic-csr/edge')
  572 |
  573 |     await retry(async () => {

  at it (e2e/app-dir/actions/app-action.test.ts:570:3)
  at Object.describe (e2e/app-dir/actions/app-action.test.ts:19:1)

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

@eps1lon eps1lon closed this Apr 7, 2025
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 21, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants