tape-six 1.7.3 → 1.7.4

package/README.md

@@ -420,6 +420,7 @@ Test output can be controlled by flags. See [Supported flags](https://github.com
 
 The most recent releases:
 
+- 1.7.4 _Bug fixes: uncaught exception handling, `StopTest` suppression in parallel runners._
 - 1.7.3 _Bug fixes in reporters, runners, and assertions. Documentation corrections and improvements._
 - 1.7.2 _Minor internal refactoring and fixes._
 - 1.7.1 _Added AI support, added timeout to start test runners, fixed some bugs in the sequential test runner._

package/bin/tape6-bun.js

@@ -119,9 +119,10 @@ const main = async () => {
   await init();
   await selectTimer();
 
-  process.on('uncaughtException', (error, origin) =>
-    console.error('UNHANDLED ERROR:', origin, error)
-  );
+  process.on('uncaughtException', (error, origin) => {
+    console.error('UNHANDLED ERROR:', origin, error);
+    process.exit(1);
+  });
 
   const reporter = getReporter(),
     worker = new TestWorker(reporter, parallel, options);

package/bin/tape6-node.js

@@ -132,9 +132,10 @@ const main = async () => {
   await init();
   await selectTimer();
 
-  process.on('uncaughtException', (error, origin) =>
-    console.error('UNHANDLED ERROR:', origin, error)
-  );
+  process.on('uncaughtException', (error, origin) => {
+    console.error('UNHANDLED ERROR:', origin, error);
+    process.exit(1);
+  });
 
   if (!files.length) {
     console.log('No files found.');

package/bin/tape6-seq.js

@@ -104,9 +104,10 @@ const main = async () => {
 
   const console = globalThis.console;
 
-  process.on('uncaughtException', (error, origin) =>
-    console.error('UNHANDLED ERROR:', origin, error)
-  );
+  process.on('uncaughtException', (error, origin) => {
+    console.error('UNHANDLED ERROR:', origin, error);
+    process.exit(1);
+  });
 
   const reporter = getReporter(),
     worker = new TestWorker(reporter, 1, {...options, testRunner});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "tape-six",
-  "version": "1.7.3",
+  "version": "1.7.4",
   "description": "TAP-based unit test library for Node, Deno, Bun, and browsers. ES modules, TypeScript, zero dependencies.",
   "type": "module",
   "main": "index.js",

package/src/State.js

@@ -246,10 +246,10 @@ export class State {
     return event;
   }
 
-  postprocess(event) {
+  postprocess(event, suppressStopTest = false) {
     switch (event.type) {
       case 'assert':
-        if (event.stopTest && event.operator !== 'exception') {
+        if (!suppressStopTest && event.stopTest && event.operator !== 'exception') {
           const stopTest = new StopTest('failOnce is activated');
           stopTest[signature] = signature;
           throw stopTest;
@@ -257,6 +257,7 @@ export class State {
         this.time = this.timer.now();
         break;
       case 'bail-out':
+        if (suppressStopTest) return;
         const stopTest = new StopTest('bailOut is activated');
         stopTest[signature] = signature;
         throw stopTest;

package/src/reporters/JSONLReporter.js

@@ -21,7 +21,7 @@ export class JSONLReporter extends Reporter {
     prefix ||= getEnvVar('TAPE6_JSONL_PREFIX') || '';
     this.prefix = prefix ? '\n' + prefix : '';
   }
-  report(event) {
+  report(event, suppressStopTest = false) {
     event = this.state?.preprocess(event) || event;
     if (event.type === 'assert' && this.renumberAsserts) {
       event = {...event, id: ++this.assertCounter};
@@ -39,7 +39,7 @@ export class JSONLReporter extends Reporter {
     }
     const jsonEvent = JSON.stringify(event);
     this.console.log(this.prefix ? this.prefix + jsonEvent : jsonEvent);
-    this.state?.postprocess(event);
+    this.state?.postprocess(event, suppressStopTest);
   }
 }
 

package/src/reporters/MinReporter.js

@@ -5,7 +5,7 @@ export class MinReporter extends Reporter {
     super({failOnce});
     this.console = originalConsole || console;
   }
-  report(event) {
+  report(event, suppressStopTest = false) {
     event = this.state?.preprocess(event) || event;
     const handler = Reporter.EVENT_MAP[event.type];
     typeof handler == 'string' && this[handler]?.(event);
@@ -19,7 +19,7 @@ export class MinReporter extends Reporter {
       'Fail:',
       event.fail
     );
-    this.state?.postprocess(event);
+    this.state?.postprocess(event, suppressStopTest);
   }
 }
 

package/src/reporters/ProxyReporter.js

@@ -5,7 +5,7 @@ export class ProxyReporter extends Reporter {
     super({failOnce});
     this.reportTo = reportTo;
   }
-  report(event) {
+  report(event, suppressStopTest = false) {
     event = this.state?.preprocess(event) || event;
     switch (event.type) {
       case 'test':
@@ -19,7 +19,7 @@ export class ProxyReporter extends Reporter {
         break;
     }
     this.reportTo(event);
-    this.state?.postprocess(event);
+    this.state?.postprocess(event, suppressStopTest);
   }
 }
 

package/src/reporters/Reporter.js

@@ -57,13 +57,13 @@ export class Reporter {
     }
   }
 
-  report(event) {
+  report(event, suppressStopTest = false) {
     event = this.state?.preprocess(event) || event;
 
     const handler = this.constructor.EVENT_MAP[event.type];
     typeof handler == 'string' && this[handler]?.(event);
 
-    this.state?.postprocess(event);
+    this.state?.postprocess(event, suppressStopTest);
   }
 
   static EVENT_MAP = {

package/src/reporters/TTYReporter.js

@@ -148,15 +148,15 @@ export class TTYReporter extends Reporter {
     ++this.lines;
     return this;
   }
-  report(event) {
+  report(event, suppressStopTest = false) {
     this.consoleSkipChecks = true;
     try {
-      this.reportInternal(event);
+      this.reportInternal(event, suppressStopTest);
     } finally {
       this.consoleSkipChecks = false;
     }
   }
-  reportInternal(event) {
+  reportInternal(event, suppressStopTest = false) {
     if (this.output.isTTY) {
       if (!this.consoleWasUsed && this.overrideLastLine) {
         this.output.moveCursor(0, -1);
@@ -320,7 +320,7 @@ export class TTYReporter extends Reporter {
       this.showScore();
       this.overrideLastLine = true;
     }
-    this.state?.postprocess(event);
+    this.state?.postprocess(event, suppressStopTest);
   }
   showScore() {
     this.out(

package/src/reporters/TapReporter.js

@@ -61,7 +61,7 @@ export class TapReporter extends Reporter {
       this.opened = true;
     }
   }
-  report(event) {
+  report(event, suppressStopTest = false) {
     event = this.state?.preprocess(event) || event;
     let text;
     switch (event.type) {
@@ -192,6 +192,7 @@ export class TapReporter extends Reporter {
         }
         break;
     }
+    this.state?.postprocess(event, suppressStopTest);
   }
   showSummary(state, diffTime) {
     const success = state.asserts - state.failed - state.skipped;

package/src/runners/bun/TestWorker.js

@@ -1,6 +1,5 @@
 import {sep} from 'node:path';
 
-import {isStopTest} from '../../State.js';
 import EventServer from '../../utils/EventServer.js';
 
 const srcName = new URL('../../', import.meta.url),
@@ -21,11 +20,7 @@ export default class TestWorker extends EventServer {
     this.idToWorker[id] = worker;
     worker.addEventListener('message', event => {
       const msg = event.data;
-      try {
-        this.report(id, msg);
-      } catch (error) {
-        if (!isStopTest(error)) throw error;
-      }
+      this.report(id, msg);
       if (msg.type === 'end' && msg.test === 0) {
         this.close(id);
       }
@@ -36,20 +31,16 @@ export default class TestWorker extends EventServer {
         name: 'fail to load: ' + (error.message || 'Worker error'),
         test: 0
       });
-      try {
-        this.report(id, {
-          name: String(error),
-          test: 0,
-          marker: new Error(),
-          operator: 'error',
-          fail: true,
-          data: {
-            actual: error
-          }
-        });
-      } catch (error) {
-        if (!isStopTest(error)) throw error;
-      }
+      this.report(id, {
+        name: String(error),
+        test: 0,
+        marker: new Error(),
+        operator: 'error',
+        fail: true,
+        data: {
+          actual: error
+        }
+      });
       this.close(id);
     });
     worker.addEventListener('messageerror', error => {
@@ -58,20 +49,16 @@ export default class TestWorker extends EventServer {
         name: 'fail to load: ' + (error.message || 'Worker error'),
         test: 0
       });
-      try {
-        this.report(id, {
-          name: String(error),
-          test: 0,
-          marker: new Error(),
-          operator: 'error',
-          fail: true,
-          data: {
-            actual: error
-          }
-        });
-      } catch (error) {
-        if (!isStopTest(error)) throw error;
-      }
+      this.report(id, {
+        name: String(error),
+        test: 0,
+        marker: new Error(),
+        operator: 'error',
+        fail: true,
+        data: {
+          actual: error
+        }
+      });
       this.close(id);
     });
     worker.postMessage({

package/src/runners/deno/TestWorker.js

@@ -1,7 +1,6 @@
 import {pathToFileURL} from 'node:url';
 import {sep} from 'node:path';
 
-import {isStopTest} from '../../State.js';
 import EventServer from '../../utils/EventServer.js';
 
 const srcName = new URL('../../', import.meta.url),
@@ -23,11 +22,7 @@ export default class TestWorker extends EventServer {
     this.idToWorker[id] = worker;
     worker.addEventListener('message', event => {
       const msg = event.data;
-      try {
-        this.report(id, msg);
-      } catch (error) {
-        if (!isStopTest(error)) throw error;
-      }
+      this.report(id, msg);
       if (msg.type === 'end' && msg.test === 0) {
         this.close(id);
       }
@@ -38,20 +33,16 @@ export default class TestWorker extends EventServer {
         name: 'fail to load: ' + (error.message || 'Worker error'),
         test: 0
       });
-      try {
-        this.report(id, {
-          name: String(error),
-          test: 0,
-          marker: new Error(),
-          operator: 'error',
-          fail: true,
-          data: {
-            actual: error
-          }
-        });
-      } catch (error) {
-        if (!isStopTest(error)) throw error;
-      }
+      this.report(id, {
+        name: String(error),
+        test: 0,
+        marker: new Error(),
+        operator: 'error',
+        fail: true,
+        data: {
+          actual: error
+        }
+      });
       this.close(id);
     });
     worker.postMessage({

package/src/runners/node/TestWorker.js

@@ -3,7 +3,6 @@ import {sep} from 'node:path';
 import {pathToFileURL} from 'node:url';
 import {Worker} from 'node:worker_threads';
 
-import {isStopTest} from '../../State.js';
 import EventServer from '../../utils/EventServer.js';
 
 const srcName = new URL('../../', import.meta.url),
@@ -23,11 +22,7 @@ export default class TestWorker extends EventServer {
       });
     this.idToWorker[id] = worker;
     worker.on('message', msg => {
-      try {
-        this.report(id, msg);
-      } catch (error) {
-        if (!isStopTest(error)) throw error;
-      }
+      this.report(id, msg);
       if (msg.type === 'end' && msg.test === 0) {
         this.close(id);
       }
@@ -38,20 +33,16 @@ export default class TestWorker extends EventServer {
         name: 'fail to load: ' + (error.message || 'Worker error'),
         test: 0
       });
-      try {
-        this.report(id, {
-          name: String(error),
-          test: 0,
-          marker: new Error(),
-          operator: 'error',
-          fail: true,
-          data: {
-            actual: error
-          }
-        });
-      } catch (error) {
-        if (!isStopTest(error)) throw error;
-      }
+      this.report(id, {
+        name: String(error),
+        test: 0,
+        marker: new Error(),
+        operator: 'error',
+        fail: true,
+        data: {
+          actual: error
+        }
+      });
       this.close(id);
     });
     worker.on('messageerror', error => {
@@ -60,20 +51,16 @@ export default class TestWorker extends EventServer {
         name: 'fail to load: ' + (error.message || 'Worker error'),
         test: 0
       });
-      try {
-        this.report(id, {
-          name: String(error),
-          test: 0,
-          marker: new Error(),
-          operator: 'error',
-          fail: true,
-          data: {
-            actual: error
-          }
-        });
-      } catch (error) {
-        if (!isStopTest(error)) throw error;
-      }
+      this.report(id, {
+        name: String(error),
+        test: 0,
+        marker: new Error(),
+        operator: 'error',
+        fail: true,
+        data: {
+          actual: error
+        }
+      });
       this.close(id);
     });
     worker.postMessage({

package/src/runners/seq/BypassReporter.js

@@ -29,8 +29,8 @@ export class BypassReporter {
     this.reporter.abort();
   }
 
-  report(event) {
-    this.reportTo(event);
+  report(event, suppressStopTest = false) {
+    this.reportTo(event, suppressStopTest);
   }
 }
 

package/src/runners/seq/TestWorker.js

@@ -2,7 +2,6 @@ import process from 'node:process';
 import {sep} from 'node:path';
 import {pathToFileURL} from 'node:url';
 
-import {isStopTest} from '../../State.js';
 import EventServer from '../../utils/EventServer.js';
 import {getTimeoutValue} from '../../utils/config.js';
 import {getOriginalConsole, setCurrentReporter} from '../../utils/capture-console.js';
@@ -35,11 +34,7 @@ export default class TestWorker extends EventServer {
           clearTimeout(this.timeoutId);
           this.timeoutId = null;
         }
-        try {
-          this.report(id, event);
-        } catch (error) {
-          if (!isStopTest(error)) throw error;
-        }
+        this.report(id, event);
         if (event.type === 'end' && event.test === 0) {
           this.close(id);
         }
@@ -79,20 +74,16 @@ export default class TestWorker extends EventServer {
       name: 'fail to load: ' + (error.message || 'Worker error'),
       test: 0
     });
-    try {
-      this.report(id, {
-        name: String(error),
-        test: 0,
-        marker: new Error(),
-        operator: 'error',
-        fail: true,
-        data: {
-          actual: error
-        }
-      });
-    } catch (error) {
-      if (!isStopTest(error)) throw error;
-    }
+    this.report(id, {
+      name: String(error),
+      test: 0,
+      marker: new Error(),
+      operator: 'error',
+      fail: true,
+      data: {
+        actual: error
+      }
+    });
     this.close(id);
   }
   #reportTimeout(id, fileName) {
@@ -101,17 +92,13 @@ export default class TestWorker extends EventServer {
       test: 0,
       name: 'FILE: /' + fileName
     });
-    try {
-      this.report(id, {
-        name: `No tests found in ${this.timeout}ms`,
-        test: 0,
-        marker: new Error(),
-        operator: 'error',
-        fail: true
-      });
-    } catch (error) {
-      if (!isStopTest(error)) throw error;
-    }
+    this.report(id, {
+      name: `No tests found in ${this.timeout}ms`,
+      test: 0,
+      marker: new Error(),
+      operator: 'error',
+      fail: true
+    });
     this.report(id, {
       type: 'end',
       test: 0,

package/src/utils/EventServer.js

@@ -18,11 +18,11 @@ export default class EventServer {
     if (this.passThroughId === id) {
       if (events) {
         for (const event of events) {
-          this.reporter.report(event);
+          this.reporter.report(event, true);
         }
         delete this.retained[id];
       }
-      return this.reporter.report(event);
+      return this.reporter.report(event, true);
     }
     if (Array.isArray(events)) {
       events.push(event);
@@ -46,7 +46,7 @@ export default class EventServer {
       // dump ready events
       for (const events of this.readyQueue) {
         for (const event of events) {
-          this.reporter.report(event);
+          this.reporter.report(event, true);
         }
       }
       this.readyQueue = [];
@@ -57,7 +57,7 @@ export default class EventServer {
         if (this.passThroughId === null) {
           // dump events
           for (const event of events) {
-            this.reporter.report(event);
+            this.reporter.report(event, true);
           }
         } else {
           // add to the ready queue

package/web-app/DashReporter.js

@@ -27,7 +27,7 @@ export class DashReporter extends Reporter {
     this.spinnerNode = document.querySelector('.tape6 tape6-spinner');
     this.running = true;
   }
-  report(event) {
+  report(event, suppressStopTest = false) {
     event = this.state?.preprocess(event) || event;
     switch (event.type) {
       case 'test':
@@ -57,7 +57,7 @@ export class DashReporter extends Reporter {
         this.updateDashboard();
         break;
     }
-    this.state?.postprocess(event);
+    this.state?.postprocess(event, suppressStopTest);
   }
   updateDashboard() {
     this.updateDonut();

package/web-app/DomReporter.js

@@ -28,7 +28,7 @@ class DomReporter extends Reporter {
     this.stack = []; // previous test nodes
     this.current = null; // the current test node
   }
-  report(event) {
+  report(event, suppressStopTest = false) {
     event = this.state?.preprocess(event) || event;
     let text;
     switch (event.type) {
@@ -194,7 +194,7 @@ class DomReporter extends Reporter {
         }
         break;
     }
-    this.state?.postprocess(event);
+    this.state?.postprocess(event, suppressStopTest);
   }
 }