Skip to content

Commit

Permalink
test: line numbers in eval are not reliably source mapped
Browse files Browse the repository at this point in the history
  • Loading branch information
isaacs committed Sep 5, 2023
1 parent b695140 commit fa45fb9
Showing 1 changed file with 7 additions and 7 deletions.
14 changes: 7 additions & 7 deletions src/test/repl/repl-environment.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -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],
},
Expand Down Expand Up @@ -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],
},
Expand All @@ -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'],
},
Expand All @@ -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'],
},
Expand All @@ -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],
},
Expand Down Expand Up @@ -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)}:`),
},
});
}
Expand Down Expand Up @@ -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,
},
});
Expand Down

0 comments on commit fa45fb9

Please sign in to comment.