fixup! test_runner: cover cancelled subtest order and random seed docs

Author: pmarchini

lib/internal/test_runner/runner.js

@@ -90,7 +90,6 @@ const {
   countCompletedTest,
   kDefaultPattern,
   parseCommandLine,
-  shuffleArrayWithSeed,
 } = require('internal/test_runner/utils');
 const { Glob } = require('internal/fs/glob');
 const { once } = require('events');

test/parallel/test-runner-randomize-first-draw.mjs

@@ -1,3 +1,4 @@
+import '../common/index.mjs';
 import assert from 'node:assert';
 import { test, run } from 'node:test';
 import * as fixtures from '../common/fixtures.mjs';

test/parallel/test-runner-run.mjs

@@ -106,19 +106,24 @@ describe('require(\'node:test\').run', { concurrency: true }, () => {
         files: [fixtures.path('test-runner', 'cancelled-report-order.mjs')],
         isolation,
       });
-      const cancelledNames = [];
+      const cancelledFailures = [];
 
       stream.on('test:fail', ({ name, details }) => {
         if (name === 'second' || name === 'third') {
-          cancelledNames.push(name);
-          assert.strictEqual(details.error.failureType, 'cancelledByParent');
+          cancelledFailures.push({
+            name,
+            failureType: details.error.failureType,
+          });
         }
       });
 
       // eslint-disable-next-line no-unused-vars
       for await (const _ of stream);
 
-      assert.deepStrictEqual(cancelledNames, ['second', 'third']);
+      assert.deepStrictEqual(cancelledFailures, [
+        { name: 'second', failureType: 'cancelledByParent' },
+        { name: 'third', failureType: 'cancelledByParent' },
+      ]);
     });
   }