mirror of
https://github.com/nodejs/node.git
synced 2025-08-16 14:18:44 +02:00
test: fix edge snapshot stack traces
PR-URL: https://github.com/nodejs/node/pull/49659 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com> Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
This commit is contained in:
parent
7e12d0e16d
commit
a71e3a65bb
5 changed files with 29 additions and 21 deletions
|
@ -11,14 +11,15 @@ const skipForceColors =
|
|||
(common.isWindows && (Number(os.release().split('.')[0]) !== 10 || Number(os.release().split('.')[2]) < 14393)); // See https://github.com/nodejs/node/pull/33132
|
||||
|
||||
|
||||
function replaceNodeVersion(str) {
|
||||
return str.replaceAll(process.version, '*');
|
||||
}
|
||||
|
||||
function replaceStackTrace(str) {
|
||||
return snapshot.replaceStackTrace(str, '$1at *$7\n');
|
||||
}
|
||||
|
||||
function replaceForceColorsStackTrace(str) {
|
||||
// eslint-disable-next-line no-control-regex
|
||||
return str.replaceAll(/(\[90m\W+)at .*node:.*/g, '$1at *[39m');
|
||||
}
|
||||
|
||||
describe('errors output', { concurrency: true }, () => {
|
||||
function normalize(str) {
|
||||
return str.replaceAll(snapshot.replaceWindowsPaths(process.cwd()), '')
|
||||
|
@ -35,9 +36,12 @@ describe('errors output', { concurrency: true }, () => {
|
|||
}
|
||||
const common = snapshot
|
||||
.transform(snapshot.replaceWindowsLineEndings, snapshot.replaceWindowsPaths);
|
||||
const defaultTransform = snapshot.transform(common, normalize, replaceNodeVersion);
|
||||
const errTransform = snapshot.transform(common, normalizeNoNumbers, replaceNodeVersion);
|
||||
const promiseTransform = snapshot.transform(common, replaceStackTrace, normalizeNoNumbers, replaceNodeVersion);
|
||||
const defaultTransform = snapshot.transform(common, normalize, snapshot.replaceNodeVersion);
|
||||
const errTransform = snapshot.transform(common, normalizeNoNumbers, snapshot.replaceNodeVersion);
|
||||
const promiseTransform = snapshot.transform(common, replaceStackTrace,
|
||||
normalizeNoNumbers, snapshot.replaceNodeVersion);
|
||||
const forceColorsTransform = snapshot.transform(common, normalize,
|
||||
replaceForceColorsStackTrace, snapshot.replaceNodeVersion);
|
||||
|
||||
const tests = [
|
||||
{ name: 'errors/async_error_eval_cjs.js' },
|
||||
|
@ -57,7 +61,8 @@ describe('errors output', { concurrency: true }, () => {
|
|||
{ name: 'errors/throw_in_line_with_tabs.js', transform: errTransform },
|
||||
{ name: 'errors/throw_non_error.js', transform: errTransform },
|
||||
{ name: 'errors/promise_always_throw_unhandled.js', transform: promiseTransform },
|
||||
{ skip: skipForceColors, name: 'errors/force_colors.js', env: { FORCE_COLOR: 1 } },
|
||||
{ skip: skipForceColors, name: 'errors/force_colors.js',
|
||||
transform: forceColorsTransform, env: { FORCE_COLOR: 1 } },
|
||||
];
|
||||
for (const { name, transform = defaultTransform, env, skip = false } of tests) {
|
||||
it(name, { skip }, async () => {
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue