risei 3.1.0 → 3.1.1

package/README.md

@@ -46,7 +46,7 @@ And they have a summary bar at the bottom:
 
 ## Status
 
-Risei's major features are now complete, but new enhancements or fixes may appear from time to time.   The latest release, **3.1.0**, includes significant improvements and bug fixes, on top of the major improvements in releases **3.0.0** and **2.0.0**.
+Risei's major features are now complete, but new enhancements or fixes may appear from time to time.   The latest release, **3.1.1**, includes significant improvements and bug fixes, on top of the major improvements in releases **3.0.0** and **2.0.0**.
 
 Check out the [full list of changes](#version-history).
 
@@ -219,10 +219,14 @@ If errors are thrown while testing, gold bars listing them appear at the bottom,
 
 ## Version history
 
+- Release **3.1.1** (January, 2025) contains these changes:
+  - Extra commas and non-objects in arrays of test objects are disregarded, and no longer cause a throw.
+  - Throws in wider scopes are now listed at the end, like others already found there.
+
 - Release **3.1.0** (January, 2025) contains these changes:
   - Accessor properties (formally _accessor descriptors_) are now compared for test results and displayed in test outputs.
     - Any accessor properties that throw errors during display are displayed as `(threw)`.
-  - Throws in tested code and in test framing code are now listed at the end and displayed amid the tests.
+  - Throws in tested code and in test framing code are now listed at the end, and also displayed amid the tests.
   - `File` objects now have a succinct custom display in outputs.
   - **(Breaking change)**   In `constructor` tests, constructed instances are available in `.from` functions only as `actual` / `test.actual`.
     - **(Breaking change)**   In `constructor` tests, `test.target` is now the `prototype` of the tested class.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "risei",
-  "version": "3.1.0",
+  "version": "3.1.1",
   "description": "Risei allows you to write unit tests as simple JavaScript objects, so it's easy and fast, with no drag on redesign.",
   "keywords": [
     "unit test",
@@ -43,7 +43,7 @@
     "fs": "^0.0.1-security",
     "minimatch": "^9.0.1",
     "npm": "^11.0.0",
-    "risei": "^3.0.0"
+    "risei": "^3.1.0"
   },
   "devDependencies": {
     "chai": "^4.3.6",

package/system/TerminalReporter.js

@@ -289,18 +289,25 @@ export default class TerminalReporter {
 
         console.log(this.#summary(this.#toFullWidth(` ${ summary.summary }`)));
 
-        /* Post-summary reporting of unsought throws. */
+        /* Post-summary reporting of any throws. */
 
+        // Unsought throws in tested code.
         if (summary.anyThrows.length !== 0) {
             console.log();
             console.log(this.#problems(summary.anyThrows.join("\n")));
         }
 
-        // And/or reporting of outer-scope throws.
+        // Throws in code in scope of test frame.
         if (summary.anyTroubles.length !== 0) {
             console.log();
             console.log(this.#problems(summary.anyTroubles.join("\n")));
         }
+
+        // Throws in code in scope of test runner.
+        if (summary.anyCaught.length !== 0) {
+            console.log();
+            console.log(this.#problems(summary.anyCaught.join("\n")));
+        }
    }
 
     // endregion Summary dependency of reportNext()

package/system/TestDef.js

@@ -344,6 +344,11 @@ export default class TestDef {
 
         // Looping over all.
         for (let nonce of nonces) {
+            // No possible test definition to output.
+            if (!TestDef.isANonce(nonce)) {
+                continue;
+            }
+
             // Get latest test definition.
             let latest = TestDef.fromNonceTuple(nonce);
 
@@ -398,6 +403,11 @@ export default class TestDef {
 
     // region Dependencies of nonce-tuple initing methods
 
+    static isANonce(topic) /* passed */ {
+        let type = TypeIdentifier.identify(topic);
+        return type === Types.isObject;
+    }
+
     /* Replaces some or all of the 'full' that gathers test 
        props for reuse when targeting changes significantly 
        in the middle of the collapsing-forward process. */

package/system/TestRunner.js

@@ -26,8 +26,10 @@ export default class TestRunner {
     #numberPassed;
     #numberFailed;
 
-    #thrown;
-    #trouble;
+    // Problems in different scopes.
+    #thrown;  // In tested code.
+    #trouble;  // In framing code.
+    #caught;  // In outer system.
 
     #allDidPass;
 
@@ -100,6 +102,7 @@ export default class TestRunner {
 
         this.#thrown = [];
         this.#trouble = [];
+        this.#caught = [];
 
         this.#allDidPass = true;
     }
@@ -124,37 +127,43 @@ export default class TestRunner {
             );
         }
     
-        // Converting so that property names can be relied on.
-        this.#tests = TestDef.fromNonceTuples(this.tests);
-
-        // Needed for displaying classes and methods as groups.
-        let classGroup = new ClassTestGroup();
-        let methodGroup = new MethodTestGroup();
-        let atFirstClassMethod = false;
-
-        // Iterative running of all tests in current order.
-        for (let test of this.#tests) {
-            // Each new class should be a group, and its start 
-            // should be retained for grouping of methods.
-            if (test.on.name !== classGroup.group) {
-                classGroup.group = test.on.name;
-                atFirstClassMethod = true;
-                yield classGroup;
+        try {
+            // Converting so that property names can be relied on.
+            this.#tests = TestDef.fromNonceTuples(this.tests);
+
+            // Needed for displaying classes and methods as groups.
+            let classGroup = new ClassTestGroup();
+            let methodGroup = new MethodTestGroup();
+            let atFirstClassMethod = false;
+
+            // Iterative running of all tests in current order.
+            for (let test of this.#tests) {
+                // Each new class should be a group, and its start 
+                // should be retained for grouping of methods.
+                if (test.on.name !== classGroup.group) {
+                    classGroup.group = test.on.name;
+                    atFirstClassMethod = true;
+                    yield classGroup;
+                }
+
+                // Each new method / prop name should be a group, 
+                // and so should the first method for a class.
+                let runName = test.runName;
+
+                if (runName !== methodGroup.group || atFirstClassMethod) {
+                    methodGroup.group = runName;
+                    atFirstClassMethod = false;
+                    yield methodGroup;
+                }
+
+                let result = await this.#runOneTest(test);
+
+                yield result;
             }
-
-            // Each new method / prop name should be a group, 
-            // and so should the first method for a class.
-            let runName = test.runName;
-
-            if (runName !== methodGroup.group || atFirstClassMethod) {
-                methodGroup.group = runName;
-                atFirstClassMethod = false;
-                yield methodGroup;
-            }
-
-            let result = await this.#runOneTest(test);
-
-            yield result;
+        }
+        catch (caught) {
+            // Top of .stack is .message.
+            this.#caught.push(caught.stack);
         }
 
         // Iterative returning of final results summary.
@@ -217,7 +226,7 @@ export default class TestRunner {
 
         let anyWereRun = this.#numberRun > 0;
 
-        return new TestSummary(summary, this.#allDidPass, anyWereRun, this.#thrown, this.#trouble);
+        return new TestSummary(summary, this.#allDidPass, anyWereRun, this.#thrown, this.#trouble, this.#caught);
     }
 
     // endregion Running tests

package/system/TestSummary.js

@@ -7,6 +7,7 @@ export default class TestSummary {
 
     #anyThrows;
     #anyTroubles;
+    #anyCaught;
     
     get summary() {
         return this.#summary;
@@ -28,11 +29,16 @@ export default class TestSummary {
         return this.#anyTroubles;
     }
 
-    constructor(summary, allDidPass, anyWereRun, anyThrows, anyTroubles) {
+    get anyCaught() {
+        return this.#anyCaught;
+    }
+
+    constructor(summary, allDidPass, anyWereRun, anyThrows, anyTroubles, anyCaught) {
         this.#summary = summary;
         this.#allDidPass = allDidPass;
         this.#anyWereRun = anyWereRun;
         this.#anyThrows = anyThrows;
         this.#anyTroubles = anyTroubles;
+        this.#anyCaught = anyCaught;
     }
 }