Merge pull request #1147 from webrtcHacks/fix-chai

update test setup and bring back chai as a dependency

.github/workflows/e2e-tests.yaml

@@ -13,8 +13,8 @@ jobs:
           - browser: firefox
             version: nightly
     steps:
-    - uses: actions/checkout@v3
-    - uses: actions/setup-node@v3
+    - uses: actions/checkout@v4
+    - uses: actions/setup-node@v4
     - run: npm install
     - run: Xvfb :99 &
     - name: e2e-tests

.github/workflows/lint-and-unit-tests.yml

@@ -6,7 +6,7 @@ jobs:
   test:
     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 lint-and-unit-tests

package.json

@@ -37,6 +37,7 @@
     "@puppeteer/browsers": "^1.4.1",
     "babel-preset-env": "^1.7.0",
     "brfs": "^1.5.0",
+    "chai": "^3.5.0",
     "eslint-plugin-jest": "^27.4.0",
     "grunt": "^1.1.0",
     "grunt-babel": "^8.0.0",

test/e2e/maxMessageSize.js

@@ -42,12 +42,13 @@ describe('maxMessageSize', () => {
 
   function patchMaxMessageSizeFactory(maxMessageSize) {
     return ((description) => {
-      description.sdp = description.sdp.replace(
-        /^a=max-message-size:\s*(\d+)\s*$/gm, '');
-      description.sdp = description.sdp.replace(
-        /(^m=application\s+\d+\s+[\w/]*SCTP.*$)/m,
-        '$1\r\na=max-message-size:' + maxMessageSize);
-      return description;
+      return {
+        type: description.type,
+        sdp: description.sdp
+          .replace(/^a=max-message-size:\s*(\d+)\s*$/gm, '')
+          .replace(/(^m=application\s+\d+\s+[\w/]*SCTP.*$)/m,
+            '$1\r\na=max-message-size:' + maxMessageSize),
+      };
     });
   }