test: fix edge snapshot stack traces

PR-URL: #49659 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Moshe Atlow <[email protected]>

Author: GeoffreyBooth

test/common/assertSnapshot.js

@@ -8,6 +8,10 @@ const assert = require('node:assert/strict');
 conststackFramesRegexp=/(?<=\n)(\s+)((.+?)\s+\()?(?:\(?(.+?):(\d+)(?::(\d+))?)\)?(\s+\{)?(\[\d+m)?(\n|$)/g;
 constwindowNewlineRegexp=/\r/g;
 
+functionreplaceNodeVersion(str){
+returnstr.replaceAll(process.version,'*');
+}
+
 functionreplaceStackTrace(str,replacement='$1*$7$8\n'){
 returnstr.replace(stackFramesRegexp,replacement);
 }
@@ -84,6 +88,7 @@ module.exports ={
  assertSnapshot,
  getSnapshotPath,
  replaceFullPaths,
+ replaceNodeVersion,
  replaceStackTrace,
  replaceWindowsLineEndings,
  replaceWindowsPaths,

test/fixtures/errors/force_colors.snapshot

@@ -4,11 +4,11 @@ throw new Error('Should include grayed stack trace')
 
 Error: Should include grayed stack trace
  at Object.<anonymous> (/test*force_colors.js:1:7)
- at Module._compile (node:internal*modules*cjs*loader:1264:14)
- at Module._extensions..js (node:internal*modules*cjs*loader:1318:10)
- at Module.load (node:internal*modules*cjs*loader:1113:32)
- at Module._load (node:internal*modules*cjs*loader:954:12)
- at Function.executeUserEntryPoint [as runMain] (node:internal*modules*run_main:80:12)
- at node:internal*main*run_main_module:23:47
+ at *
+ at *
+ at *
+ at *
+ at *
+ at *
 
 Node.js *

test/fixtures/source-map/output/source_map_sourcemapping_url_string.snapshot

@@ -1,3 +1,3 @@
 Error: an exception.
  at Object.<anonymous> (*typescript-sourcemapping_url_string.ts:3:7)
-at Module._compile (node:internal*modules*cjs*loader:1264:14)
+*

test/parallel/test-node-output-errors.mjs

@@ -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
 
 
-functionreplaceNodeVersion(str){
-returnstr.replaceAll(process.version,'*');
-}
-
 functionreplaceStackTrace(str){
 returnsnapshot.replaceStackTrace(str,'$1at *$7\n');
 }
 
+functionreplaceForceColorsStackTrace(str){
+// eslint-disable-next-line no-control-regex
+returnstr.replaceAll(/(\[90m\W+)at.*node:.*/g,'$1at *');
+}
+
 describe('errors output',{concurrency: true},()=>{
 functionnormalize(str){
 returnstr.replaceAll(snapshot.replaceWindowsPaths(process.cwd()),'')
@@ -35,9 +36,12 @@ describe('errors output',{concurrency: true }, () =>{
 }
 constcommon=snapshot
 .transform(snapshot.replaceWindowsLineEndings,snapshot.replaceWindowsPaths);
-constdefaultTransform=snapshot.transform(common,normalize,replaceNodeVersion);
-consterrTransform=snapshot.transform(common,normalizeNoNumbers,replaceNodeVersion);
-constpromiseTransform=snapshot.transform(common,replaceStackTrace,normalizeNoNumbers,replaceNodeVersion);
+constdefaultTransform=snapshot.transform(common,normalize,snapshot.replaceNodeVersion);
+consterrTransform=snapshot.transform(common,normalizeNoNumbers,snapshot.replaceNodeVersion);
+constpromiseTransform=snapshot.transform(common,replaceStackTrace,
+normalizeNoNumbers,snapshot.replaceNodeVersion);
+constforceColorsTransform=snapshot.transform(common,normalize,
+replaceForceColorsStackTrace,snapshot.replaceNodeVersion);
 
 consttests=[
 {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}oftests){
 it(name,{ skip },async()=>{

test/parallel/test-node-output-sourcemaps.mjs

@@ -4,10 +4,6 @@ import * as snapshot from '../common/assertSnapshot.js'
 import*aspathfrom'node:path';
 import{describe,it}from'node:test';
 
-functionreplaceNodeVersion(str){
-returnstr.replaceAll(process.version,'*');
-}
-
 describe('sourcemaps output',{concurrency: true},()=>{
 functionnormalize(str){
 constresult=str
@@ -16,7 +12,8 @@ describe('sourcemaps output',{concurrency: true }, () =>{
 .replaceAll('/Users/bencoe/oss/coffee-script-test','')
 .replaceAll(/\/(\w)/g,'*$1')
 .replaceAll('*test*','*')
-.replaceAll('*fixtures*source-map*','*');
+.replaceAll('*fixtures*source-map*','*')
+.replaceAll(/(\W+).*node:internal\*modules.*/g,'$1*');
 if(common.isWindows){
 constcurrentDeviceLetter=path.parse(process.cwd()).root.substring(0,1).toLowerCase();
 constregex=newRegExp(`${currentDeviceLetter}:/?`,'gi');
@@ -25,7 +22,8 @@ describe('sourcemaps output',{concurrency: true }, () =>{
 returnresult;
 }
 constdefaultTransform=snapshot
-.transform(snapshot.replaceWindowsLineEndings,snapshot.replaceWindowsPaths,normalize,replaceNodeVersion);
+.transform(snapshot.replaceWindowsLineEndings,snapshot.replaceWindowsPaths,
+normalize,snapshot.replaceNodeVersion);
 
 consttests=[
 {name: 'source-map/output/source_map_disabled_by_api.js'},