@testsmith/testblocks 0.8.2 → 0.8.4

package/dist/cli/index.js

@@ -177,7 +177,7 @@ program
     .argument('<patterns...>', 'Test file patterns (glob supported)')
     .option('-H, --headed', 'Run tests in headed mode (show browser)', false)
     .option('-t, --timeout <ms>', 'Test timeout in milliseconds', '30000')
-    .option('-r, --reporter <type>', 'Reporter type: console, json, junit, html', 'console')
+    .option('-r, --reporter <types>', 'Reporter types (comma-separated): console, json, junit, html', 'console')
     .option('-o, --output <dir>', 'Output directory for reports', './testblocks-results')
     .option('-b, --base-url <url>', 'Base URL for relative URLs')
     .option('-v, --var <vars...>', 'Variables in key=value format')
@@ -275,8 +275,8 @@ program
             variables,
             procedures: globalProcedures,
         };
-        // Create reporter
-        const reporter = createReporter(options.reporter, options.output);
+        // Create reporters (supports multiple, comma-separated)
+        const reporters = createReporters(options.reporter, options.output);
         // Run tests
         const allResults = [];
         let hasFailures = false;
@@ -312,8 +312,8 @@ program
             const executor = new executor_1.TestExecutor(executorOptions);
             const results = await executor.runTestFile(testFile);
             allResults.push({ file, results });
-            // Report results
-            reporter.onTestFileComplete(file, testFile, results);
+            // Report results to all reporters
+            reporters.forEach(r => r.onTestFileComplete(file, testFile, results));
             // Check for failures
             const failed = results.some(r => r.status !== 'passed');
             if (failed) {
@@ -324,8 +324,8 @@ program
                 }
             }
         }
-        // Generate final report
-        reporter.onComplete(allResults);
+        // Generate final reports
+        reporters.forEach(r => r.onComplete(allResults));
         // Exit with appropriate code
         process.exit(hasFailures ? 1 : 0);
     }
@@ -432,9 +432,10 @@ program
                 'test:headed': 'testblocks run tests/**/*.testblocks.json --headed',
                 'test:html': 'testblocks run tests/**/*.testblocks.json -r html -o reports',
                 'test:junit': 'testblocks run tests/**/*.testblocks.json -r junit -o reports',
+                'test:ci': 'testblocks run tests/**/*.testblocks.json -r console,html,junit -o reports',
             },
             devDependencies: {
-                '@testsmith/testblocks': '^0.8.2',
+                '@testsmith/testblocks': '^0.8.4',
             },
         };
         fs.writeFileSync(packagePath, JSON.stringify(packageJson, null, 2));
@@ -577,15 +578,31 @@ jobs:
         run: npx playwright install --with-deps chromium
 
       - name: Run tests
-        run: npm test
+        run: npm run test:ci
 
-      - name: Upload test reports
+      - name: Upload HTML report
         uses: actions/upload-artifact@v4
         if: always()
         with:
-          name: test-reports
-          path: reports/
+          name: html-report
+          path: reports/*.html
           retention-days: 30
+
+      - name: Upload JUnit report
+        uses: actions/upload-artifact@v4
+        if: always()
+        with:
+          name: junit-report
+          path: reports/*.xml
+          retention-days: 30
+
+      - name: Publish Test Results
+        uses: dorny/test-reporter@v1
+        if: always()
+        with:
+          name: Test Results
+          path: reports/*.xml
+          reporter: java-junit
 `;
         fs.writeFileSync(workflowPath, workflow);
         console.log('  Created: .github/workflows/testblocks.yml');
@@ -655,18 +672,35 @@ program
         open: options.open,
     });
 });
-function createReporter(type, outputDir) {
-    switch (type) {
-        case 'json':
-            return new reporters_1.JSONReporter(outputDir);
-        case 'junit':
-            return new reporters_1.JUnitReporter(outputDir);
-        case 'html':
-            return new reporters_1.HTMLReporter(outputDir);
-        case 'console':
-        default:
-            return new reporters_1.ConsoleReporter();
+function createReporters(types, outputDir) {
+    const reporterTypes = types.split(',').map(t => t.trim().toLowerCase());
+    const reporters = [];
+    for (const type of reporterTypes) {
+        switch (type) {
+            case 'json':
+                reporters.push(new reporters_1.JSONReporter(outputDir));
+                break;
+            case 'junit':
+                reporters.push(new reporters_1.JUnitReporter(outputDir));
+                break;
+            case 'html':
+                reporters.push(new reporters_1.HTMLReporter(outputDir));
+                break;
+            case 'console':
+                reporters.push(new reporters_1.ConsoleReporter());
+                break;
+            default:
+                console.warn(`Unknown reporter type: ${type}, using console`);
+                if (!reporters.some(r => r instanceof reporters_1.ConsoleReporter)) {
+                    reporters.push(new reporters_1.ConsoleReporter());
+                }
+        }
+    }
+    // Always include console reporter if not already included
+    if (!reporters.some(r => r instanceof reporters_1.ConsoleReporter)) {
+        reporters.unshift(new reporters_1.ConsoleReporter());
     }
+    return reporters;
 }
 function validateTestFile(testFile) {
     const errors = [];