test: remove empty lines from snapshots

nodejs · Dec 24, 2024 · f263f51 · f263f51
1 parent da3f388
commit f263f51
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 68 deletions.
diff --git a/test/fixtures/eval/eval_messages.snapshot b/test/fixtures/eval/eval_messages.snapshot
@@ -5,13 +5,6 @@ with(this){__filename}
 
 SyntaxError: Strict mode code may not include a with statement
 
-
-
-
-
-
-
-
 Node.js *
 42
 42
@@ -21,27 +14,13 @@ throw new Error("hello")
 
 Error: hello
 
-
-
-
-
-
-
-
 Node.js *
 [eval]:1
 throw new Error("hello")
 ^
 
 Error: hello
 
-
-
-
-
-
-
-
 Node.js *
 100
 [eval]:1
@@ -50,13 +29,6 @@ var x = 100; y = x;
 
 ReferenceError: y is not defined
 
-
-
-
-
-
-
-
 Node.js *
 
 [eval]:1

diff --git a/test/fixtures/eval/stdin_messages.snapshot b/test/fixtures/eval/stdin_messages.snapshot
@@ -5,16 +5,6 @@ with(this){__filename}
 
 SyntaxError: Strict mode code may not include a with statement
 
-
-
-
-
-
-
-
-
-
-
 Node.js *
 42
 42
@@ -24,33 +14,13 @@ throw new Error("hello")
 
 Error: hello
 
-
-
-
-
-
-
-
-
-
-
 Node.js *
 [stdin]:1
 throw new Error("hello")
 ^
 
 Error: hello
 
-
-
-
-
-
-
-
-
-
-
 Node.js *
 100
 [stdin]:1
@@ -59,16 +29,6 @@ let x = 100; y = x;
 
 ReferenceError: y is not defined
 
-
-
-
-
-
-
-
-
-
-
 Node.js *
 
 [stdin]:1

diff --git a/test/parallel/test-node-output-eval.mjs b/test/parallel/test-node-output-eval.mjs
@@ -11,6 +11,7 @@ describe('eval output', { concurrency: true }, () => {
 
   const defaultTransform = snapshot.transform(
     removeStackTraces,
+    filterEmptyLines,
     normalize,
     snapshot.replaceWindowsLineEndings,
     snapshot.replaceWindowsPaths,
@@ -21,6 +22,10 @@ describe('eval output', { concurrency: true }, () => {
     return output.replaceAll(/^ *at .+$/gm, '');
   }
 
+  function filterEmptyLines(output) {
+    return output.replaceAll(/^\s*$/gm, '');
+  }
+
   const tests = [
     { name: 'eval/eval_messages.js' },
     { name: 'eval/stdin_messages.js' },