Update Dependencies and Modernise Codebase

.github/workflows/run-tests.yaml

@@ -13,7 +13,7 @@ jobs:
       - uses: actions/checkout@v2
       - uses: actions/setup-node@v2
         with:
-          node-version: '14'
+          node-version: '22'
       - uses: nanasess/setup-chromedriver@master
       - name: "Install deps"
         run: npm install
@@ -23,6 +23,8 @@ jobs:
         run: npm run test:unit
       - name: "Run end-to-end tests"
         run: |
+          google-chrome --version
+          chromedriver --version
           export DISPLAY=:99
           chromedriver --url-base=/wd/hub &
           sudo Xvfb -ac :99 -screen 0 1280x1024x24 > /dev/null 2>&1 &

end-to-end-tests/sdk-get-registration-state.test.js

@@ -39,7 +39,7 @@ test('.getState should return PERMISSION_GRANTED_REGISTERED_WITH_BEAMS if start
   });
 
   expect(state).toBe(PusherPushNotifications.RegistrationState.PERMISSION_GRANTED_REGISTERED_WITH_BEAMS);
-});
+}, 60000);
 
 test('.getState should return PERMISSION_PROMPT_REQUIRED if start has not been called and permissions are default', async () => {
   await prepareServer(NOTIFICATIONS_DEFAULT)

end-to-end-tests/sdk-set-user-id.test.js

@@ -6,48 +6,43 @@ import {
   unregisterServiceWorker
 } from './test-utils';
 
+jest.setTimeout(60000);
 let killServer = null;
 let chromeDriver = null;
 
-beforeAll(() => {
-  return launchServer()
-    .then(killFunc => {
-      killServer = killFunc;
-    })
-    .then(() => createChromeWebDriver(NOTIFICATIONS_GRANTED))
-    .then(driver => {
-      chromeDriver = driver;
-    })
-    .then(async () => {
-      const errolClient = new ErrolTestClient(
-        '1b880590-6301-4bb5-b34f-45db1c5f5644'
-      );
-      const response = await errolClient.deleteUser('cucas');
-      expect(response.statusCode).toBe(200);
-    });
+beforeAll(async () => {
+  killServer = await launchServer();
+  chromeDriver = await createChromeWebDriver(NOTIFICATIONS_GRANTED);
+  const errolClient = new ErrolTestClient(
+    '1b880590-6301-4bb5-b34f-45db1c5f5644'
+  );
+  const response = await errolClient.deleteUser('cucas');
+  expect(response.statusCode).toBe(200);
 });
 
-beforeEach(() => {
-  return (async () => {
-    await chromeDriver.get('http://localhost:3000');
-    await chromeDriver.wait(() => {
-      return chromeDriver.getTitle().then(title => title.includes('Test Page'));
-    }, 2000);
-
-    return chromeDriver.executeAsyncScript(() => {
-      const asyncScriptReturnCallback = arguments[arguments.length - 1];
-
-      let deleteDbRequest = window.indexedDB.deleteDatabase(
-        'beams-1b880590-6301-4bb5-b34f-45db1c5f5644'
-      );
-      deleteDbRequest.onsuccess = asyncScriptReturnCallback;
-      deleteDbRequest.onerror = asyncScriptReturnCallback;
-    });
+beforeEach(async () => {
+  await chromeDriver.get('http://localhost:3000');
+
+  await chromeDriver.wait(async () => {
+    const title = await chromeDriver.getTitle();
+    return title.includes('Test Page');
+  },
+    2000
+  );
 
-  })();
+  await chromeDriver.executeAsyncScript(() => {
+    const callback = arguments[arguments.length - 1];
+    const req = window.indexedDB.deleteDatabase(
+      'beams-1b880590-6301-4bb5-b34f-45db1c5f5644'
+    );
+    req.onsuccess = callback;
+    req.onerror = callback;
+  });
 });
 
-afterEach(() => unregisterServiceWorker(chromeDriver));
+afterEach(async () => {
+  await unregisterServiceWorker(chromeDriver);
+});
 
 test('SDK should set user id with errol', async () => {
   await chromeDriver.get('http://localhost:3000');
@@ -81,8 +76,8 @@ test('SDK should set user id with errol', async () => {
         if (userId !== 'cucas') {
           throw new Error(
             'Unexpected user ID ' +
-              userId +
-              ', this token provider is hardcoded to "cucas"'
+            userId +
+            ', this token provider is hardcoded to "cucas"'
           );
         } else {
           return {
@@ -112,7 +107,7 @@ test('SDK should set user id with errol', async () => {
   const response = await errolClient.getWebDevice(deviceId);
   expect(response.statusCode).toBe(200);
   expect(JSON.parse(response.body).userId).toBe('cucas');
-});
+}, 60000);
 
 test('SDK should return an error if we try to reassign the user id', async () => {
   await chromeDriver.get('http://localhost:3000');
@@ -272,11 +267,7 @@ test('SDK should return an error if user ID is not a string', async () => {
   expect(setUserIdError).toBe('User ID must be a string (was undefined)');
 });
 
-afterAll(() => {
-  if (killServer) {
-    killServer();
-  }
-  if (chromeDriver) {
-    chromeDriver.quit();
-  }
+afterAll(async () => {
+  if (killServer) killServer();
+  if (chromeDriver) await chromeDriver.quit();
 });

end-to-end-tests/sdk-start.test.js

@@ -2,7 +2,8 @@ import {
   launchServer,
   createChromeWebDriver,
   NOTIFICATIONS_GRANTED,
-  unregisterServiceWorker
+  unregisterServiceWorker,
+  SCRIPT_TIMEOUT_MS
 } from './test-utils';
 
 let killServer = null;
@@ -24,23 +25,28 @@ afterEach(() => unregisterServiceWorker(chromeDriver));
 test('SDK should register a device with errol', async () => {
   await chromeDriver.get('http://localhost:3000');
   await chromeDriver.wait(() => {
-    return chromeDriver.getTitle().then(title => title.includes('Test Page'));
+    return chromeDriver.getTitle().then(title => {
+      return title.includes('Test Page');
+    });
   }, 2000);
 
   const initialDeviceId = await chromeDriver.executeAsyncScript(() => {
     const asyncScriptReturnCallback = arguments[arguments.length - 1];
-
     const instanceId = 'deadc0de-2ce6-46e3-ad9a-5c02d0ab119b';
     const beamsClient = new PusherPushNotifications.Client({ instanceId });
     beamsClient
       .start()
       .then(() => beamsClient.getDeviceId())
-      .then(deviceId => asyncScriptReturnCallback(deviceId))
-      .catch(e => asyncScriptReturnCallback(e.message));
+      .then(deviceId => {
+        asyncScriptReturnCallback(deviceId);
+      })
+      .catch(e => {
+        asyncScriptReturnCallback(e.message);
+      });
   });
 
   expect(initialDeviceId).toContain('web-');
-});
+}, 60000);
 
 test('SDK should remember the device ID', async () => {
   await chromeDriver.get('http://localhost:3000');

end-to-end-tests/sdk-stop.test.js

@@ -104,7 +104,7 @@ test('Calling .stop should clear SDK state', async () => {
   // Assert that the device no longer exists on the server
   const response = await errolClient.getWebDevice(deviceIdBeforeStop);
   expect(response.statusCode).toBe(404);
-});
+}, 60000);
 
 test('Calling .stop before .start should do nothing', async () => {
   const errolClient = new ErrolTestClient(

end-to-end-tests/test-utils.js

@@ -23,10 +23,6 @@ export const NOTIFICATIONS_DEFAULT = 0
 export const NOTIFICATIONS_GRANTED = 1
 export const NOTIFICATIONS_DENIED = 2
 
-beforeAll(() => {
-  jest.setTimeout(SCRIPT_TIMEOUT_MS);
-});
-
 /**
  * Helper for launching a test application server
  * @async

eslint.config.mjs

@@ -0,0 +1,41 @@
+import globals from "globals";
+import path from "node:path";
+import { fileURLToPath } from "node:url";
+import js from "@eslint/js";
+import { FlatCompat } from "@eslint/eslintrc";
+
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = path.dirname(__filename);
+const compat = new FlatCompat({
+    baseDirectory: __dirname,
+    recommendedConfig: js.configs.recommended,
+    allConfig: js.configs.all
+});
+
+export default [...compat.extends("eslint:recommended", "prettier"), {
+    languageOptions: {
+        globals: {
+            ...globals.browser,
+            ...globals.jest,
+            ...globals.node,
+            ...globals.serviceworker,
+        },
+
+        ecmaVersion: 2018,
+        sourceType: "module",
+    },
+
+    rules: {
+        "no-unused-vars": ["error", {
+            argsIgnorePattern: "^_",
+            varsIgnorePattern: "^_",
+            caughtErrorsIgnorePattern: "^_",
+        }],
+
+        "no-console": 0,
+        indent: ["error", 2],
+        quotes: [2, "single"],
+        "linebreak-style": [2, "unix"],
+        semi: [2, "always"],
+    },
+}];

jest.config.js

@@ -1,4 +1,5 @@
 module.exports = {
   automock: false,
   setupFiles: ['./test-setup.js'],
+  testEnvironment: 'jsdom'
 };