test: remove empty lines from snapshots

PR-URL: #56358 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Jacob Smith <[email protected]>
nodejs · Dec 30, 2024 · 0e04aea · 0e04aea
1 parent d2af881
commit 0e04aea
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 59 deletions.
diff --git a/test/fixtures/eval/eval_messages.snapshot b/test/fixtures/eval/eval_messages.snapshot
@@ -27,27 +27,13 @@ throw new Error("hello")
 
 Error: hello
 
-
-
-
-
-
-
-
 Node.js *
 [eval]:1
 throw new Error("hello")
 ^
 
 Error: hello
 
-
-
-
-
-
-
-
 Node.js *
 100
 [eval]:1
@@ -56,13 +42,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
@@ -14,14 +14,6 @@ Caused by:
 
 SyntaxError: Strict mode code may not include a with statement
 
-
-
-
-
-
-
-
-
 Node.js *
 42
 42
@@ -31,33 +23,13 @@ throw new Error("hello")
 
 Error: hello
 
-
-
-
-
-
-
-
-
-
-
 Node.js *
 [stdin]:1
 throw new Error("hello")
 ^
 
 Error: hello
 
-
-
-
-
-
-
-
-
-
-
 Node.js *
 100
 [stdin]:1
@@ -66,16 +38,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' },