@pareto4/pp-core 1.20.0 → 1.21.0

package/scripts/run-tests.cjs

@@ -31,7 +31,7 @@
 
 const { readdirSync } = require('fs');
 const { join, basename } = require('path');
-const { execFileSync } = require('child_process');
+const { execFileSync, spawnSync } = require('child_process');
 const { ExitError, runMain } = require('./lib/cli-exit.cjs');
 
 const SUITES = ['all', 'unit', 'integration', 'install', 'security', 'slow'];
@@ -548,42 +548,66 @@ function main() {
     if (chunks.length > 1) {
       console.error(`run-tests: chunk ${i + 1}/${chunks.length} — ${chunks[i].length} files`);
     }
-    try {
-      execFileSync(
-        process.execPath,
-        ['--test', ...(forceExit ? ['--test-force-exit'] : []), concurrency, ...chunks[i]],
-        {
-          stdio: 'inherit',
-          env: { ...process.env },
-          timeout: chunkTimeoutMs,
-        },
+    const chunkFiles = chunks[i].map(f => f.split(/[\\/]/).pop()).join(' ');
+    // stdout is captured, not inherited, so the chunk's node:test summary can be
+    // checked; it is written through unchanged once the chunk ends. stderr stays
+    // live. A chunk can exit 0 with its output cut off and no summary — on CI
+    // (2026-10-28) two of five chunks did, and ~1,000 tests never reported while
+    // the job stayed green — so a missing summary fails the run.
+    const result = spawnSync(
+      process.execPath,
+      ['--test', ...(forceExit ? ['--test-force-exit'] : []), concurrency, ...chunks[i]],
+      {
+        stdio: ['inherit', 'pipe', 'inherit'],
+        env: { ...process.env },
+        timeout: chunkTimeoutMs,
+        maxBuffer: 1024 * 1024 * 1024,
+      },
+    );
+    if (result.stdout && result.stdout.length > 0) process.stdout.write(result.stdout);
+    // When the per-chunk timeout fires, spawnSync kills the child and reports
+    // error.code === 'ETIMEDOUT' (POSIX) and/or a kill signal (platform-dependent).
+    // Check both so detection holds on Windows and POSIX.
+    const timedOut = (result.error && result.error.code === 'ETIMEDOUT') ||
+      (result.signal !== null && result.status === null && !result.error);
+    let code = result.status === null ? 1 : result.status;
+    if (result.error && result.error.code === 'ETIMEDOUT') code = 1;
+    if (timedOut) {
+      console.error(
+        `run-tests: chunk ${i + 1}/${chunks.length} exceeded the per-chunk timeout ` +
+          `of ${chunkTimeoutMs}ms and was killed — a test in this chunk is likely leaking ` +
+          `an open handle (un-terminated Worker, un-killed child process, or ref'd timer) ` +
+          `so node --test never exits. Files: ${chunkFiles}`,
       );
-    } catch (err) {
-      // When the per-chunk timeout fires, execFileSync kills the child and
-      // surfaces it as err.code === 'ETIMEDOUT' (POSIX) and/or err.killed === true
-      // (platform-dependent). Check both so detection holds on Windows and POSIX.
-      const timedOut = err.killed === true || err.code === 'ETIMEDOUT';
-      if (timedOut) {
-        console.error(
-          `run-tests: chunk ${i + 1}/${chunks.length} exceeded the per-chunk timeout ` +
-            `of ${chunkTimeoutMs}ms and was killed — a test in this chunk is likely leaking ` +
-            `an open handle (un-terminated Worker, un-killed child process, or ref'd timer) ` +
-            `so node --test never exits. Files: ${chunks[i]
-              .map(f => f.split(/[\\/]/).pop())
-              .join(' ')}`,
-        );
-      }
-      const code = err.status || 1;
-      // Run every chunk so the operator sees all failures in one pass; report
-      // the first non-zero exit at the end.
-      if (firstFailureExit === 0) firstFailureExit = code;
+    } else if (result.error) {
+      console.error(`run-tests: chunk ${i + 1}/${chunks.length} could not run: ${result.error.message}`);
+      code = 1;
+    } else if (code === 0 && !hasTestSummary(result.stdout)) {
+      console.error(
+        `run-tests: chunk ${i + 1}/${chunks.length} exited without a test summary ` +
+          `(no "ℹ tests N" / "ℹ fail N" lines) — its output was cut off, so some of its ` +
+          `tests never reported. Files: ${chunkFiles}`,
+      );
+      code = 1;
     }
+    // Run every chunk so the operator sees all failures in one pass; report
+    // the first non-zero exit at the end.
+    if (code !== 0 && firstFailureExit === 0) firstFailureExit = code;
   }
   if (firstFailureExit !== 0) return firstFailureExit;
 }
 
+/**
+ * Whether a chunk's stdout carries node:test's closing summary: both the
+ * `tests N` and `fail N` lines, from the spec reporter (`ℹ`) or TAP (`#`).
+ */
+function hasTestSummary(stdout) {
+  const text = stdout ? stdout.toString('utf8') : '';
+  return /^(?:ℹ|#) tests \d+\s*$/m.test(text) && /^(?:ℹ|#) fail \d+\s*$/m.test(text);
+}
+
 if (require.main === module) {
   runMain(main);
 }
 
-module.exports = { suiteOf, ensureBuiltArtifacts, parseShardArg, selectShard };
+module.exports = { suiteOf, ensureBuiltArtifacts, parseShardArg, selectShard, hasTestSummary };