Fix browser support bug (#111)

* Add checks for availability of `process` to ensure browser compat.
* Add browser tests using jasmine-browser-runner

Author: tonypconway

package-lock.json

@@ -32,7 +32,8 @@
     "test:lint": "npx eslint .",
     "test:browserslist": "mkdir test-browserslist && cd test-browserslist && npm init -y && npm i ../../baseline-browser-mapping browserslist &&jq '. += {\"browserslist\":[\"baseline widely available with downstream\"]}' package.json >p && mv p package.json && npx browserslist && cd ../ && rm -rf test-browserslist",
     "test:jasmine": "npx jasmine",
-    "test": "npm run build && npm run fix-cli-permissions && rm -rf test-browserslist && npm run test:format && npm run test:lint && npx jasmine && npm run test:browserslist",
+    "test:jasmine-browser": "npx jasmine-browser-runner runSpecs --config ./spec/support/jasmine-browser.js",
+    "test": "npm run build && npm run fix-cli-permissions && rm -rf test-browserslist && npm run test:format && npm run test:lint && npm run test:jasmine && npm run test:jasmine-browser && npm run test:browserslist",
     "build": "rm -rf dist; npx prettier . --write; rollup -c; rm -rf ./dist/scripts/expose-data.d.ts ./dist/cli.d.ts",
     "refresh-downstream": "npx tsx scripts/refresh-downstream.ts",
     "refresh-static": "npx tsx scripts/refresh-static.ts",
@@ -48,6 +49,7 @@
     "@types/node": "^22.15.17",
     "eslint-plugin-new-with-error": "^5.0.0",
     "jasmine": "^5.8.0",
+    "jasmine-browser-runner": "^3.0.0",
     "jasmine-spec-reporter": "^7.0.0",
     "prettier": "^3.5.3",
     "rollup": "^4.44.0",

package.json

@@ -0,0 +1,35 @@
+export default {
+  srcDir: "src",
+  // srcFiles should usually be left empty when using ES modules, because you'll
+  // explicitly import sources from your specs.
+  srcFiles: [],
+  specDir: ".",
+  specFiles: ["spec/tests/index.browser.js"],
+  // helpers: ["spec/helpers/**/*.?(m)js"],
+  esmFilenameExtension: ".js",
+  // Set to true if you need to load module src files instead of loading via the spec files.
+  modulesWithSideEffectsInSrcFiles: false,
+  // Allows the use of top-level await in src/spec/helper files. This is off by
+  // default because it makes files load more slowly.
+  enableTopLevelAwait: false,
+  env: {
+    stopSpecOnExpectationFailure: false,
+    stopOnSpecFailure: false,
+    random: true,
+    // Fail if a suite contains multiple suites or specs with the same name.
+    forbidDuplicateNames: true,
+  },
+
+  // For security, listen only to localhost. You can also specify a different
+  // hostname or IP address, or remove the property or set it to "*" to listen
+  // to all network interfaces.
+  listenAddress: "localhost",
+
+  // The hostname that the browser will use to connect to the server.
+  hostname: "localhost",
+
+  browser: {
+    name: "headlessChrome",
+  },
+  useConsoleReporter: "true",
+};

spec/support/jasmine-browser.js

@@ -1,7 +1,7 @@
 export default {
   spec_dir: "spec",
   helpers: ["helpers/**/*.?(m)js"],
-  spec_files: ["tests/*.js"],
+  spec_files: ["tests/*.node.js"],
   env: {
     stopSpecOnExpectationFailure: false,
     random: true,

spec/support/jasmine.js

@@ -0,0 +1,22 @@
+import { getCompatibleVersions, getAllVersions } from "../../dist/index.js";
+
+describe("baseline-browser-mapping in browser", function () {
+  it("should load getCompatibleVersions", function () {
+    expect(typeof getCompatibleVersions).toBe("function");
+  });
+
+  it("should load getAllVersions", function () {
+    expect(typeof getAllVersions).toBe("function");
+  });
+
+  it("getCompatibleVersions should return an array", function () {
+    const versions = getCompatibleVersions();
+    expect(Array.isArray(versions)).toBe(true);
+  });
+
+  it("getAllVersions should return an object", function () {
+    const versions = getAllVersions();
+    expect(typeof versions).toBe("object");
+    expect(versions).not.toBeNull();
+  });
+});

spec/tests/cli.js renamed to spec/tests/cli.node.js

@@ -6,7 +6,10 @@ import {
 } from "./scripts/expose-data.js";
 
 try {
-  if (typeof process.loadEnvFile === "function") {
+  if (
+    typeof process !== "undefined" &&
+    typeof process.loadEnvFile === "function"
+  ) {
     process.loadEnvFile();
   }
 } catch (e) {
@@ -22,8 +25,10 @@ export function _resetHasWarned() {
 const checkUpdate = (targetDate: Date, lastUpdatedOverride?: number) => {
   if (
     hasWarned ||
-    process.env.BROWSERSLIST_IGNORE_OLD_DATA ||
-    process.env.BASELINE_BROWSER_MAPPING_IGNORE_OLD_DATA
+    (typeof process !== "undefined" &&
+      process.env &&
+      (process.env.BROWSERSLIST_IGNORE_OLD_DATA ||
+        process.env.BASELINE_BROWSER_MAPPING_IGNORE_OLD_DATA))
   ) {
     return;
   }
@@ -144,7 +149,13 @@ const kaiOSWarning = (options: Options | AllVersionsOptions) => {
         "KaiOS is a downstream browser and can only be included if you include other downstream browsers. Please ensure you use `includeDownstreamBrowsers: true`.",
       ),
     );
-    process.exit(1);
+    if (typeof process !== "undefined" && process.exit) {
+      process.exit(1);
+    } else {
+      throw new Error(
+        "KaiOS configuration error: process.exit is not available",
+      );
+    }
   }
 };
 
@@ -469,7 +480,13 @@ export function getCompatibleVersions(userOptions?: Options): BrowserVersion[] {
         "You cannot use targetYear and widelyAvailableOnDate at the same time.  Please remove one of these options and try again.",
       ),
     );
-    process.exit(1);
+    if (typeof process !== "undefined" && process.exit) {
+      process.exit(1);
+    } else {
+      throw new Error(
+        "Configuration error: targetYear and widelyAvailableOnDate cannot be used together",
+      );
+    }
   } else if (options.widelyAvailableOnDate) {
     targetDate = new Date(options.widelyAvailableOnDate);
   } else if (options.targetYear) {