test: remove empty lines from snapshots

marco-ippolito · aduh95 · commit 8a87e3905243 · 2025-01-02T15:28:50.000+01:00
PR-URL: #56358 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com> Reviewed-By: Jacob Smith <jacob@frende.me>
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' },
