@dionlarson/playwright-orchestrator-core 1.3.2 → 1.3.4

package/dist/playwright-tools/test-result-reporter.js

@@ -1,3 +1,4 @@
+import { writeFileSync } from 'node:fs';
 export default class TestResultReporter {
     testResults = [];
     testCases = [];
@@ -58,6 +59,13 @@ export default class TestResultReporter {
                 retry,
             })),
         };
-        console.log(JSON.stringify(output));
+        const outputFile = process.env.PLAYWRIGHT_ORCHESTRATOR_RESULT_FILE;
+        if (outputFile) {
+            writeFileSync(outputFile, JSON.stringify(output));
+        }
+        else {
+            // Fallback to stdout if no file specified
+            console.log(JSON.stringify(output));
+        }
     }
 }

package/dist/test-runner.d.ts

@@ -12,7 +12,6 @@ export declare class TestRunner {
     private runTestsUntilAvailable;
     private removePreviousReports;
     private runTest;
-    private parseTestResult;
     private buildParams;
     private createTempConfig;
 }

package/dist/test-runner.js

@@ -1,37 +1,19 @@
 import { createHash } from 'node:crypto';
 import child_process, { spawn } from 'node:child_process';
 import { promisify } from 'node:util';
-import { rm, writeFile } from 'node:fs/promises';
+import { mkdir, readFile, rm, writeFile } from 'node:fs/promises';
 import { TestExecutionReporter } from './reporters/test-execution-reporter.js';
 import path from 'node:path';
 import * as uuid from 'uuid';
 const exec = promisify(child_process.exec);
-function spawnWithOutput(command, args, env) {
+function spawnPassthrough(command, env) {
     return new Promise((resolve, reject) => {
-        const proc = spawn(command, args, {
+        const proc = spawn(command, {
             env,
             shell: true,
-            stdio: ['inherit', 'pipe', 'inherit'] // stdin inherit, stdout pipe, stderr inherit
-        });
-        let stdout = '';
-        proc.stdout?.on('data', (data) => {
-            const str = data.toString();
-            stdout += str;
-            // Stream output but filter out the JSON reporter output
-            if (!str.trim().startsWith('{')) {
-                process.stdout.write(data);
-            }
-        });
-        proc.on('close', (code) => {
-            if (code === 0) {
-                resolve(stdout);
-            }
-            else {
-                const error = new Error(`Process exited with code ${code}`);
-                error.stdout = stdout;
-                reject(error);
-            }
+            stdio: 'inherit' // Full passthrough - all output goes directly to terminal
         });
+        proc.on('close', (code) => resolve(code ?? 1));
         proc.on('error', reject);
     });
 }
@@ -88,45 +70,55 @@ export class TestRunner {
     }
     async removePreviousReports() {
         await rm(`./${this.outputFolder}`, { recursive: true, force: true });
+        await mkdir(`./${this.outputFolder}`, { recursive: true });
     }
     async runTest(test, config) {
         const testPosition = `${test.file}:${test.position}`;
         const testName = `[${test.project}] > ${testPosition}`;
         const testHash = createHash('md5').update(testName).digest('hex');
+        const resultFile = `${this.outputFolder}/${testHash}.result.json`;
         const args = [testPosition, ...this.buildParams(test, config, testHash).split(' ')];
         const env = {
             ...process.env,
             PLAYWRIGHT_BLOB_OUTPUT_FILE: `${this.outputFolder}/${testHash}.zip`,
+            PLAYWRIGHT_ORCHESTRATOR_RESULT_FILE: resultFile,
         };
+        const fullCommand = `npx playwright test ${args.join(' ')}`;
+        const exitCode = await spawnPassthrough(fullCommand, env);
+        let testResult;
         try {
-            const stdout = await spawnWithOutput('npx', ['playwright', 'test', ...args], env);
+            const resultJson = await readFile(resultFile, 'utf-8');
+            testResult = JSON.parse(resultJson);
+            await rm(resultFile, { force: true });
+        }
+        catch (e) {
+            console.error(`[orchestrator] Failed to read test result for [${test.project}] ${test.file}:${test.position}`, e);
+            throw e;
+        }
+        if (exitCode === 0) {
+            console.log(`[orchestrator] Test passed: [${test.project}] ${test.file}:${test.position}`);
             this.reporter.finishTest(test);
             await this.adapter.finishTest({
                 runId: this.runId,
                 test,
-                testResult: this.parseTestResult(stdout),
+                testResult,
                 config,
             });
         }
-        catch (error) {
-            if (!error.stdout)
-                throw error;
+        else {
+            console.log(`[orchestrator] Test failed: [${test.project}] ${test.file}:${test.position}`);
             this.reporter.failTest(test);
             await this.adapter.failTest({
                 runId: this.runId,
                 test,
-                testResult: this.parseTestResult(error.stdout),
+                testResult,
                 config,
             });
         }
     }
-    parseTestResult(stdout) {
-        return JSON.parse(stdout);
-    }
     buildParams(test, config, testHash) {
         const args = [...config.args];
         args.push('--workers', '1');
-        args.push('--reporter', 'blob,@dionlarson/playwright-orchestrator-core/test-result-reporter');
         args.push('--project', `"${test.project}"`);
         args.push('--output', `"${this.outputFolder}/${testHash}"`);
         if (config.configFile) {
@@ -137,10 +129,22 @@ export class TestRunner {
     async createTempConfig(file) {
         if (!file)
             return;
-        // Remove webServer from the config. Not supported in the orchestrator
+        // Modify config: remove webServer, add our reporters while preserving existing ones
         const content = `
         import config from '${path.resolve(file)}';
+
+        // Remove webServer - not supported in orchestrator (server should already be running)
         delete config.webServer;
+
+        // Add our reporters while preserving existing reporters from config
+        const existingReporters = Array.isArray(config.reporter) ? config.reporter :
+            config.reporter ? [[config.reporter]] : [];
+        config.reporter = [
+            ...existingReporters,
+            ['blob'],
+            ['@dionlarson/playwright-orchestrator-core/test-result-reporter'],
+        ];
+
         export default config;`;
         const tempFile = `.playwright-${uuid.v7()}.config.tmp.ts`;
         await writeFile(tempFile, content);

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@dionlarson/playwright-orchestrator-core",
-    "version": "1.3.2",
+    "version": "1.3.4",
     "description": "Core lib and cli for Playwright test orchestration",
     "keywords": [
         "playwright",