test_runner: warn on test file patterns that do not exist

Emit a warning for each literal positional argument that does not
match an existing file instead of silently dropping it. A warning
rather than an error keeps existing passing invocations passing.

Fixes: #64109

Author: UditDewan

lib/internal/test_runner/runner.js

@@ -34,6 +34,7 @@ const {
 } = primordials;
 
 const { spawn } = require('child_process');
+const { existsSync } = require('fs');
 const { finished } = require('internal/streams/end-of-stream');
 const { availableParallelism } = require('os');
 const { resolve, sep, isAbsolute } = require('path');
@@ -161,9 +162,19 @@ function createTestFileList(patterns, cwd) {
   });
   const results = glob.globSync();
 
-  if (hasUserSuppliedPattern && results.length === 0 && ArrayPrototypeEvery(glob.matchers, (m) => !m.hasMagic())) {
-    console.error(`Could not find '${ArrayPrototypeJoin(patterns, ', ')}'`);
-    process.exit(kGenericUserError);
+  if (hasUserSuppliedPattern) {
+    if (results.length === 0 && ArrayPrototypeEvery(glob.matchers, (m) => !m.hasMagic())) {
+      console.error(`Could not find '${ArrayPrototypeJoin(patterns, ', ')}'`);
+      process.exit(kGenericUserError);
+    }
+
+    const missing = ArrayPrototypeFilter(patterns, (pattern, i) => {
+      return !glob.matchers[i].hasMagic() && !existsSync(resolve(cwd, pattern));
+    });
+
+    if (missing.length > 0) {
+      process.emitWarning(`Could not find '${ArrayPrototypeJoin(missing, ', ')}'`);
+    }
   }
 
   return ArrayPrototypeSort(results);

test/parallel/test-runner-cli.js

@@ -23,6 +23,43 @@ for (const isolation of ['none', 'process']) {
     assert.match(child.stderr.toString(), /^Could not find/);
   }
 
+  {
+    // A file that is not found should warn even when other patterns match,
+    // and the matching tests should still run.
+    const args = [
+      '--test',
+      `--test-isolation=${isolation}`,
+      'a-random-file-that-does-not-exist.js',
+      join(testFixtures, 'default-behavior/test/random.cjs'),
+    ];
+    const child = spawnSync(process.execPath, args);
+
+    assert.strictEqual(child.status, 0);
+    assert.strictEqual(child.signal, null);
+    assert.match(child.stderr.toString(),
+                 /Warning: Could not find 'a-random-file-that-does-not-exist\.js'/);
+    assert.match(child.stdout.toString(), /this should pass/);
+  }
+
+  {
+    // Options after positional arguments are treated as patterns and should
+    // warn instead of being silently dropped.
+    const args = [
+      '--test',
+      `--test-isolation=${isolation}`,
+      join(testFixtures, 'default-behavior/test/random.cjs'),
+      '--test-reporter',
+      'tap',
+    ];
+    const child = spawnSync(process.execPath, args);
+
+    assert.strictEqual(child.status, 0);
+    assert.strictEqual(child.signal, null);
+    assert.match(child.stderr.toString(),
+                 /Warning: Could not find '--test-reporter, tap'/);
+    assert.match(child.stdout.toString(), /this should pass/);
+  }
+
   {
     // Default behavior. node_modules is ignored. Files that don't match the
     // pattern are ignored except in test/ directories.