Update error-recovery test snapshots for alternative bundler [2/n]

vercel · Feb 23, 2025 · 83e983b · 83e983b
1 parent 49c26c7
commit 83e983b
Show file tree

Hide file tree

Showing 2 changed files with 48 additions and 8 deletions.
diff --git a/test/development/acceptance/__snapshots__/error-recovery.test.ts.snap b/test/development/acceptance/__snapshots__/error-recovery.test.ts.snap
@@ -1,6 +1,6 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
-exports[`ReactRefreshLogBox default syntax > runtime error 1`] = `
+exports[`ReactRefreshLogBox turbo syntax > runtime error: isWin32-false--isRspack-false 1`] = `
 "index.js (5:9) @ eval
 
   3 | setInterval(() => {
@@ -12,8 +12,8 @@ exports[`ReactRefreshLogBox default syntax > runtime error 1`] = `
   8 |   return <div />"
 `;
 
-exports[`ReactRefreshLogBox turbo syntax > runtime error 1`] = `
-"index.js (5:9) @ eval
+exports[`ReactRefreshLogBox default syntax > runtime error: isWin32-false--isRspack-true 1`] = `
+"index.js (5:9) @ <unknown>
 
   3 | setInterval(() => {
   4 |   i++

diff --git a/test/development/acceptance/error-recovery.test.ts b/test/development/acceptance/error-recovery.test.ts
@@ -409,11 +409,9 @@ describe.each(['default', 'turbo'])('ReactRefreshLogBox %s', () => {
 
     await new Promise((resolve) => setTimeout(resolve, 1000))
     await session.assertHasRedbox()
-    if (process.platform === 'win32') {
-      expect(await session.getRedboxSource()).toMatchSnapshot()
-    } else {
-      expect(await session.getRedboxSource()).toMatchSnapshot()
-    }
+    expect(await session.getRedboxSource()).toMatchSnapshot(
+      `isWin32-${process.platform === 'win32'}--isRspack-${Boolean(process.env.NEXT_RSPACK)}`
+    )
 
     // Make a syntax error.
     await session.patch(
@@ -446,6 +444,27 @@ describe.each(['default', 'turbo'])('ReactRefreshLogBox %s', () => {
 
        Expected '}', got '<eof>'"
       `)
+    } else if (process.env.NEXT_RSPACK) {
+      expect(redboxSource).toMatchInlineSnapshot(`
+       "./index.js
+         × Module build failed:
+         ├─▶   ×
+         │     │   x Expected '}', got '<eof>'
+         │     │    ,-[TEST_DIR/index.js:7:1]
+         │     │  4 |   i++
+         │     │  5 |   throw Error('no ' + i)
+         │     │  6 | }, 1000)
+         │     │  7 | export default function FunctionNamed() {
+         │     │    :                                         ^
+         │     │    \`----
+         │     │
+         │
+         ╰─▶ Syntax Error
+
+       Import trace for requested module:
+       ./index.js
+       ./pages/index.js"
+      `)
     } else {
       expect(redboxSource).toMatchInlineSnapshot(`
         "./index.js
@@ -483,6 +502,27 @@ describe.each(['default', 'turbo'])('ReactRefreshLogBox %s', () => {
 
        Expected '}', got '<eof>'"
       `)
+    } else if (process.env.NEXT_RSPACK) {
+      expect(redboxSource).toMatchInlineSnapshot(`
+       "./index.js
+         × Module build failed:
+         ├─▶   ×
+         │     │   x Expected '}', got '<eof>'
+         │     │    ,-[TEST_DIR/index.js:7:1]
+         │     │  4 |   i++
+         │     │  5 |   throw Error('no ' + i)
+         │     │  6 | }, 1000)
+         │     │  7 | export default function FunctionNamed() {
+         │     │    :                                         ^
+         │     │    \`----
+         │     │
+         │
+         ╰─▶ Syntax Error
+
+       Import trace for requested module:
+       ./index.js
+       ./pages/index.js"
+      `)
     } else {
       expect(redboxSource).toMatchInlineSnapshot(`
         "./index.js