Merge pull request #1654 from fippo/bye-travis

fippo · web-flow · commit 0eecfeb3ac9d · 2024-04-22T06:38:35.000-07:00
test: replace travis-multirunner with @puppeter/browsers
diff --git a/.github/workflows/interop-tests.yml b/.github/workflows/interop-tests.yml
@@ -13,11 +13,9 @@ jobs:
         browserB: [firefox, chrome]
         bver: ['unstable']
     steps:
-    - uses: actions/checkout@v3
-    - uses: actions/setup-node@v3
+    - uses: actions/checkout@v4
+    - uses: actions/setup-node@v4
     - run: npm install
-    - 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: sudo rm /usr/bin/chromedriver /usr/bin/geckodriver # remove preinstalled github chromedriver/geckodriver from $PATH
     - run: Xvfb :99 &
     - run: BROWSER_A=${{matrix.browserA}} BROWSER_B=${{matrix.browserB}} BVER=${{matrix.bver}} DISPLAY=:99.0 node_modules/.bin/jest --retries=3 test/interop/
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -6,8 +6,8 @@ jobs:
   lint:
     runs-on: ubuntu-latest
     steps:
-    - uses: actions/checkout@v3
-    - uses: actions/setup-node@v3
+    - uses: actions/checkout@v4
+    - uses: actions/setup-node@v4
     - run: npm install
     - run: npm run eslint
     - run: npm run stylelint
@@ -20,10 +20,9 @@ jobs:
         browser: [chrome]
         version: [stable]
     steps:
-    - uses: actions/checkout@v2
-    - uses: actions/setup-node@v3
+    - uses: actions/checkout@v4
+    - uses: actions/setup-node@v4
     - run: npm install
-    - run: BROWSER=${{matrix.browser}} BVER=${{matrix.version}} ./node_modules/travis-multirunner/setup.sh
     - run: sudo rm /usr/bin/chromedriver # remove preinstalled github chromedriver from $PATH
     - run: Xvfb :99 &
     - run: BROWSER=${{matrix.browser}} BVER=${{matrix.version}} DISPLAY=:99.0 npm run jest -- --retries=3
diff --git a/package.json b/package.json
@@ -1,14 +1,12 @@
 {
   "name": "webrtc-samples",
+  "private": true,
   "version": "1.0.0",
   "description": "Project checking for WebRTC GitHub samples repo",
   "keywords": [
-    "webrtc",
-    "demos",
-    "samples",
-    "javascript"
+    "webrtc"
   ],
-  "homepage": "https://github.com/webrtc/samples",
+  "homepage": "https://webrtc.github.io/samples/",
   "bugs": {
     "url": "https://github.com/webrtc/samples/issues"
   },
@@ -22,21 +20,21 @@
     "start": "http-server . -c-1",
     "test": "npm run eslint && npm run stylelint",
     "eslint": "eslint 'test/**.js' 'src/content/**/*.js'",
-    "jest": "jest --testTimeout 5000 --maxWorkers=1 'src/content/**/test.js'",
+    "jest": "node test/download-browsers.js && jest --testTimeout 5000 --maxWorkers=1 test/download.test.js src/content/",
     "stylelint": "stylelint 'src/**/*.css'"
   },
   "eslintIgnore": [
     "'**/third_party/*.js'"
   ],
   "devDependencies": {
+    "@puppeteer/browsers": "^2.2.0",
     "eslint": "^8.9.0",
     "eslint-config-google": "^0.14.0",
     "eslint-plugin-jest": "^27.4.0",
     "http-server": "^14.1.0",
     "jest": "^29.7.0",
-    "selenium-webdriver": "^4.12.0",
+    "selenium-webdriver": "^4.19.0",
     "stylelint": "^14.5.3",
-    "stylelint-config-recommended": "^7.0.0",
-    "travis-multirunner": "^5.0.1"
+    "stylelint-config-recommended": "^7.0.0"
   }
 }
