Merge pull request #1624 from fippo/jest

test: port to jest

Author: fippo

.babelrc

@@ -8,6 +8,7 @@ module.exports = {
     'browser': true,
     'es6': true,
     'node': true,
+    'jest': true
   },
   'rules': {
     'max-len': 'off',
@@ -42,4 +43,5 @@ module.exports = {
     "VideoEncoder": true,
     "VideoDecoder": true,
   },
+  "plugins": ["jest"]
 };

.eslintrc.js

@@ -18,4 +18,4 @@ jobs:
     - run: BROWSER=${{matrix.browserA}} BVER=${{matrix.bver}} ./node_modules/travis-multirunner/setup.sh
     - run: BROWSER=${{matrix.browserB}} BVER=${{matrix.bver}} ./node_modules/travis-multirunner/setup.sh
     - run: Xvfb :99 &
-    - run: BROWSER_A=${{matrix.browserA}} BROWSER_B=${{matrix.browserB}} BVER=${{matrix.bver}} DISPLAY=:99.0 node_modules/.bin/mocha --retries=3 test/interop/connection.js
+    - run: BROWSER_A=${{matrix.browserA}} BROWSER_B=${{matrix.browserB}} BVER=${{matrix.bver}} DISPLAY=:99.0 node_modules/.bin/jest --retries=3 test/interop/connection.js

.github/workflows/interop-tests.yml

@@ -25,4 +25,4 @@ jobs:
     - run: npm install
     - run: BROWSER=${{matrix.browser}} BVER=${{matrix.version}} ./node_modules/travis-multirunner/setup.sh
     - run: Xvfb :99 &
-    - run: BROWSER=${{matrix.browser}} BVER=${{matrix.version}} DISPLAY=:99.0 npm run mocha -- --retries=3
+    - run: BROWSER=${{matrix.browser}} BVER=${{matrix.version}} DISPLAY=:99.0 npm run jest -- --retries=3

.github/workflows/test.yml

@@ -22,20 +22,20 @@
     "start": "http-server . -c-1",
     "test": "npm run eslint && npm run stylelint",
     "eslint": "eslint 'test/**.js' 'src/content/**/*.js'",
