Fix JSON reporting properly

bin/test-browser.js

@@ -71,6 +71,7 @@ class ArrayMap extends Map {
 
 class RemoteRunner {
   constructor(browser) {
+    this.failed = false;
     this.browser = browser;
     this.handlers = new ArrayMap();
     this.onceHandlers = new ArrayMap();
@@ -119,21 +120,21 @@ class RemoteRunner {
     }
   }
 
-  async handleEnd(failed) {
+  async handleEnd() {
     closeRequested = true;
     await this.browser.close();
-    this.completedOk = !failed;
-    process.exit(failed ? 1 : 0);
+    process.exit(this.failed ? 1 : 0);
   }
 
   handleFailed() {
+    this.failed = true;
     if (bail) {
       try {
         this.triggerHandlers('end');
       } catch (e) {
         console.log('An error occurred while bailing:', e);
       } finally {
-        this.handleEnd(true);
+        this.handleEnd();
       }
     }
   }
@@ -147,7 +148,9 @@ function BenchmarkConsoleReporter(runner) {
 
 function BenchmarkJsonReporter(runner) {
   runner.on('end', results => {
-    if (runner.completedOk) {
+    if (runner.failed) {
+      console.log('Runner failed; JSON will not be writted.');
+    } else {
       const { mkdirSync, writeFileSync } = require('fs');
 
       results.srcRoot = process.env.SRC_ROOT;
@@ -158,8 +161,6 @@ function BenchmarkJsonReporter(runner) {
       const jsonPath = `${resultsDir}/${new Date().toISOString()}.json`;
       writeFileSync(jsonPath, JSON.stringify(results, null, 2));
       console.log('Wrote JSON results to:', jsonPath);
-    } else {
-      console.log('Runner failed; JSON will not be writted.');
     }
   });
 }