diff --git a/src/content/datachannel/basic/js/test.js b/src/content/datachannel/basic/js/test.js
@@ -13,8 +13,8 @@ const path = '/src/content/datachannel/basic/index.html';
 const url = `${process.env.BASEURL ? process.env.BASEURL : ('file://' + process.cwd())}${path}`;
 
 describe('datachannel basic', () => {
-  beforeAll(() => {
-    driver = seleniumHelpers.buildDriver();
+  beforeAll(async () => {
+    driver = await seleniumHelpers.buildDriver();
   });
   afterAll(() => {
     return driver.quit();
diff --git a/src/content/datachannel/channel/js/test.js b/src/content/datachannel/channel/js/test.js
@@ -16,8 +16,8 @@ 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', () => {
-  beforeAll(() => {
-    driver = seleniumHelpers.buildDriver();
+  beforeAll(async () => {
+    driver = await seleniumHelpers.buildDriver();
   });
   afterAll(() => {
     return driver.quit();
diff --git a/src/content/datachannel/datatransfer/js/test.js b/src/content/datachannel/datatransfer/js/test.js
@@ -16,8 +16,8 @@ const path = '/src/content/datachannel/datatransfer/index.html';
 const url = `${process.env.BASEURL ? process.env.BASEURL : ('file://' + process.cwd())}${path}`;
 
 describe('datachannel datatransfer', () => {
-  beforeAll(() => {
-    driver = seleniumHelpers.buildDriver();
+  beforeAll(async () => {
+    driver = await seleniumHelpers.buildDriver();
   });
   afterAll(() => {
     return driver.quit();
diff --git a/src/content/datachannel/filetransfer/js/test.js b/src/content/datachannel/filetransfer/js/test.js
@@ -16,8 +16,8 @@ const path = '/src/content/datachannel/filetransfer/index.html';
 const url = `${process.env.BASEURL ? process.env.BASEURL : ('file://' + process.cwd())}${path}`;
 
 describe('datachannel filetransfer', () => {
-  beforeAll(() => {
-    driver = seleniumHelpers.buildDriver();
+  beforeAll(async () => {
+    driver = await seleniumHelpers.buildDriver();
   });
   afterAll(() => {
     return driver.quit();
diff --git a/src/content/devices/input-output/js/test.js b/src/content/devices/input-output/js/test.js
@@ -15,8 +15,8 @@ const path = '/src/content/devices/input-output/index.html';
 const url = `${process.env.BASEURL ? process.env.BASEURL : ('file://' + process.cwd())}${path}`;
 
 describe('input-output', () => {
-  beforeAll(() => {
-    driver = seleniumHelpers.buildDriver();
+  beforeAll(async () => {
+    driver = await seleniumHelpers.buildDriver();
   });
   afterAll(() => {
     return driver.quit();
diff --git a/src/content/getusermedia/gum/js/test.js b/src/content/getusermedia/gum/js/test.js
@@ -16,8 +16,8 @@ const path = '/src/content/getusermedia/gum/index.html';
 const url = `${process.env.BASEURL ? process.env.BASEURL : ('file://' + process.cwd())}${path}`;
 
 describe('getUserMedia', () => {
-  beforeAll(() => {
-    driver = seleniumHelpers.buildDriver();
+  beforeAll(async () => {
+    driver = await seleniumHelpers.buildDriver();
   });
   afterAll(() => {
     return driver.quit();
diff --git a/src/content/getusermedia/resolution/js/test.js b/src/content/getusermedia/resolution/js/test.js
@@ -17,8 +17,8 @@ const path = '/src/content/getusermedia/resolution/index.html';
 const url = `${process.env.BASEURL ? process.env.BASEURL : ('file://' + process.cwd())}${path}`;
 
 describe('getUserMedia resolutions', () => {
-  beforeAll(() => {
-    driver = seleniumHelpers.buildDriver();
+  beforeAll(async () => {
+    driver = await seleniumHelpers.buildDriver();
   });
   afterAll(() => {
     return driver.quit();
diff --git a/src/content/insertable-streams/endtoend-encryption/js/test.js b/src/content/insertable-streams/endtoend-encryption/js/test.js
@@ -16,8 +16,8 @@ const path = '/src/content/insertable-streams/endtoend-encryption/index.html';
 const url = `${process.env.BASEURL ? process.env.BASEURL : ('file://' + process.cwd())}${path}`;
 
 describe('insertable streams e2ee', () => {
-  beforeAll(() => {
-    driver = seleniumHelpers.buildDriver();
+  beforeAll(async () => {
+    driver = await seleniumHelpers.buildDriver();
   });
   afterAll(() => {
     return driver.quit();
diff --git a/src/content/peerconnection/audio/js/test.js b/src/content/peerconnection/audio/js/test.js
@@ -15,8 +15,8 @@ const path = '/src/content/peerconnection/audio/index.html';
 const url = `${process.env.BASEURL ? process.env.BASEURL : ('file://' + process.cwd())}${path}`;
 
 describe('audio-only peerconnection', () => {
-  beforeAll(() => {
-    driver = seleniumHelpers.buildDriver();
+  beforeAll(async () => {
+    driver = await seleniumHelpers.buildDriver();
   });
   afterAll(() => {
     return driver.quit();
diff --git a/src/content/peerconnection/change-codecs/js/test.js b/src/content/peerconnection/change-codecs/js/test.js
@@ -16,8 +16,8 @@ const path = '/src/content/peerconnection/change-codecs/index.html';
 const url = `${process.env.BASEURL ? process.env.BASEURL : ('file://' + process.cwd())}${path}`;
 
 describe('peerconnection with setCodecPreferences', () => {
-  beforeAll(() => {
-    driver = seleniumHelpers.buildDriver();
+  beforeAll(async () => {
+    driver = await seleniumHelpers.buildDriver();
   });
   afterAll(() => {
     return driver.quit();
diff --git a/src/content/peerconnection/channel/js/test.js b/src/content/peerconnection/channel/js/test.js
@@ -16,8 +16,8 @@ const path = '/src/content/peerconnection/channel/index.html';
 const url = `${process.env.BASEURL ? process.env.BASEURL : ('file://' + process.cwd())}${path}`;
 
 describe('peerconnection and broadcast channels', () => {
-  beforeAll(() => {
-    driver = seleniumHelpers.buildDriver();
+  beforeAll(async () => {
+    driver = await seleniumHelpers.buildDriver();
   });
   afterAll(() => {
     return driver.quit();
diff --git a/src/content/peerconnection/dtmf/js/test.js b/src/content/peerconnection/dtmf/js/test.js
@@ -17,8 +17,8 @@ const path = '/src/content/peerconnection/dtmf/index.html';
 const url = `${process.env.BASEURL ? process.env.BASEURL : ('file://' + process.cwd())}${path}`;
 
 describe('peerconnection dtmf', () => {
-  beforeAll(() => {
-    driver = seleniumHelpers.buildDriver();
+  beforeAll(async () => {
+    driver = await seleniumHelpers.buildDriver();
   });
   afterAll(() => {
     return driver.quit();
diff --git a/src/content/peerconnection/multiple/js/test.js b/src/content/peerconnection/multiple/js/test.js
@@ -16,8 +16,8 @@ const path = '/src/content/peerconnection/multiple/index.html';
 const url = `${process.env.BASEURL ? process.env.BASEURL : ('file://' + process.cwd())}${path}`;
 
 describe('multiple peerconnections', () => {
-  beforeAll(() => {
-    driver = seleniumHelpers.buildDriver();
+  beforeAll(async () => {
+    driver = await seleniumHelpers.buildDriver();
   });
   afterAll(() => {
     return driver.quit();
diff --git a/src/content/peerconnection/munge-sdp/js/test.js b/src/content/peerconnection/munge-sdp/js/test.js
@@ -16,8 +16,8 @@ const path = '/src/content/peerconnection/munge-sdp/index.html';
 const url = `${process.env.BASEURL ? process.env.BASEURL : ('file://' + process.cwd())}${path}`;
 
 describe('peerconnection sdp munging', () => {
-  beforeAll(() => {
-    driver = seleniumHelpers.buildDriver();
+  beforeAll(async () => {
+    driver = await seleniumHelpers.buildDriver();
   });
   afterAll(() => {
     return driver.quit();
diff --git a/src/content/peerconnection/negotiate-timing/js/test.js b/src/content/peerconnection/negotiate-timing/js/test.js
@@ -16,8 +16,8 @@ const path = '/src/content/peerconnection/negotiate-timing/index.html';
 const url = `${process.env.BASEURL ? process.env.BASEURL : ('file://' + process.cwd())}${path}`;
 
 describe('peerconnection with negotiation timing', () => {
-  beforeAll(() => {
-    driver = seleniumHelpers.buildDriver();
+  beforeAll(async () => {
+    driver = await seleniumHelpers.buildDriver();
   });
   afterAll(() => {
     return driver.quit();
diff --git a/src/content/peerconnection/pc1/js/test.js b/src/content/peerconnection/pc1/js/test.js
@@ -16,8 +16,8 @@ const path = '/src/content/peerconnection/pc1/index.html';
 const url = `${process.env.BASEURL ? process.env.BASEURL : ('file://' + process.cwd())}${path}`;
 
 describe('simple peerconnection', () => {
-  beforeAll(() => {
-    driver = seleniumHelpers.buildDriver();
+  beforeAll(async () => {
+    driver = await seleniumHelpers.buildDriver();
   });
   afterAll(() => {
     return driver.quit();
diff --git a/src/content/peerconnection/restart-ice/js/test.js b/src/content/peerconnection/restart-ice/js/test.js
@@ -16,8 +16,8 @@ const path = '/src/content/peerconnection/restart-ice/index.html';
 const url = `${process.env.BASEURL ? process.env.BASEURL : ('file://' + process.cwd())}${path}`;
 
 describe('peerconnection ice restart', () => {
-  beforeAll(() => {
-    driver = seleniumHelpers.buildDriver();
+  beforeAll(async () => {
+    driver = await seleniumHelpers.buildDriver();
   });
   afterAll(() => {
     return driver.quit();
diff --git a/src/content/peerconnection/states/js/test.js b/src/content/peerconnection/states/js/test.js
@@ -16,8 +16,8 @@ const path = '/src/content/peerconnection/states/index.html';
 const url = `${process.env.BASEURL ? process.env.BASEURL : ('file://' + process.cwd())}${path}`;
 
 describe('peerconnection states', () => {
-  beforeAll(() => {
-    driver = seleniumHelpers.buildDriver();
+  beforeAll(async () => {
+    driver = await seleniumHelpers.buildDriver();
   });
   afterAll(() => {
     return driver.quit();
diff --git a/src/content/peerconnection/trickle-ice/js/test.js b/src/content/peerconnection/trickle-ice/js/test.js
@@ -16,8 +16,8 @@ const path = '/src/content/peerconnection/trickle-ice/index.html';
 const url = `${process.env.BASEURL ? process.env.BASEURL : ('file://' + process.cwd())}${path}`;
 
 describe('Trickle-Ice', () => {
-  beforeAll(() => {
-    driver = seleniumHelpers.buildDriver();
+  beforeAll(async () => {
+    driver = await seleniumHelpers.buildDriver();
   });
   afterAll(() => {
     return driver.quit();
diff --git a/src/content/peerconnection/upgrade/js/test.js b/src/content/peerconnection/upgrade/js/test.js
@@ -16,8 +16,8 @@ const path = '/src/content/peerconnection/upgrade/index.html';
 const url = `${process.env.BASEURL ? process.env.BASEURL : ('file://' + process.cwd())}${path}`;
 
 describe('peerconnection upgrade from audio-only to audio-video', () => {
-  beforeAll(() => {
-    driver = seleniumHelpers.buildDriver();
+  beforeAll(async () => {
+    driver = await seleniumHelpers.buildDriver();
   });
   afterAll(() => {
     return driver.quit();
diff --git a/test/download-browsers.js b/test/download-browsers.js
@@ -0,0 +1,11 @@
+const {buildDriver} = require('./webdriver');
+// Download the browser(s).
+async function download() {
+  if (process.env.browserA && process.env.browserB) {
+    (await buildDriver(process.env.browserA)).quit();
+    (await buildDriver(process.env.browserB)).quit();
+  } else {
+    (await buildDriver()).quit();
+  }
+}
+download();
diff --git a/test/interop/connection.test.js b/test/interop/connection.test.js
@@ -21,8 +21,8 @@ describe(`basic interop test ${browserA} => ${browserB}`, function() {
             browserLogging: true,
         }
         drivers = [
-            buildDriver(browserA, options),
-            buildDriver(browserB, options),
+            await buildDriver(browserA, options),
+            await buildDriver(browserB, options),
         ];
         clients = drivers.map(driver => {
             return {
diff --git a/test/webdriver.js b/test/webdriver.js
