rooibos-roku 5.16.5 → 5.16.6

package/.mocharc.cjs

@@ -0,0 +1,15 @@
+const nodeVersion = +process.versions.node.split('.')[0];
+const config = {
+    spec: 'src/**/*.spec.ts',
+    require: [
+        'source-map-support/register',
+        'ts-node/register'
+    ],
+    fullTrace: true,
+    timeout: 2000,
+    watchExtensions: ['ts']
+};
+if (nodeVersion >= 22) {
+    config['node-option'] = ['no-experimental-strip-types'];
+}
+module.exports = config;

package/CHANGELOG.md

@@ -6,6 +6,27 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 
 
+## [5.16.6](https://github.com/rokucommunity/rooibos/compare/5.16.5...v5.16.6) - 2026-10-01
+### Changed
+ - Process every test suite in a file, not just the first ([#441](https://github.com/rokucommunity/rooibos/pull/441))
+ - Support enums and constants in @params annotations ([#438](https://github.com/rokucommunity/rooibos/pull/438))
+ - Security enhancements ([#426](https://github.com/rokucommunity/rooibos/pull/426), [#429](https://github.com/rokucommunity/rooibos/pull/429), [#439](https://github.com/rokucommunity/rooibos/pull/439))
+ - upgrade to [brighterscript@0.73.5](https://github.com/rokucommunity/brighterscript/blob/v0.73.5/CHANGELOG.md#0735---2026-09-15). Notable changes since 0.73.3:
+     - Fix crash stripping sourceMappingURL comment from non-transpiled files ([#1821](https://github.com/rokucommunity/brighterscript/pull/1821))
+     - Tolerate older BrsTranspileState in continue back-transpile ([#1812](https://github.com/rokucommunity/brighterscript/pull/1812))
+     - Security enhancements ([#1804](https://github.com/rokucommunity/brighterscript/pull/1804), [#1805](https://github.com/rokucommunity/brighterscript/pull/1805))
+     - Locate the super() call when injecting field initializers ([#1803](https://github.com/rokucommunity/brighterscript/pull/1803))
+ - upgrade to [roku-debug@0.24.6](https://github.com/rokucommunity/roku-debug/blob/v0.24.6/CHANGELOG.md#0246---2026-10-01). Notable changes since 0.24.3:
+     - Security enhancements ([#419](https://github.com/rokucommunity/roku-debug/pull/419), [#427](https://github.com/rokucommunity/roku-debug/pull/427))
+ - upgrade to [roku-deploy@4.0.0-alpha.9](https://github.com/rokucommunity/roku-deploy/blob/v4.0.0-alpha.9/CHANGELOG.md#400-alpha9---2026-09-28). Notable changes since 4.0.0-alpha.5:
+     - Security enhancements ([#422](https://github.com/rokucommunity/roku-deploy/pull/422))
+     - upgrade to [@rokucommunity/logger@0.4.2](https://github.com/rokucommunity/logger/blob/master/CHANGELOG.md#042---2026-09-02). Notable changes since 0.4.1:
+         - Security enhancements ([#38](https://github.com/rokucommunity/logger/pull/38), [#39](https://github.com/rokucommunity/logger/pull/39))
+### Fixed
+ - Fixed ignored tests count reporting in the summary output ([#435](https://github.com/rokucommunity/rooibos/pull/435))
+
+
+
 ## [5.16.5](https://github.com/rokucommunity/rooibos/compare/5.16.4...v5.16.5) - 2026-09-09
 ### Changed
  - Security enhancements ([#405](https://github.com/rokucommunity/rooibos/pull/405), [#406](https://github.com/rokucommunity/rooibos/pull/406), [#407](https://github.com/rokucommunity/rooibos/pull/407), [#409](https://github.com/rokucommunity/rooibos/pull/409), [#410](https://github.com/rokucommunity/rooibos/pull/410), [#411](https://github.com/rokucommunity/rooibos/pull/411), [#412](https://github.com/rokucommunity/rooibos/pull/412), [#414](https://github.com/rokucommunity/rooibos/pull/414), [#421](https://github.com/rokucommunity/rooibos/pull/421))

package/audit-ci.jsonc

@@ -3,13 +3,5 @@
     // why each allowlisted entry is safe to defer.
     "$schema": "https://github.com/IBM/audit-ci/raw/main/docs/schema.json",
     "high": true,
-    "allowlist": [
-        {
-            "GHSA-w5hq-g745-h8pq": {
-                // Added: 2026-05-22
-                "active": true,
-                "notes": "`uuid` <11.1.1 (via `postman-request` and dev-only `nyc>istanbul-lib-processinfo`): vulnerable code path is `v3()`/`v5()`/`v6()` when a caller-provided buffer is passed. All consumers in this dep graph call only `v4()` (no buffer arg), so the vulnerable path is unreachable."
-            }
-        }
-    ]
+    "allowlist": []
 }

package/dist/lib/framework/source/rooibos/ConsoleTestReporter.bs

@@ -72,12 +72,12 @@ namespace rooibos
             m.printLine(0, "  Passed: " + rooibos.common.AsString(m.allStats.passedCount))
             m.printLine(0, "  Crashed: " + rooibos.common.AsString(m.allStats.crashedCount))
             m.printLine(0, "  Failed: " + rooibos.common.AsString(m.allStats.failedCount))
-            m.printLine(0, "  Ignored: " + rooibos.common.AsString(ignoredInfo.count))
+            m.printLine(0, "  Ignored: " + rooibos.common.AsString(m.allStats.ignoredCount))
             m.printLine(0, "  Time: " + rooibos.common.AsString(m.allStats.time) + "ms")
             m.printLine()
             m.printLine()
 
-            if ignoredInfo.count > 0 then
+            if m.allStats.ignoredCount > 0 then
                 m.printLine(0, "IGNORED TESTS:")
                 for each ignoredItemName in ignoredInfo.items
                     m.printLine(1, ignoredItemName)

package/dist/lib/rooibos/Annotation.js

@@ -2,6 +2,7 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.getAnnotationType = exports.RooibosAnnotation = exports.AnnotationParams = exports.AnnotationType = void 0;
 const Diagnostics_1 = require("../utils/Diagnostics");
+const Utils_1 = require("./Utils");
 var AnnotationType;
 (function (AnnotationType) {
     AnnotationType["None"] = "none";
@@ -46,9 +47,12 @@ let annotationLookup = {
 };
 const defaultSlowDuration = 75;
 class AnnotationParams {
-    constructor(annotation, text, lineNumber, params, isIgnore = false, isSolo = false, noCatch = false, noEarlyexit = false) {
+    constructor(annotation, lineNumber, 
+    /**
+     * The argument expressions of the annotation. These are cloned into the generated test suite data so they get transpiled like any other code
+     */
+    params, isIgnore = false, isSolo = false, noCatch = false, noEarlyexit = false) {
         this.annotation = annotation;
-        this.text = text;
         this.lineNumber = lineNumber;
         this.params = params;
         this.isIgnore = isIgnore;
@@ -167,6 +171,7 @@ class RooibosAnnotation {
                     asyncTimeout = -1;
                 }
                 for (const annotation of getAnnotationsOfType(AnnotationType.Params, AnnotationType.SoloParams, AnnotationType.IgnoreParams)) {
+                    (0, Utils_1.linkAnnotationToStatement)(annotation, statement);
                     if (testAnnotation) {
                         testAnnotation.parseParams(file, annotation, getAnnotationType(annotation.name), noCatch);
                     }
@@ -194,23 +199,13 @@ class RooibosAnnotation {
         return { blockAnnotation: blockAnnotation, testAnnotation: testAnnotation };
     }
     parseParams(file, annotation, annotationType, noCatch) {
-        let rawParams = JSON.stringify(annotation.getArguments());
+        var _a, _b;
         let isSolo = annotationType === AnnotationType.SoloParams;
         let isIgnore = annotationType === AnnotationType.IgnoreParams;
         if (isSolo) {
             this.hasSoloParams = true;
         }
-        try {
-            if (rawParams) {
-                this.params.push(new AnnotationParams(annotation, rawParams, annotation.range.start.line, annotation.getArguments(), isIgnore, isSolo, noCatch));
-            }
-            else {
-                (0, Diagnostics_1.diagnosticIllegalParams)(file, annotation);
-            }
-        }
-        catch (e) {
-            (0, Diagnostics_1.diagnosticIllegalParams)(file, annotation);
-        }
+        this.params.push(new AnnotationParams(annotation, annotation.range.start.line, (_b = (_a = annotation.call) === null || _a === void 0 ? void 0 : _a.args) !== null && _b !== void 0 ? _b : [], isIgnore, isSolo, noCatch));
     }
 }
 exports.RooibosAnnotation = RooibosAnnotation;

package/dist/lib/rooibos/TestCase.js

@@ -21,7 +21,6 @@ class TestCase {
         this.isParamTest = this.expectedNumberOfParams > 0;
     }
     asText() {
-        let rawParamsText = this.fixBadJson(this.rawParams).replace(/null/g, 'invalid');
         return `
         {
           isSolo: ${this.isSolo}
@@ -36,28 +35,12 @@ class TestCase {
           lineNumber: ${this.lineNumber + 1}
           paramLineNumber: ${this.isParamTest ? this.paramLineNumber + 1 : 0}
           assertIndex: ${this.assertIndex}
-          rawParams: ${rawParamsText}
+          rawParams: ${this.rawParams ? '[]' /* filled with the actual param expressions by `addParamsToTestSuiteData` */ : 'invalid'}
           paramTestIndex: ${this.paramTestIndex}
           expectedNumberOfParams: ${this.expectedNumberOfParams}
           isParamsValid: ${(this.rawParams || []).length === this.expectedNumberOfParams}
         }`;
     }
-    fixBadJson(o) {
-        // In case of an array we'll stringify all objects.
-        if (Array.isArray(o)) {
-            return `[${o.map(obj => `${this.fixBadJson(obj)}`).join(',')}]`;
-        }
-        // not an object, stringify using native function
-        if (typeof o !== 'object' || o === null) {
-            return JSON.stringify(o);
-        }
-        return `{${Object
-            .keys(o)
-            .map(key => {
-            return `"${key.replace(/"/g, '')}":${this.fixBadJson(o[key])}`;
-        })
-            .join(',')}}`;
-    }
 }
 exports.TestCase = TestCase;
 //# sourceMappingURL=TestCase.js.map

package/dist/lib/rooibos/TestSuite.js

@@ -78,7 +78,17 @@ class TestSuite extends TestBlock {
     }
     addDataFunctions(editor) {
         if (this.isIncluded) {
-            (0, Utils_1.addOverriddenMethod)(this.file, this.annotation.annotation, this.classStatement, 'getTestSuiteData', `return ${this.asText()}`, editor);
+            const method = (0, Utils_1.addOverriddenMethod)(this.file, this.annotation.annotation, this.classStatement, 'getTestSuiteData', `return ${this.asText()}`, editor);
+            if (method) {
+                //same order as the `rawParams` placeholders in `asText()`
+                const paramExpressionsList = [];
+                for (const testGroup of [...this.testGroups.values()].filter((tg) => tg.isIncluded)) {
+                    for (const testCase of [...testGroup.testCases].filter((tc) => tc.isIncluded && tc.rawParams)) {
+                        paramExpressionsList.push(testCase.rawParams);
+                    }
+                }
+                (0, Utils_1.addParamsToTestSuiteData)(this.file, method, paramExpressionsList, editor);
+            }
         }
     }
     getTestGroups() {

package/dist/lib/rooibos/Utils.js

@@ -1,8 +1,12 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.getScopeForSuite = exports.getPathValuePartAsString = exports.getStringPathFromDottedGet = exports.getRootObjectFromDottedGet = exports.getAllDottedGetParts = exports.functionRequiresReturnValue = exports.sanitizeBsJsonString = exports.addOverriddenMethod = void 0;
+exports.getScopeForSuite = exports.addParamsToTestSuiteData = exports.linkAnnotationToStatement = exports.getPathValuePartAsString = exports.getStringPathFromDottedGet = exports.getRootObjectFromDottedGet = exports.getAllDottedGetParts = exports.functionRequiresReturnValue = exports.sanitizeBsJsonString = exports.addOverriddenMethod = void 0;
 const brighterscript_1 = require("brighterscript");
 const Diagnostics_1 = require("../utils/Diagnostics");
+/**
+ * Add a generated method to the class.
+ * @returns the added method, or undefined if the source could not be parsed
+ */
 function addOverriddenMethod(file, annotation, target, name, source, editor) {
     var _a, _b;
     let { method, diagnostics, text } = createMethod(file, name, source);
@@ -11,11 +15,12 @@ function addOverriddenMethod(file, annotation, target, name, source, editor) {
         //trigger that functionality BEFORE performing AstEditor operations. TODO remove this whenever bsc stops doing this.
         (_b = (_a = target).ensureConstructorFunctionExists) === null || _b === void 0 ? void 0 : _b.call(_a);
         editor.addToArray(target.body, target.body.length, method);
-        return true;
+        method.parent = target;
+        return method;
     }
     const error = (diagnostics === null || diagnostics === void 0 ? void 0 : diagnostics.length) > 0 ? diagnostics[0].message : 'unknown error';
     (0, Diagnostics_1.diagnosticCorruptTestProduced)(file, annotation, error, text);
-    return false;
+    return undefined;
 }
 exports.addOverriddenMethod = addOverriddenMethod;
 /**
@@ -130,6 +135,73 @@ function getPathValuePartAsString(expr) {
     }
 }
 exports.getPathValuePartAsString = getPathValuePartAsString;
+/**
+ * bsc does not link annotations into the AST, so the expressions in their arguments have no parent (and therefore no symbol table or namespace).
+ * That makes the bsc validator flag references like `@params(SomeEnum.value)` as unknown names. Link the annotation (and its arguments) to
+ * the statement it decorates so those references are validated and resolved like any other expression in that statement's scope.
+ */
+function linkAnnotationToStatement(annotation, statement) {
+    if (!(annotation === null || annotation === void 0 ? void 0 : annotation.call)) {
+        return;
+    }
+    (0, brighterscript_1.walkArray)(annotation.call.args, () => { }, { walkMode: brighterscript_1.WalkMode.visitAllRecursive }, annotation.call);
+    annotation.call.parent = annotation;
+    annotation.parent = statement;
+}
+exports.linkAnnotationToStatement = linkAnnotationToStatement;
+/**
+ * Fill the `rawParams: []` placeholders in a generated `getTestSuiteData` method with clones of each test case's actual `@params` argument expressions.
+ * The clones are registered in the file's references, so bsc's own pre-transpile processing (i.e. inlining enums and constants) applies to them
+ * exactly like it does for handwritten code. This must run before bsc's `beforeFileTranspile` (i.e. during `beforeProgramTranspile`).
+ * @param file the file containing the test suite
+ * @param method the generated `getTestSuiteData` method
+ * @param paramExpressionsList the `@params` argument expressions for each placeholder, in the order they appear in the method
+ * @param editor the editor used to make (and later undo) the changes
+ */
+function addParamsToTestSuiteData(file, method, paramExpressionsList, editor) {
+    const placeholders = [];
+    method.walk((node) => {
+        if ((0, brighterscript_1.isAAMemberExpression)(node) && node.keyToken.text === 'rawParams' && (0, brighterscript_1.isArrayLiteralExpression)(node.value)) {
+            placeholders.push(node.value);
+        }
+    }, { walkMode: brighterscript_1.WalkMode.visitExpressionsRecursive });
+    const references = new Set();
+    const addReferences = (expression) => {
+        references.add(expression);
+        if ((0, brighterscript_1.isArrayLiteralExpression)(expression)) {
+            for (const element of expression.elements) {
+                if (!(0, brighterscript_1.isCommentStatement)(element)) {
+                    addReferences(element);
+                }
+            }
+        }
+        else if ((0, brighterscript_1.isAALiteralExpression)(expression)) {
+            for (const member of expression.elements) {
+                if ((0, brighterscript_1.isAAMemberExpression)(member)) {
+                    addReferences(member.value);
+                }
+            }
+        }
+    };
+    for (let i = 0; i < placeholders.length && i < paramExpressionsList.length; i++) {
+        const clones = paramExpressionsList[i].map(x => x.clone());
+        editor.arrayPush(placeholders[i].elements, ...clones);
+        //link the clones into the AST
+        (0, brighterscript_1.walkArray)(placeholders[i].elements, () => { }, { walkMode: brighterscript_1.WalkMode.visitAllRecursive }, placeholders[i]);
+        clones.forEach(addReferences);
+    }
+    const fileReferences = file.parser.references.expressions;
+    editor.edit(() => {
+        for (const expression of references) {
+            fileReferences.add(expression);
+        }
+    }, () => {
+        for (const expression of references) {
+            fileReferences.delete(expression);
+        }
+    });
+}
+exports.addParamsToTestSuiteData = addParamsToTestSuiteData;
 function getScopeForSuite(testSuite) {
     if (testSuite.isNodeTest) {
         return testSuite.file.program.getScopesForFile(testSuite.file).find((scope) => {

package/dist/plugin.js

@@ -130,14 +130,18 @@ class RooibosPlugin {
     }
     beforeProgramTranspile(program, entries, editor) {
         this.session.prepareForTranspile(editor, program, this.mockUtil);
+        //this must happen before bsc's `beforeFileTranspile` (which runs before ours), so bsc will inline any enums/constants used in the `@params`
+        for (const testSuite of this.session.sessionInfo.testSuitesToRun) {
+            testSuite.addDataFunctions(editor);
+        }
     }
     afterProgramTranspile(program, entries, editor) {
         this.session.addLaunchHookFileIfNotPresent();
         this.codeCoverageProcessor.generateMetadata(this.config.isRecordingCodeCoverage, program);
     }
     beforeFileTranspile(event) {
-        let testSuite = this.session.sessionInfo.testSuitesToRun.find((ts) => ts.file.pkgPath === event.file.pkgPath);
-        if (testSuite) {
+        const testSuites = this.session.sessionInfo.testSuitesToRun.filter((ts) => ts.file.pkgPath === event.file.pkgPath);
+        for (const testSuite of testSuites) {
             const scope = (0, Utils_1.getScopeForSuite)(testSuite);
             let noEarlyExit = testSuite.annotation.noEarlyExit;
             if (noEarlyExit) {
@@ -145,7 +149,6 @@ class RooibosPlugin {
             }
             const modifiedTestCases = new Set();
             const modifiedHookFunctions = new Set();
-            testSuite.addDataFunctions(event.editor);
             for (let group of [...testSuite.testGroups.values()].filter((tg) => tg.isIncluded)) {
                 for (const hookName of [group.setupFunctionName, group.tearDownFunctionName, group.beforeEachFunctionName, group.afterEachFunctionName]) {
                     if (hookName) {

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "rooibos-roku",
-    "version": "5.16.5",
+    "version": "5.16.6",
     "description": "simple, flexible, fun brightscript test framework for roku scenegraph apps - roku brighterscript plugin",
     "scripts": {
         "preversion": "npm run build && npm run lint && npm run test",
@@ -20,12 +20,12 @@
         "package": "npm run build && npm pack"
     },
     "dependencies": {
-        "brighterscript": "^0.73.3",
+        "brighterscript": "^0.73.5",
         "fast-glob": "^3.2.12",
         "fs-extra": "^10.1.0",
         "minimatch": "^3.0.4",
-        "roku-debug": "^0.24.3",
-        "roku-deploy": "^4.0.0-alpha.5",
+        "roku-debug": "^0.24.6",
+        "roku-deploy": "^4.0.0-alpha.9",
         "rooibos_promises": "npm:@rokucommunity/promises@^0.5.0",
         "source-map": "^0.7.3",
         "undent": "^1.0.1",
@@ -62,10 +62,13 @@
         "typescript": "^4.7.2"
     },
     "overrides": {
-        "serialize-javascript": "^7.0.5",
+        "serialize-javascript": "^7",
         "js-yaml@^3": "^3.15.2",
-        "js-yaml@^4": "^4.3.2",
-        "browserslist": "^4.28.8"
+        "js-yaml@^4": "^4",
+        "browserslist": "^4.28.8",
+        "istanbul-lib-processinfo": "^3.0.1",
+        "uuid": "^11.1.1",
+        "@babel/core": "^7.29.7"
     },
     "main": "dist/plugin.js",
     "bin": {
@@ -113,17 +116,6 @@
         "functions": 5,
         "branches": 5
     },
-    "mocha": {
-        "spec": "src/**/*.spec.ts",
-        "fullTrace": true,
-        "require": [
-            "source-map-support/register",
-            "ts-node/register"
-        ],
-        "watchExtensions": [
-            "ts"
-        ]
-    },
     "ropm": {
         "rootDir": "./framework/src"
     }