test_runner: wait for filtered suite build

Signed-off-by: semimikoh <ejffjeosms@gmail.com>
PR-URL: #64208Fixes: #64203
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: Aviv Keller <me@aviv.sh>
Reviewed-By: Pietro Marchini <pietro.marchini94@gmail.com>

Author: semimikoh

lib/internal/test_runner/test.js

@@ -1854,6 +1854,11 @@ class Suite extends Test {
 return{__proto__: null, ctx,args: [ctx]};
 }
 
+asyncfilteredRun(){
+awaitthis.buildSuite;
+returnsuper.filteredRun();
+}
+
 asyncrun(){
 this.computeInheritedHooks();
 consthookArgs=this.getRunArgs();

test/fixtures/test-runner/filtered-suite-async-build.mjs

@@ -0,0 +1,17 @@
+importtestfrom'node:test';
+import{setTimeoutasdelay}from'node:timers/promises';
+
+test.suite('Outer',async()=>{
+awaitdelay(1);
+
+// This suite is filtered out by name. Its build is still pending when the
+// filtered run starts, so the subtest below is registered late.
+test.suite('Nested A',async()=>{
+awaitdelay(1);
+test('Nested A test',async()=>{});
+});
+
+test.suite('Nested C',async()=>{
+test('Nested C test',async()=>{});
+});
+});

test/parallel/test-runner-no-isolation-filtering.js

@@ -7,6 +7,8 @@ const { test } = require('node:test');
 
 constfixture1=fixtures.path('test-runner','no-isolation','one.test.js');
 constfixture2=fixtures.path('test-runner','no-isolation','two.test.js');
+constasyncBuildFilteredSuite=
+fixtures.path('test-runner','filtered-suite-async-build.mjs');
 
 test('works with --test-only',()=>{
 constargs=[
@@ -71,6 +73,27 @@ test('works with --test-name-pattern', () => {
 assert.match(stdout,/#suites0/);
 });
 
+test('filtered suites with an async build do not leave cancelled tests',()=>{
+constargs=[
+'--test',
+'--test-reporter=tap',
+'--test-isolation=none',
+'--test-name-pattern=C',
+asyncBuildFilteredSuite,
+];
+constchild=spawnSync(process.execPath,args);
+conststdout=child.stdout.toString();
+
+assert.strictEqual(child.status,0);
+assert.strictEqual(child.signal,null);
+assert.match(stdout,/#tests1/);
+assert.match(stdout,/#suites2/);
+assert.match(stdout,/#pass1/);
+assert.match(stdout,/#fail0/);
+assert.match(stdout,/#cancelled0/);
+assert.doesNotMatch(stdout,/parentAlreadyFinished/);
+});
+
 test('works with --test-skip-pattern',()=>{
 constargs=[
 '--test',