chore: say goodbye to chai.js

Author: aradzie

package-lock.json

@@ -78,11 +78,9 @@
     "@testing-library/dom": "^10.4.0",
     "@testing-library/react": "^16.0.1",
     "@testing-library/user-event": "^14.5.2",
-    "@types/chai": "^5.0.1",
     "@types/node": "^22.9.1",
     "@types/react": "^18.3.12",
     "@types/react-dom": "^18.3.1",
-    "chai": "^5.1.1",
     "cheerio": "^1.0.0",
     "confusing-browser-globals": "^1.0.11",
     "rich-assert": "^0.0.3",

package.json

@@ -1,13 +1,12 @@
 import { test } from "node:test";
 import { Language } from "@keybr/keyboard";
-import { assert } from "chai";
-import { fail } from "rich-assert";
+import { fail, isTrue } from "rich-assert";
 import { loadWordList } from "./load.ts";
 
 for (const language of Language.ALL) {
   test(`words:${language}`, async () => {
     const words = await loadWordList(language);
-    assert.isAbove(words.length, 1500);
+    isTrue(words.length > 1500);
     const unique = new Set();
     for (const word of words) {
       if (!language.test(word)) {

packages/keybr-content-words/lib/words.test.ts

@@ -1,8 +1,15 @@
 import { test } from "node:test";
 import { PublicId } from "@keybr/publicid";
-import { expect } from "chai";
 import { ValidationError } from "objection";
-import { deepEqual, equal, isNotNull, isNull, like } from "rich-assert";
+import {
+  deepEqual,
+  doesNotThrow,
+  equal,
+  isNotNull,
+  isNull,
+  like,
+  throws,
+} from "rich-assert";
 import { User, UserExternalId, UserLoginRequest } from "./model.ts";
 import { useDatabase } from "./testing.ts";
 import { Random } from "./util.ts";
@@ -14,87 +21,75 @@ const now = new Date("2001-02-03T04:05:06Z");
 test("validate models", (ctx) => {
   ctx.mock.timers.enable({ apis: ["Date"], now });
 
-  expect(() => {
+  throws(() => {
     User.fromJson({});
-  })
-    .to.throw(ValidationError)
-    .to.have.property("data");
+  }, ValidationError);
 
-  expect(() => {
+  throws(() => {
     User.fromJson({
       name: null,
       email: null,
     });
-  })
-    .to.throw(ValidationError)
-    .to.have.property("data");
+  }, ValidationError);
 
-  expect(() => {
+  throws(() => {
     User.fromJson({
       name: "",
       email: "",
     });
-  })
-    .to.throw(ValidationError)
-    .to.have.property("data");
+  }, ValidationError);
 
-  expect(() => {
+  doesNotThrow(() => {
     User.fromJson({
       name: "name",
       email: "email",
     });
-  }).to.not.throw();
+  });
 
-  expect(() => {
+  throws(() => {
     UserExternalId.fromJson({});
-  })
-    .to.throw(ValidationError)
-    .to.have.property("data");
+  }, ValidationError);
 
-  expect(() => {
+  throws(() => {
     UserExternalId.fromJson({
       provider: null,
       externalId: null,
     });
-  })
-    .to.throw(ValidationError)
-    .to.have.property("data");
+  }, ValidationError);
 
-  expect(() => {
+  throws(() => {
     UserExternalId.fromJson({
       provider: "",
       externalId: "",
     });
-  })
-    .to.throw(ValidationError)
-    .to.have.property("data");
+  }, ValidationError);
 
-  expect(() => {
+  doesNotThrow(() => {
     UserExternalId.fromJson({
       provider: "provider",
       externalId: "externalId",
     });
-  }).to.not.throw();
+  });
 
-  expect(() => {
+  doesNotThrow(() => {
     UserExternalId.fromJson({
       provider: "provider",
       externalId: "externalId",
       name: null,
       url: null,
       imageUrl: null,
     });
-  }).to.not.throw();
+  });
 
-  expect(() => {
+  doesNotThrow(() => {
     UserExternalId.fromJson({
       provider: "provider",
       externalId: "externalId",
       name: "name",
       url: "url",
       imageUrl: "imageUrl",
     });
-  }).to.not.throw();
+  });
 });
 
 test("automatically populate createdAt", async (ctx) => {

packages/keybr-database/lib/model.test.ts

@@ -1,7 +1,6 @@
 import { test } from "node:test";
-import { assert } from "chai";
 import { createIntl } from "react-intl";
-import { equal } from "rich-assert";
+import { equal, isNotEmpty } from "rich-assert";
 import { makeIntlDisplayNames } from "./displaynames.ts";
 import { allLocales } from "./locale.ts";
 
@@ -71,7 +70,7 @@ test("format all language names in all locales", () => {
       createIntl({ locale: outer }),
     );
     for (const inner of allLocales) {
-      assert.isAbove(formatLanguageName(inner).length, 0);
+      isNotEmpty(formatLanguageName(inner));
     }
   }
 });

packages/keybr-intl/lib/displaynames.test.ts

@@ -1,7 +1,6 @@
 import { readFileSync } from "node:fs";
 import { test } from "node:test";
-import { assert } from "chai";
-import { deepEqual } from "rich-assert";
+import { deepEqual, doesNotMatch } from "rich-assert";
 import { allLocales, defaultLocale } from "./locale.ts";
 
 for (const locale of allLocales) {
@@ -27,33 +26,33 @@ for (const locale of allLocales) {
   test(`typography [${locale}]`, () => {
     const messages = loadTranslations(locale);
     for (const [id, message] of Object.entries(messages)) {
-      assert.notMatch(message, /^\s/, `Message ${id} starts with whitespace`);
-      assert.notMatch(message, /\s$/, `Message ${id} ends with whitespace`);
-      assert.notMatch(message, /\s\s/, `Message ${id} has repeated whitespace`);
-      assert.notMatch(message, /\t/, `Message ${id} has tab whitespace`);
-      assert.notMatch(
+      doesNotMatch(message, /^\s/, `Message ${id} starts with whitespace`);
+      doesNotMatch(message, /\s$/, `Message ${id} ends with whitespace`);
+      doesNotMatch(message, /\s\s/, `Message ${id} has repeated whitespace`);
+      doesNotMatch(message, /\t/, `Message ${id} has tab whitespace`);
+      doesNotMatch(
         message,
         /[\u2000-\u200B\u2028\u2029]/,
         `Message ${id} has irregular whitespace`,
       );
-      assert.notMatch(
+      doesNotMatch(
         message,
         /\s[.,:;!?]/,
         `Message ${id} has whitespace before punctuation`,
       );
-      assert.notMatch(message, /'/, `Message ${id} has straight single quote`);
-      assert.notMatch(message, /"/, `Message ${id} has straight double quote`);
-      assert.notMatch(
+      doesNotMatch(message, /'/, `Message ${id} has straight single quote`);
+      doesNotMatch(message, /"/, `Message ${id} has straight double quote`);
+      doesNotMatch(
         message,
         /\s\u002D\s/,
         `Message ${id} uses Hyphen-Minus as Dash`,
       );
-      assert.notMatch(
+      doesNotMatch(
         message,
         /\s\u2010|\u2010\s/,
         `Message ${id} has space around Hyphen`,
       );
-      assert.notMatch(
+      doesNotMatch(
         message,
         /\s\u2011|\u2011\s/,
         `Message ${id} has space around Non-Breaking Hyphen`,