test: line numbers in eval are not reliably source mapped

Re: TypeStrong#2009 (comment)
isaacs · Sep 5, 2023 · fa45fb9 · fa45fb9
1 parent b695140
commit fa45fb9
Showing 1 changed file with 7 additions and 7 deletions.
diff --git a/src/test/repl/repl-environment.spec.ts b/src/test/repl/repl-environment.spec.ts
@@ -160,7 +160,7 @@ test.suite('[eval], <repl>, and [stdin] execute with correct globals', (test) =>
         modulePaths,
         exportsTest: true,
         // Note: vanilla node uses different name. See #1360
-        stackTest: expect.stringContaining(`    at ${join(TEST_DIR, `[stdin].ts`)}:1:`),
+        stackTest: expect.stringContaining(`    at ${join(TEST_DIR, `[stdin].ts`)}:`),
         moduleAccessorsTest: null,
         argv: [tsNodeExe],
       },
@@ -237,7 +237,7 @@ test.suite('[eval], <repl>, and [stdin] execute with correct globals', (test) =>
         modulePaths: [...modulePaths],
         exportsTest: true,
         // Note: vanilla node uses different name. See #1360
-        stackTest: expect.stringContaining(`    at ${join(TEST_DIR, `[eval].ts`)}:1:`),
+        stackTest: expect.stringContaining(`    at ${join(TEST_DIR, `[eval].ts`)}:`),
         moduleAccessorsTest: true,
         argv: [tsNodeExe],
       },
@@ -262,7 +262,7 @@ test.suite('[eval], <repl>, and [stdin] execute with correct globals', (test) =>
         modulePaths,
         exportsTest: true,
         // Note: vanilla node uses different name. See #1360
-        stackTest: expect.stringContaining(`    at ${join(TEST_DIR, `[eval].ts`)}:1:`),
+        stackTest: expect.stringContaining(`    at ${join(TEST_DIR, `[eval].ts`)}:`),
         moduleAccessorsTest: true,
         argv: [tsNodeExe, './repl/script.js'],
       },
@@ -287,7 +287,7 @@ test.suite('[eval], <repl>, and [stdin] execute with correct globals', (test) =>
         modulePaths,
         exportsTest: true,
         // Note: vanilla node uses different name. See #1360
-        stackTest: expect.stringContaining(`    at ${join(TEST_DIR, `[eval].ts`)}:1:`),
+        stackTest: expect.stringContaining(`    at ${join(TEST_DIR, `[eval].ts`)}:`),
         moduleAccessorsTest: true,
         argv: [tsNodeExe, './does-not-exist.js'],
       },
@@ -312,7 +312,7 @@ test.suite('[eval], <repl>, and [stdin] execute with correct globals', (test) =>
         modulePaths,
         exportsTest: true,
         // Note: vanilla node uses different name. See #1360
-        stackTest: expect.stringContaining(`    at ${join(TEST_DIR, `[eval].ts`)}:1:`),
+        stackTest: expect.stringContaining(`    at ${join(TEST_DIR, `[eval].ts`)}:`),
         moduleAccessorsTest: true,
         argv: [tsNodeExe],
       },
@@ -404,7 +404,7 @@ test.suite('[eval], <repl>, and [stdin] execute with correct globals', (test) =>
           // moduleAccessorsTest: true,
 
           // Note: vanilla node uses different name. See #1360
-          stackTest: expect.stringContaining(`    at ${join(TEST_DIR, replFile)}:1:`),
+          stackTest: expect.stringContaining(`    at ${join(TEST_DIR, replFile)}:`),
         },
       });
     }
@@ -433,7 +433,7 @@ test.suite('[eval], <repl>, and [stdin] execute with correct globals', (test) =>
           // Note: vanilla node REPL does not set exports
           exportsTest: true,
           // Note: vanilla node uses different name. See #1360
-          stackTest: expect.stringContaining(`    at ${join(TEST_DIR, replFile)}:1:`),
+          stackTest: expect.stringContaining(`    at ${join(TEST_DIR, replFile)}:`),
           moduleAccessorsTest: true,
         },
       });