test: also check source code for node:internal related errors (#62542)

* Still check if the origin line thrown nodejs errors could still
display source code
* Need to separate the different error into different test otherwise the
previous error will be preserved

Closes NEXT-2597

---------

Co-authored-by: Balázs Orbán <[email protected]>

Author: huozhi

test/development/acceptance-app/ReactRefreshLogBox.test.ts

@@ -839,17 +839,17 @@ describe.each(['default', 'turbo'])('ReactRefreshLogBox app %s', () => {
     await cleanup()
   })
 
-  test('useless frames are hidden in stack trace', async () => {
+  test('should hide unrelated frames in stack trace with unknown anonymous calls', async () => {
     const { session, browser, cleanup } = await sandbox(
       next,
       new Map([
         [
           'app/page.js',
+          // TODO: repro stringify (<anonymous>)
           outdent`
         export default function Page() {
           const e = new Error("Boom!");
           e.stack += \`
-          // REVIEW: how to reliably test the presence of these stack frames?
           at stringify (<anonymous>)
           at <unknown> (<anonymous>)
           at foo (bar:1:1)\`;
@@ -869,23 +869,37 @@ describe.each(['default', 'turbo'])('ReactRefreshLogBox app %s', () => {
     expect(texts).not.toContain('<unknown>\n<anonymous>')
     expect(texts).toContain('foo\nbar (1:1)')
 
-    // Test that node:internal errors should be hidden
+    await cleanup()
+  })
 
-    next.patchFile(
-      'app/page.js',
-      // Node.js will throw an error about the invalid URL since this is a server component
-      outdent`
-      export default function Page() {
-        new URL("/", "invalid");
-      }`
+  test('should hide unrelated frames in stack trace with node:internal calls', async () => {
+    const { session, browser, cleanup } = await sandbox(
+      next,
+      new Map([
+        [
+          'app/page.js',
+          // Node.js will throw an error about the invalid URL since this is a server component
+          outdent`
+          export default function Page() {
+            new URL("/", "invalid");
+          }`,
+        ],
+      ])
     )
 
     expect(await session.hasRedbox()).toBe(true)
     await expandCallStack(browser)
-    callStackFrames = await browser.elementsByCss(
+
+    // Should still show the errored line in source code
+    const source = await session.getRedboxSource()
+    expect(source).toContain('app/page.js')
+    expect(source).toContain(`new URL("/", "invalid")`)
+
+    await expandCallStack(browser)
+    const callStackFrames = await browser.elementsByCss(
       '[data-nextjs-call-stack-frame]'
     )
-    texts = await Promise.all(callStackFrames.map((f) => f.innerText()))
+    const texts = await Promise.all(callStackFrames.map((f) => f.innerText()))
 
     expect(texts.filter((t) => t.includes('node:internal'))).toHaveLength(0)
 

test/development/acceptance/ReactRefreshLogBox.test.ts

@@ -784,17 +784,17 @@ describe.each(['default', 'turbo'])('ReactRefreshLogBox %s', () => {
     await cleanup()
   })
 
-  test('useless frames are hidden in stack trace for pages error', async () => {
+  test('should hide unrelated frames in stack trace with unknown anonymous calls', async () => {
     const { session, browser, cleanup } = await sandbox(
       next,
       new Map([
         [
           'pages/index.js',
+          // TODO: repro stringify (<anonymous>)
           outdent`
           export default function Page() {
             const e = new Error("Client error!");
             e.stack += \`
-              // REVIEW: how to reliably test the presence of these stack frames?
               at stringify (<anonymous>)
               at <unknown> (<anonymous>)
               at foo (bar:1:1)\`;
@@ -814,26 +814,38 @@ describe.each(['default', 'turbo'])('ReactRefreshLogBox %s', () => {
     expect(texts).not.toContain('<unknown>\n<anonymous>')
     expect(texts).toContain('foo\nbar (1:1)')
 
-    // Test that node:internal errors should be hidden
+    await cleanup()
+  })
 
-    next.patchFile(
-      'pages/index.js',
-      // Node.js will throw an error about the invalid URL since it happens server-side
-      outdent`
+  test('should hide unrelated frames in stack trace with node:internal calls', async () => {
+    const { session, browser, cleanup } = await sandbox(
+      next,
+      new Map([
+        [
+          'pages/index.js',
+          // Node.js will throw an error about the invalid URL since it happens server-side
+          outdent`
       export default function Page() {}
       
       export function getServerSideProps() {
         new URL("/", "invalid");
         return { props: {} };
-      }`
+      }`,
+        ],
+      ])
     )
-
     expect(await session.hasRedbox()).toBe(true)
     await expandCallStack(browser)
-    callStackFrames = await browser.elementsByCss(
+
+    // Should still show the errored line in source code
+    const source = await session.getRedboxSource()
+    expect(source).toContain('pages/index.js')
+    expect(source).toContain(`new URL("/", "invalid")`)
+
+    const callStackFrames = await browser.elementsByCss(
       '[data-nextjs-call-stack-frame]'
     )
-    texts = await Promise.all(callStackFrames.map((f) => f.innerText()))
+    const texts = await Promise.all(callStackFrames.map((f) => f.innerText()))
 
     expect(texts.filter((t) => t.includes('node:internal'))).toHaveLength(0)