Merge pull request #2228 from deathandmayhem/dependabot/npm_and_yarn/…

…multi-a92cf2f9ff

Bump chai and @types/chai

package.json

@@ -78,7 +78,7 @@
   },
   "devDependencies": {
     "@testing-library/react": "^14.3.1",
-    "@types/chai": "^4.3.11",
+    "@types/chai": "^4.3.17",
     "@types/chai-as-promised": "^7.1.8",
     "@types/dompurify": "^3.0.5",
     "@types/element-resize-detector": "^1.1.6",
@@ -107,7 +107,7 @@
     "@typescript-eslint/utils": "^5.62.0",
     "babel-plugin-add-react-displayname": "0.0.5",
     "babel-plugin-styled-components": "^2.1.4",
-    "chai": "^4.4.1",
+    "chai": "^4.5.0",
     "chai-as-promised": "^8.0.0",
     "eslint": "^8.57.0",
     "eslint-config-airbnb": "^19.0.4",

tests/acceptance/profiles.tsx

@@ -248,7 +248,7 @@ if (Meteor.isClient) {
         let u2 = await MeteorUsers.findOneAsync(sameHuntUserId);
         assert.isDefined(u2, "Should show users in the same hunt");
         assert.sameMembers(
-          u2!.hunts!,
+          u2.hunts!,
           [huntId],
           "Should not show membership in other hunts even if user is visible",
         );

tests/unit/imports/server/MigrationRegistry.ts

@@ -108,10 +108,10 @@ describe("MigrationRegistry", function () {
     // Expect that the database is still locked.
     const control = await testCollection.findOneAsync("control");
     assert.isDefined(control);
-    const { locked, lockedAt } = control!;
+    const { locked, lockedAt } = control;
     assert.equal(locked, true);
     assert.isDefined(lockedAt);
-    assert.isAtLeast(lockedAt!.getTime(), start);
+    assert.isAtLeast(lockedAt.getTime(), start);
 
     // Expect that the first migration ran, once.
     assert.equal(runCount, 1);
@@ -164,10 +164,10 @@ describe("MigrationRegistry", function () {
     // Expect that the database is still locked.
     const control = await testCollection.findOneAsync("control");
     assert.isDefined(control);
-    const { locked, lockedAt } = control!;
+    const { locked, lockedAt } = control;
     assert.equal(locked, true);
     assert.isDefined(lockedAt);
-    assert.isAtLeast(lockedAt!.getTime(), start);
+    assert.isAtLeast(lockedAt.getTime(), start);
 
     // Expect that the first migration ran, once.
     assert.equal(runCount, 1);

tests/unit/imports/server/Model.ts

@@ -60,7 +60,7 @@ describe("Model", function () {
 
       const record = await model.findOneAsync(result);
       assert.isOk(record);
-      assert.isUndefined(record!.string);
+      assert.isUndefined(record.string);
     });
 
     it("works on update", async function () {
@@ -88,7 +88,7 @@ describe("Model", function () {
 
       const record = await model.findOneAsync(id);
       assert.isOk(record);
-      assert.isUndefined(record!.string);
+      assert.isUndefined(record.string);
     });
   });