test: unify assertSnapshot common patterns

test/common/assertSnapshot.js

@@ -4,13 +4,14 @@ const path = require('node:path');
 const test = require('node:test');
 const fs = require('node:fs/promises');
 const assert = require('node:assert/strict');
+const { pathToFileURL } = require('node:url');
 const { hostname } = require('node:os');
 
 const stackFramesRegexp = /(?<=\n)(\s+)((.+?)\s+\()?(?:\(?(.+?):(\d+)(?::(\d+))?)\)?(\s+\{)?(\[\d+m)?(\n|$)/g;
 const windowNewlineRegexp = /\r/g;
 
 function replaceNodeVersion(str) {
-  return str.replaceAll(process.version, '*');
+  return str.replaceAll(process.version, '<node-version>');
 }
 
 function replaceStackTrace(str, replacement = '$1*$7$8\n') {
@@ -31,10 +32,28 @@ function replaceWindowsPaths(str) {
   return common.isWindows ? str.replaceAll(path.win32.sep, path.posix.sep) : str;
 }
 
+function replaceTrailingSpaces(str) {
+  return str.replaceAll(/[\t ]+\n/g, '\n');
+}
+
+// Replaces customized or platform specific executable names to be `node`.
+function generalizeExeName(str) {
+  const baseName = path.basename(process.argv0 || 'node', '.exe');
+  return str.replaceAll(`${baseName} --`, 'node --');
+}
+
+function replaceWarningPid(str) {
+  return str.replaceAll(/\(node:\d+\)/g, '(node:<pid>)');
+}
+
 function transformProjectRoot(replacement = '') {
   const projectRoot = path.resolve(__dirname, '../..');
+  // Handles URL encoded project root in file URL strings as well.
+  const urlEncoded = pathToFileURL(projectRoot).pathname;
   return (str) => {
-    return str.replaceAll('\\\'', "'").replaceAll(projectRoot, replacement);
+    return str.replaceAll('\\\'', "'")
+      .replaceAll(projectRoot, replacement)
+      .replaceAll(urlEncoded, replacement);
   };
 }
 
@@ -152,32 +171,41 @@ function pickTestFileFromLcov(str) {
   );
 }
 
-const defaultTransform = transform(
+// Transforms basic patterns like:
+// - platform specific path and line endings,
+// - line trailing spaces,
+// - executable specific path and versions.
+const basicTransform = transform(
   replaceWindowsLineEndings,
-  replaceStackTrace,
+  replaceTrailingSpaces,
   removeWindowsPathEscaping,
-  transformProjectRoot(),
   replaceWindowsPaths,
+  replaceNodeVersion,
+  generalizeExeName,
+  replaceWarningPid,
+);
+
+const defaultTransform = transform(
+  basicTransform,
+  replaceStackTrace,
+  transformProjectRoot(),
   replaceTestDuration,
   replaceTestLocationLine,
 );
 const specTransform = transform(
   replaceSpecDuration,
-  replaceWindowsLineEndings,
+  basicTransform,
   replaceStackTrace,
-  replaceWindowsPaths,
 );
 const junitTransform = transform(
   replaceJunitDuration,
-  replaceWindowsLineEndings,
+  basicTransform,
   replaceStackTrace,
-  replaceWindowsPaths,
 );
 const lcovTransform = transform(
-  replaceWindowsLineEndings,
+  basicTransform,
   replaceStackTrace,
   transformProjectRoot(),
-  replaceWindowsPaths,
   pickTestFileFromLcov,
 );
 
@@ -204,6 +232,7 @@ module.exports = {
   transform,
   transformProjectRoot,
   replaceTestDuration,
+  basicTransform,
   defaultTransform,
   specTransform,
   junitTransform,

test/fixtures/console/stack_overflow.snapshot

@@ -1,8 +1,8 @@
 before
-*test*fixtures*console*stack_overflow.js:*
+/test/fixtures/console/stack_overflow.js:*
 JSON.stringify(array);
      ^
 
 [RangeError: Maximum call stack size exceeded]
 
-Node.js *
+Node.js <node-version>

test/fixtures/errors/async_error_eval_esm.snapshot

@@ -1,7 +1,7 @@
 Error: test
-    at one (file:*/[eval1]:2:9)
-    at two (file:*/[eval1]:15:9)
-    at async three (file:*/[eval1]:18:3)
-    at async four (file:*/[eval1]:22:3)
-    at async main (file:*/[eval1]:28:5)
+    at one (file:///[eval1]:2:9)
+    at two (file:///[eval1]:15:9)
+    at async three (file:///[eval1]:18:3)
+    at async four (file:///[eval1]:22:3)
+    at async main (file:///[eval1]:28:5)
 

test/fixtures/errors/async_error_microtask_main.snapshot

@@ -1,6 +1,6 @@
 Error: test
-    at one (*fixtures*async-error.js:4:9)
-    at two (*fixtures*async-error.js:17:9)
-    at async three (*fixtures*async-error.js:20:3)
-    at async four (*fixtures*async-error.js:24:3)
-    at async main (*async_error_microtask_main.js:7:5)
+    at one (/test/fixtures/async-error.js:4:9)
+    at two (/test/fixtures/async-error.js:17:9)
+    at async three (/test/fixtures/async-error.js:20:3)
+    at async four (/test/fixtures/async-error.js:24:3)
+    at async main (/test/fixtures/errors/async_error_microtask_main.js:7:5)

test/fixtures/errors/async_error_nexttick_main.snapshot

@@ -1,7 +1,7 @@
 Error: test
-    at one (*fixtures*async-error.js:4:9)
-    at two (*fixtures*async-error.js:17:9)
-    at process.processTicksAndRejections (node:internal*process*task_queues:104:5)
-    at async three (*fixtures*async-error.js:20:3)
-    at async four (*fixtures*async-error.js:24:3)
-    at async main (*async_error_nexttick_main.js:7:5)
+    at one (/test/fixtures/async-error.js:4:9)
+    at two (/test/fixtures/async-error.js:17:9)
+    at process.processTicksAndRejections (node:internal/process/task_queues:104:5)
+    at async three (/test/fixtures/async-error.js:20:3)
+    at async four (/test/fixtures/async-error.js:24:3)
+    at async main (/test/fixtures/errors/async_error_nexttick_main.js:7:5)

test/fixtures/errors/async_error_sync_esm.snapshot

@@ -1,6 +1,6 @@
 Error: test
-    at one (*fixtures*async-error.js:4:9)
-    at two (*fixtures*async-error.js:17:9)
-    at async three (*fixtures*async-error.js:20:3)
-    at async four (*fixtures*async-error.js:24:3)
-    at async main (file:*/async_error_sync_esm.mjs:6:5)
+    at one (/test/fixtures/async-error.js:4:9)
+    at two (/test/fixtures/async-error.js:17:9)
+    at async three (/test/fixtures/async-error.js:20:3)
+    at async four (/test/fixtures/async-error.js:24:3)
+    at async main (file:///test/fixtures/errors/async_error_sync_esm.mjs:6:5)

test/fixtures/errors/async_error_sync_main.snapshot

@@ -1,6 +1,6 @@
 Error: test
-    at one (*fixtures*async-error.js:4:9)
-    at two (*fixtures*async-error.js:17:9)
-    at async three (*fixtures*async-error.js:20:3)
-    at async four (*fixtures*async-error.js:24:3)
-    at async main (*async_error_sync_main.js:7:5)
+    at one (/test/fixtures/async-error.js:4:9)
+    at two (/test/fixtures/async-error.js:17:9)
+    at async three (/test/fixtures/async-error.js:20:3)
+    at async four (/test/fixtures/async-error.js:24:3)
+    at async main (/test/fixtures/errors/async_error_sync_main.js:7:5)

test/fixtures/errors/core_line_numbers.snapshot

@@ -5,6 +5,6 @@ node:punycode:54
 RangeError: Invalid input
     at error (node:punycode:54:8)
     at Object.decode (node:punycode:247:5)
-    at Object.<anonymous> (*core_line_numbers.js:13:10)
+    at Object.<anonymous> (/test/fixtures/errors/core_line_numbers.js:13:10)
 
-Node.js *
+Node.js <node-version>

test/fixtures/errors/error_aggregateTwoErrors.snapshot

@@ -1,20 +1,20 @@
-*error_aggregateTwoErrors.js:*
+/test/fixtures/errors/error_aggregateTwoErrors.js:*
 throw aggregateTwoErrors(err, originalError);
 ^
 
 AggregateError: original
-    at Object.<anonymous> (*error_aggregateTwoErrors.js:*:*) {
+    at Object.<anonymous> (/test/fixtures/errors/error_aggregateTwoErrors.js:*:*) {
   code: 'ERR0',
   [errors]: [
     Error: original
-        at Object.<anonymous> (*error_aggregateTwoErrors.js:*:*) {
+        at Object.<anonymous> (/test/fixtures/errors/error_aggregateTwoErrors.js:*:*) {
       code: 'ERR0'
     },
     Error: second error
-        at Object.<anonymous> (*error_aggregateTwoErrors.js:*:*) {
+        at Object.<anonymous> (/test/fixtures/errors/error_aggregateTwoErrors.js:*:*) {
       code: 'ERR1'
     }
   ]
 }
 
-Node.js *
+Node.js <node-version>

test/fixtures/errors/error_exit.snapshot

@@ -1,13 +1,13 @@
 Exiting with code=1
-node:internal*assert*utils:*
+node:internal/assert/utils:*
   throw error;
   ^
 
 AssertionError [ERR_ASSERTION]: Expected values to be strictly equal:
 
 1 !== 2
 
-    at Object.<anonymous> (*error_exit.js:*:*) {
+    at Object.<anonymous> (/test/fixtures/errors/error_exit.js:*:*) {
   generatedMessage: true,
   code: 'ERR_ASSERTION',
   actual: 1,
@@ -16,4 +16,4 @@ AssertionError [ERR_ASSERTION]: Expected values to be strictly equal:
   diff: 'simple'
 }
 
-Node.js *
+Node.js <node-version>

test/fixtures/errors/events_unhandled_error_common_trace.snapshot

@@ -1,12 +1,12 @@
 node:events:*
-      throw er; * Unhandled 'error' event
+      throw er; // Unhandled 'error' event
       ^
 
 Error: foo:bar
-    at bar (*events_unhandled_error_common_trace.js:*:*)
-    at foo (*events_unhandled_error_common_trace.js:*:*)
+    at bar (/test/fixtures/errors/events_unhandled_error_common_trace.js:*:*)
+    at foo (/test/fixtures/errors/events_unhandled_error_common_trace.js:*:*)
 Emitted 'error' event at:
-    at quux (*events_unhandled_error_common_trace.js:*:*)
-    at Object.<anonymous> (*events_unhandled_error_common_trace.js:*:*)
+    at quux (/test/fixtures/errors/events_unhandled_error_common_trace.js:*:*)
+    at Object.<anonymous> (/test/fixtures/errors/events_unhandled_error_common_trace.js:*:*)
 
-Node.js *
+Node.js <node-version>

test/fixtures/errors/events_unhandled_error_nexttick.snapshot

@@ -1,10 +1,10 @@
 node:events:*
-      throw er; * Unhandled 'error' event
+      throw er; // Unhandled 'error' event
       ^
 
 Error
-    at Object.<anonymous> (*events_unhandled_error_nexttick.js:*:*)
+    at Object.<anonymous> (/test/fixtures/errors/events_unhandled_error_nexttick.js:*:*)
 Emitted 'error' event at:
-    at *events_unhandled_error_nexttick.js:*:*
+    at /test/fixtures/errors/events_unhandled_error_nexttick.js:*:*
 
-Node.js *
+Node.js <node-version>

test/fixtures/errors/events_unhandled_error_sameline.snapshot

@@ -1,10 +1,10 @@
 node:events:*
-      throw er; * Unhandled 'error' event
+      throw er; // Unhandled 'error' event
       ^
 
 Error
-    at Object.<anonymous> (*events_unhandled_error_sameline.js:*:*)
+    at Object.<anonymous> (/test/fixtures/errors/events_unhandled_error_sameline.js:*:*)
 Emitted 'error' event at:
-    at Object.<anonymous> (*events_unhandled_error_sameline.js:*:*)
+    at Object.<anonymous> (/test/fixtures/errors/events_unhandled_error_sameline.js:*:*)
 
-Node.js *
+Node.js <node-version>

test/fixtures/errors/events_unhandled_error_subclass.snapshot

@@ -1,10 +1,10 @@
 node:events:*
-      throw er; * Unhandled 'error' event
+      throw er; // Unhandled 'error' event
       ^
 
 Error
-    at Object.<anonymous> (*events_unhandled_error_subclass.js:*:*)
+    at Object.<anonymous> (/test/fixtures/errors/events_unhandled_error_subclass.js:*:*)
 Emitted 'error' event on Foo instance at:
-    at Object.<anonymous> (*events_unhandled_error_subclass.js:*:*)
+    at Object.<anonymous> (/test/fixtures/errors/events_unhandled_error_subclass.js:*:*)
 
-Node.js *
+Node.js <node-version>

test/fixtures/errors/force_colors.snapshot

@@ -1,9 +1,9 @@
-*force_colors.js:2
+/test/fixtures/errors/force_colors.js:2
 throw new Error('Should include grayed stack trace');
 ^
 
 Error: Should include grayed stack trace
-    at Object.<anonymous> (/test*force_colors.js:2:7)
+    at Object.<anonymous> (/test/fixtures/errors/force_colors.js:2:7)
     at *
     at *
     at *
@@ -12,4 +12,4 @@ Error: Should include grayed stack trace
     at *
     at *
 
-Node.js *
+Node.js <node-version>

test/fixtures/errors/if-error-has-good-stack.snapshot

@@ -3,24 +3,24 @@ node:assert:*
     ^
 
 AssertionError [ERR_ASSERTION]: ifError got unwanted exception: test error
-    at z (*if-error-has-good-stack.js:*:*)
-    at y (*if-error-has-good-stack.js:*:*)
-    at x (*if-error-has-good-stack.js:*:*)
-    at Object.<anonymous> (*if-error-has-good-stack.js:*:*)
-    at c (*if-error-has-good-stack.js:*:*)
-    at b (*if-error-has-good-stack.js:*:*)
-    at a (*if-error-has-good-stack.js:*:*)
-    at Object.<anonymous> (*if-error-has-good-stack.js:*:*) {
+    at z (/test/fixtures/errors/if-error-has-good-stack.js:*:*)
+    at y (/test/fixtures/errors/if-error-has-good-stack.js:*:*)
+    at x (/test/fixtures/errors/if-error-has-good-stack.js:*:*)
+    at Object.<anonymous> (/test/fixtures/errors/if-error-has-good-stack.js:*:*)
+    at c (/test/fixtures/errors/if-error-has-good-stack.js:*:*)
+    at b (/test/fixtures/errors/if-error-has-good-stack.js:*:*)
+    at a (/test/fixtures/errors/if-error-has-good-stack.js:*:*)
+    at Object.<anonymous> (/test/fixtures/errors/if-error-has-good-stack.js:*:*) {
   generatedMessage: false,
   code: 'ERR_ASSERTION',
   actual: Error: test error
-      at c (*if-error-has-good-stack.js:*:*)
-      at b (*if-error-has-good-stack.js:*:*)
-      at a (*if-error-has-good-stack.js:*:*)
-      at Object.<anonymous> (*if-error-has-good-stack.js:*:*),
+      at c (/test/fixtures/errors/if-error-has-good-stack.js:*:*)
+      at b (/test/fixtures/errors/if-error-has-good-stack.js:*:*)
+      at a (/test/fixtures/errors/if-error-has-good-stack.js:*:*)
+      at Object.<anonymous> (/test/fixtures/errors/if-error-has-good-stack.js:*:*),
   expected: null,
   operator: 'ifError',
   diff: 'simple'
 }
 
-Node.js *
+Node.js <node-version>

test/fixtures/errors/promise_always_throw_unhandled.snapshot

@@ -1,4 +1,4 @@
-*promise_always_throw_unhandled.js:*
+/test/fixtures/errors/promise_always_throw_unhandled.js:*
   throw new Error('One');
         ^
 
@@ -14,4 +14,4 @@ Error: One
     at *
     at *
 
-Node.js *
+Node.js <node-version>

test/fixtures/errors/promise_unhandled_warn_with_error.snapshot

@@ -1,4 +1,4 @@
-(node:*) UnhandledPromiseRejectionWarning: Error: alas
+(node:<pid>) UnhandledPromiseRejectionWarning: Error: alas
     at *
     at *
     at *
@@ -7,5 +7,5 @@
     at *
     at *
     at *
-(Use `* --trace-warnings ...` to show where the warning was created)
-(node:*) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https:*nodejs.org*api*cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
+(Use `node --trace-warnings ...` to show where the warning was created)
+(node:<pid>) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)

test/fixtures/errors/test-no-extra-info-on-fatal-exception.snapshot

@@ -1,6 +1,6 @@
-*test-no-extra-info-on-fatal-exception.js:7
+/test/fixtures/errors/test-no-extra-info-on-fatal-exception.js:7
 throw new Error('foo');
 ^
 
 Error: foo
-    at Object.<anonymous> (*test-no-extra-info-on-fatal-exception.js:7:7)
+    at Object.<anonymous> (/test/fixtures/errors/test-no-extra-info-on-fatal-exception.js:7:7)