-    "mocha": "mocha --timeout 5000 'src/content/**/test.js'",
+    "jest": "jest --testTimeout 5000 --maxWorkers=1 'src/content/**/test.js'",
     "stylelint": "stylelint 'src/**/*.css'"
   },
   "eslintIgnore": [
     "'**/third_party/*.js'"
   ],
   "devDependencies": {
-    "chai": "^4.3.6",
     "chromedriver": ">98.0.1",
     "eslint": "^8.9.0",
     "eslint-config-google": "^0.14.0",
+    "eslint-plugin-jest": "^27.4.0",
     "geckodriver": "^3.0.1",
     "http-server": "^14.1.0",
-    "mocha": "^9.2.1",
+    "jest": "^29.7.0",
     "selenium-webdriver": "^4.12.0",
     "stylelint": "^14.5.3",
     "stylelint-config-recommended": "^7.0.0",

package.json

@@ -5,22 +5,18 @@
  *  that can be found in the LICENSE file in the root of the source
  *  tree.
  */
-/* eslint-env node, mocha */
-
-'use strict';
 const webdriver = require('selenium-webdriver');
 const seleniumHelpers = require('../../../../../test/webdriver');
-const {expect} = require('chai');
 
 let driver;
 const path = '/src/content/datachannel/basic/index.html';
 const url = `${process.env.BASEURL ? process.env.BASEURL : ('file://' + process.cwd())}${path}`;
 
 describe('datachannel basic', () => {
-  before(() => {
+  beforeAll(() => {
     driver = seleniumHelpers.buildDriver();
   });
-  after(() => {
+  afterAll(() => {
     return driver.quit();
   });
 
@@ -50,6 +46,6 @@ describe('datachannel basic', () => {
     }));
 
     const value = await driver.findElement(webdriver.By.id('dataChannelReceive')).getAttribute('value');
-    expect(value).to.equal(text);
+    expect(value).toBe(text);
   });
 });

src/content/datachannel/basic/js/test.js

@@ -5,22 +5,21 @@
  *  that can be found in the LICENSE file in the root of the source
  *  tree.
  */
-/* eslint-env node, mocha */
+/* eslint-env node */
 
 'use strict';
 const webdriver = require('selenium-webdriver');
 const seleniumHelpers = require('../../../../../test/webdriver');
-const {expect} = require('chai');
 
 let driver;
 const path = '/src/content/datachannel/channel/index.html';
 const url = `${process.env.BASEURL ? process.env.BASEURL : ('file://' + process.cwd())}${path}`;
 
 describe('datachannel and broadcast channels', () => {
-  before(() => {
+  beforeAll(() => {
     driver = seleniumHelpers.buildDriver();
   });
-  after(() => {
+  afterAll(() => {
     return driver.quit();
   });
 
@@ -68,7 +67,7 @@ describe('datachannel and broadcast channels', () => {
       return document.getElementById('dataChannelReceive').value.length > 0;
     }));
     const fromFirst= await driver.findElement(webdriver.By.id('dataChannelReceive')).getAttribute('value');
-    expect(fromFirst).to.equal(firstHello);
+    expect(fromFirst).toBe(firstHello);
 
     // Send a message from the second tab to the first tab.
     await driver.switchTo().window(secondTab);
@@ -84,7 +83,7 @@ describe('datachannel and broadcast channels', () => {
       return document.getElementById('dataChannelReceive').value.length > 0;
     }));
     const fromSecond = await driver.findElement(webdriver.By.id('dataChannelReceive')).getAttribute('value');
-    expect(fromSecond).to.equal(secondHello);
+    expect(fromSecond).toBe(secondHello);
   });
 });
 

src/content/datachannel/channel/js/test.js

@@ -5,22 +5,21 @@
  *  that can be found in the LICENSE file in the root of the source
  *  tree.
  */
-/* eslint-env node, mocha */
+/* eslint-env node */
 
 'use strict';
 const webdriver = require('selenium-webdriver');
 const seleniumHelpers = require('../../../../../test/webdriver');
-const {expect} = require('chai');
 
 let driver;
 const path = '/src/content/datachannel/datatransfer/index.html';
 const url = `${process.env.BASEURL ? process.env.BASEURL : ('file://' + process.cwd())}${path}`;
 
 describe('datachannel datatransfer', () => {
-  before(() => {
+  beforeAll(() => {
     driver = seleniumHelpers.buildDriver();
   });
-  after(() => {
+  afterAll(() => {
     return driver.quit();
   });
 
@@ -52,6 +51,6 @@ describe('datachannel datatransfer', () => {
     }));
 
     const transferred = await driver.findElement(webdriver.By.id('receiveProgress')).getAttribute('value');
-    expect(transferred >>> 0).to.equal(megsToSend * 1024 * 1024);
+    expect(transferred >>> 0).toBe(megsToSend * 1024 * 1024);
   });
 });

src/content/datachannel/datatransfer/js/test.js

@@ -5,7 +5,7 @@
  *  that can be found in the LICENSE file in the root of the source
  *  tree.
  */
-/* eslint-env node, mocha */
+/* eslint-env node */
 
 'use strict';
 const webdriver = require('selenium-webdriver');
@@ -16,10 +16,10 @@ const path = '/src/content/datachannel/filetransfer/index.html';
 const url = `${process.env.BASEURL ? process.env.BASEURL : ('file://' + process.cwd())}${path}`;
 
 describe('datachannel filetransfer', () => {
-  before(() => {
+  beforeAll(() => {
     driver = seleniumHelpers.buildDriver();
   });
-  after(() => {
+  afterAll(() => {
     return driver.quit();
   });
 

src/content/datachannel/filetransfer/js/test.js

@@ -5,7 +5,7 @@
  *  that can be found in the LICENSE file in the root of the source
  *  tree.
  */
-/* eslint-env node, mocha */
+/* eslint-env node */
 'use strict';
 
 const seleniumHelpers = require('../../../../../test/webdriver');
@@ -15,10 +15,10 @@ const path = '/src/content/devices/input-output/index.html';
 const url = `${process.env.BASEURL ? process.env.BASEURL : ('file://' + process.cwd())}${path}`;
 
 describe('input-output', () => {
-  before(() => {
+  beforeAll(() => {
     driver = seleniumHelpers.buildDriver();
   });
-  after(() => {
+  afterAll(() => {
     return driver.quit();
   });