Attempt at fixing tests

.github/workflows/js-sdk-ci.yml

@@ -21,7 +21,7 @@ jobs:
         exclude:
           - node-version: 14.x
             os: macOS-latest
-    name: Node.js ${{ matrix.node-version }} (${{ matrix.node-version }}) test
+    name: Node.js ${{ matrix.node-version }} test
     steps:
       - uses: actions/checkout@v3
 

.vscode/launch.json

@@ -6,15 +6,15 @@
       "request": "launch",
       "name": "Run tests",
       "cwd": "${workspaceRoot}",
-      "env": { "TS_NODE_PROJECT": "./tsconfig.mocha.json" },
+      "env": { "TS_NODE_PROJECT": "./tsconfig.mocha.node.json" },
       "runtimeArgs": [
         "--expose-gc"
       ],
       "program": "${workspaceRoot}/node_modules/mocha/bin/_mocha",
       "args": [
         "--require",
         "ts-node/register",
-        "test/**/*.ts",
+        "test/node/index.ts",
         "--fgrep",
         "",
         "--color",

test/ConfigCatClientTests.ts

@@ -525,11 +525,13 @@ describe("ConfigCatClient", () => {
       setupHooks: hooks => hooks.on("configChanged", () => configChangedEventCount++)
     };
     const options: AutoPollOptions = new AutoPollOptions("APIKEY", "common", "1.0.0", userOptions, null);
-    new ConfigCatClient(options, configCatKernel);
-
-    await delay(2.5 * pollIntervalSeconds * 1000);
+    const client = new ConfigCatClient(options, configCatKernel);
+    try {
+      await delay(2.5 * pollIntervalSeconds * 1000);
 
-    assert.equal(configChangedEventCount, 3);
+      assert.equal(configChangedEventCount, 3);
+    }
+    finally { client.dispose(); }
   });
 
   it("Initialization With AutoPollOptions - config doesn't change - should fire configChanged only once", async () => {
@@ -543,11 +545,13 @@ describe("ConfigCatClient", () => {
       setupHooks: hooks => hooks.on("configChanged", () => configChangedEventCount++)
     };
     const options: AutoPollOptions = new AutoPollOptions("APIKEY", "common", "1.0.0", userOptions, null);
-    new ConfigCatClient(options, configCatKernel);
-
-    await delay(2.5 * pollIntervalSeconds * 1000);
+    const client = new ConfigCatClient(options, configCatKernel);
+    try {
+      await delay(2.5 * pollIntervalSeconds * 1000);
 
-    assert.equal(configChangedEventCount, 1);
+      assert.equal(configChangedEventCount, 1); 
+    }
+    finally { client.dispose(); }
   });
 
   it("Initialization With AutoPollOptions - with maxInitWaitTimeSeconds - getValueAsync should wait", async () => {

test/DefaultEventEmitterTests.ts

@@ -1,5 +1,6 @@
 import { assert } from "chai";
 import { DefaultEventEmitter } from "../src/DefaultEventEmitter";
+import "core-js/features/array/flat.js";
 
 function createHandler(eventName: string, capturedArgs: any[][]) {
   return (...args: any[]) => capturedArgs.push([eventName, args]);

test/helpers/fakes.ts

@@ -185,8 +185,10 @@ export class FakeConfigFetcherWithAlwaysVariableEtag extends FakeConfigFetcher {
     return '{"f":{"debug":{"t":0,"v":{"b":true},"i":"abcdefgh"}}}';
   }
 
+  private eTag = 0;
+
   getEtag(): string {
-    return Math.random().toString();
+    return `"${(this.eTag++).toString(16).padStart(8, "0")}"`;
   }
 }
 

webpack.karma.chromium-extension.config.js

@@ -41,6 +41,22 @@ module.exports = new function(options) {
               configFile: "tsconfig.karma.chromium-extension.json"
             }
           }]
+        },
+        // Some test dependencies may use modern ES features which are not supported
+        // by all the targets defined in our CI workflow.
+        // So these dependencies must be downleveled to not break the checks.
+        // (@babel/preset-env transpiles to ES5 by default.)
+        {
+          test: /\.m?js$/,
+          include: [
+            path.resolve("node_modules/mock-xmlhttprequest")
+          ],
+          use: [{
+            loader: "babel-loader",
+            options: {
+              "presets": ["@babel/preset-env"]
+            }
+          }]
         }
       ]
     }