Switch from Karma to Web Test Runner

.github/workflows/test.yml

@@ -10,13 +10,13 @@ jobs:
           - ubuntu-latest
           - windows-latest
         browser:
-          - ChromeHeadless
-          - FirefoxHeadless
+          - chrome
+          - firefox
         include:
           - os: macos-latest
-            browser: Safari
+            browser: safari
           - os: windows-latest
-            browser: EdgeHeadless
+            browser: edge
       fail-fast: false
     runs-on: ${{ matrix.os }}
     steps:

eslint.config.mjs

@@ -78,8 +78,6 @@ export default [
             globals: {
                 ...globals.node,
                 ...globals.mocha,
-                sinon: false,
-                expect: false,
             }
         },
         rules: {

package.json

@@ -18,7 +18,7 @@
   ],
   "scripts": {
     "lint": "eslint app core po/po2js po/xgettext-html tests utils",
-    "test": "karma start karma.conf.js",
+    "test": "web-test-runner",
     "prepublish": "node ./utils/convert.js --clean"
   },
   "repository": {
@@ -46,22 +46,14 @@
     "fs-extra": "latest",
     "globals": "latest",
     "jsdom": "latest",
-    "karma": "latest",
-    "karma-mocha": "latest",
-    "karma-chrome-launcher": "latest",
-    "@chiragrupani/karma-chromium-edge-launcher": "latest",
-    "karma-firefox-launcher": "latest",
-    "karma-ie-launcher": "latest",
-    "karma-mocha-reporter": "latest",
-    "karma-safari-launcher": "latest",
-    "karma-script-launcher": "latest",
     "mocha": "latest",
     "node-getopt": "latest",
     "po2json": "latest",
     "sinon": "latest",
-    "sinon-chai": "latest"
+    "sinon-chai": "latest",
+    "@web/test-runner": "latest",
+    "@web/test-runner-webdriver": "latest"
   },
