test: fix incorrect test fixture

This commit updates a test runner fixture that includes a failing child test. However, the nested test is created using the top level test() function instead t.test(). This commit updates the fixture to use t.test(), while preserving the expected failure. Refs: #47164 PR-URL: #52185 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Chemi Atlow <[email protected]>
nodejs · Mar 25, 2024 · 5905596 · 5905596
1 parent bae14b7
commit 5905596
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 3 deletions.
diff --git a/test/fixtures/test-runner/output/default_output.js b/test/fixtures/test-runner/output/default_output.js
@@ -9,7 +9,9 @@ const test = require('node:test');
 test('should pass', () => {});
 test('should fail', () => { throw new Error('fail'); });
 test('should skip', { skip: true }, () => {});
-test('parent', () => {
-  test('should fail', () => { throw new Error('fail'); });
-  test('should pass but parent fail', () => {});
+test('parent', (t) => {
+  t.test('should fail', () => { throw new Error('fail'); });
+  t.test('should pass but parent fail', (t, done) => {
+    setImmediate(done);
+  });
 });
diff --git a/test/fixtures/test-runner/output/default_output.snapshot b/test/fixtures/test-runner/output/default_output.snapshot
@@ -17,6 +17,11 @@
     *[39m
     *[39m
     *[39m
+    *[39m
+        *[39m
+    *[39m
+    *[39m
+    *[39m
     *[39m
 
   [31m✖ should pass but parent fail [90m(*ms)[39m[39m
@@ -53,6 +58,11 @@
   *[39m
   *[39m
   *[39m
+  *[39m
+      *[39m
+  *[39m
+  *[39m
+  *[39m
   *[39m
 
 *