-  "dependencies": {},
   "keywords": [
     "vnc",
     "rfb",

tests/assertions.js

@@ -1,10 +1,6 @@
-import * as chai from '../node_modules/chai/chai.js';
-import sinon from '../node_modules/sinon/pkg/sinon-esm.js';
-import sinonChai from '../node_modules/sinon-chai/lib/sinon-chai.js';
+import * as chai from 'chai';
+import sinonChai from 'sinon-chai';
 
-window.expect = chai.expect;
-
-window.sinon = sinon;
 chai.use(sinonChai);
 
 // noVNC specific assertions

tests/test.base64.js

@@ -1,3 +1,5 @@
+import { expect } from 'chai';
+
 import Base64 from '../core/base64.js';
 
 describe('Base64 Tools', function () {

tests/test.browser.js

@@ -1,3 +1,5 @@
+import { expect } from 'chai';
+
 import { isMac, isWindows, isIOS, isAndroid, isChromeOS,
          isSafari, isFirefox, isChrome, isChromium, isOpera, isEdge,
          isGecko, isWebKit, isBlink } from '../core/util/browser.js';

tests/test.copyrect.js

@@ -1,3 +1,6 @@
+import './assertions.js';
+import { expect } from 'chai';
+
 import Websock from '../core/websock.js';
 import Display from '../core/display.js';
 

tests/test.deflator.js

@@ -1,3 +1,6 @@
+import './assertions.js';
+import { expect } from 'chai';
+
 import { inflateInit, inflate } from "../vendor/pako/lib/zlib/inflate.js";
 import ZStream from "../vendor/pako/lib/zlib/zstream.js";
 import Deflator from "../core/deflator.js";

tests/test.display.js

@@ -1,3 +1,7 @@
+import './assertions.js';
+import { expect } from 'chai';
+import sinon from 'sinon';
+
 import Base64 from '../core/base64.js';
 import Display from '../core/display.js';
 

tests/test.gesturehandler.js

@@ -1,3 +1,7 @@
+import './assertions.js';
+import { expect } from 'chai';
+import sinon from 'sinon';
+
 import EventTargetMixin from '../core/util/eventtarget.js';
 
 import GestureHandler from '../core/input/gesturehandler.js';

tests/test.helper.js

@@ -1,3 +1,5 @@
+import { expect } from 'chai';
+
 import keysyms from '../core/input/keysymdef.js';
 import * as KeyboardUtil from "../core/input/util.js";
 

tests/test.hextile.js

@@ -1,3 +1,6 @@
+import './assertions.js';
+import { expect } from 'chai';
+
 import Websock from '../core/websock.js';
 import Display from '../core/display.js';
 

tests/test.inflator.js

@@ -1,3 +1,6 @@
+import './assertions.js';
+import { expect } from 'chai';
+
 import { deflateInit, deflate, Z_FULL_FLUSH } from "../vendor/pako/lib/zlib/deflate.js";
 import ZStream from "../vendor/pako/lib/zlib/zstream.js";
 import Inflator from "../core/inflator.js";

tests/test.int.js

@@ -1,3 +1,5 @@
+import { expect } from 'chai';
+
 import { toUnsigned32bit, toSigned32bit } from '../core/util/int.js';
 
 describe('Integer casting', function () {

tests/test.jpeg.js

@@ -1,3 +1,6 @@
+import './assertions.js';
+import { expect } from 'chai';
+
 import Websock from '../core/websock.js';
 import Display from '../core/display.js';
 

tests/test.keyboard.js

@@ -1,3 +1,7 @@
+import './assertions.js';
+import { expect } from 'chai';
+import sinon from 'sinon';
+
 import Keyboard from '../core/input/keyboard.js';
 
 describe('Key Event Handling', function () {
@@ -591,7 +595,7 @@ describe('Key Event Handling', function () {
             expect(kbd.onkeyevent).to.have.been.calledWith(0xffe1, 'ShiftLeft', true);
             kbd.onkeyevent.resetHistory();
 
-            kbd._handleKeyDown(keyevent('keydown', {code: 'ShiftRight', key: 'Shift', location: 2}));
+            kbd._handleKeyDown(keyevent('keydown', {code: 'ShiftRight', key: 'ShiftFoo', location: 2}));
             expect(kbd.onkeyevent).to.have.been.calledOnce;
             expect(kbd.onkeyevent).to.have.been.calledWith(0xffe2, 'ShiftRight', true);
             kbd.onkeyevent.resetHistory();

tests/test.localization.js

@@ -1,3 +1,7 @@
+import './assertions.js';
+import { expect } from 'chai';
+import sinon from 'sinon';
+
 import _, { Localizer, l10n } from '../app/localization.js';
 
 describe('Localization', function () {

tests/test.raw.js

@@ -1,3 +1,6 @@
+import './assertions.js';
+import { expect } from 'chai';
+
 import Websock from '../core/websock.js';
 import Display from '../core/display.js';
 

tests/test.rfb.js

@@ -1,8 +1,13 @@
+import './assertions.js';
+import { expect } from 'chai';
+import sinon from 'sinon';
+
 import RFB from '../core/rfb.js';
 import Websock from '../core/websock.js';
 import ZStream from "../vendor/pako/lib/zlib/zstream.js";
 import { deflateInit, deflate, Z_DEFAULT_COMPRESSION } from "../vendor/pako/lib/zlib/deflate.js";
 import { encodings } from '../core/encodings.js';
+import { initLogging } from '../core/util/logging.js';
 import { toUnsigned32bit } from '../core/util/int.js';
 import { encodeUTF8 } from '../core/util/strings.js';
 import KeyTable from '../core/input/keysym.js';
@@ -111,6 +116,9 @@ describe('Remote Frame Buffer Protocol Client', function () {
 
         // Avoiding printing the entire Websock buffer on errors
         Websock.prototype.inspect = function () { return "[object Websock]"; };
+
+        // Avoid spamming test output
+        initLogging('none');
     });
 
     after(function () {

tests/test.rre.js

@@ -1,3 +1,6 @@
+import './assertions.js';
+import { expect } from 'chai';
+
 import Websock from '../core/websock.js';
 import Display from '../core/display.js';
 

tests/test.tight.js

@@ -1,3 +1,6 @@
+import './assertions.js';
+import { expect } from 'chai';
+
 import Websock from '../core/websock.js';
 import Display from '../core/display.js';
 

tests/test.tightpng.js

@@ -1,3 +1,6 @@
+import './assertions.js';
+import { expect } from 'chai';
+
 import Websock from '../core/websock.js';
 import Display from '../core/display.js';
 

tests/test.util.js

@@ -1,4 +1,8 @@
 /* eslint-disable no-console */
+import './assertions.js';
+import { expect } from 'chai';
+import sinon from 'sinon';
+
 import * as Log from '../core/util/logging.js';
 import { encodeUTF8, decodeUTF8 } from '../core/util/strings.js';
 
@@ -7,11 +11,11 @@ describe('Utils', function () {
 
     describe('logging functions', function () {
         beforeEach(function () {
-            sinon.spy(console, 'log');
-            sinon.spy(console, 'debug');
-            sinon.spy(console, 'warn');
-            sinon.spy(console, 'error');
-            sinon.spy(console, 'info');
+            sinon.stub(console, 'log');
+            sinon.stub(console, 'debug');
+            sinon.stub(console, 'warn');
+            sinon.stub(console, 'error');
+            sinon.stub(console, 'info');
         });
 
         afterEach(function () {

tests/test.websock.js

@@ -1,3 +1,7 @@
+import './assertions.js';
+import { expect } from 'chai';
+import sinon from 'sinon';
+
 import Websock from '../core/websock.js';
 import FakeWebSocket from './fake.websocket.js';
 

tests/test.webutil.js

@@ -1,6 +1,11 @@
 /* jshint expr: true */
 
+import './assertions.js';
+import { expect } from 'chai';
+import sinon from 'sinon';
+
 import * as WebUtil from '../app/webutil.js';
+import { initLogging } from '../core/util/logging.js';
 
 describe('WebUtil', function () {
     "use strict";
@@ -63,6 +68,9 @@ describe('WebUtil', function () {
             before(function () {
                 chrome = window.chrome;
                 window.chrome = null;
+
+                // Avoid spamming test output
+                initLogging('none');
             });
             after(function () {
                 window.chrome = chrome;

tests/test.zrle.js

@@ -1,3 +1,6 @@
+import './assertions.js';
+import { expect } from 'chai';
+
 import Websock from '../core/websock.js';
 import Display from '../core/display.js';