@thepalaceproject/circulation-admin 1.43.0-post.18 → 1.43.0-post.20

package/jest.config.js

@@ -1,15 +1,17 @@
 /** @type {import('ts-jest').JestConfigWithTsJest} */
-
-// Options shared by both projects.
-const sharedProjectConfig = {
+module.exports = {
   preset: "ts-jest",
+  // `jest-fixed-jsdom` restores Node's fetch/Response globals so MSW can
+  // intercept them.
+  testEnvironment: "jest-fixed-jsdom",
   testEnvironmentOptions: {
-    // Without this, jsdom's default `["browser"]` export condition resolves sinon
-    // to its ESM bundle, which ts-jest does not transform. Every legacy suite then
-    // dies with `SyntaxError: Unexpected token 'export'`.
+    // MSW / jest-fixed-jsdom rely on the default resolution conditions; the empty
+    // string keeps jsdom's "browser" condition from pulling in ESM-only builds
+    // that ts-jest would not transform.
     customExportConditions: [""],
   },
   setupFiles: ["<rootDir>/jest.polyfills.js"],
+  setupFilesAfterEnv: ["<rootDir>/tests/jest/jest-setup.ts"],
   moduleNameMapper: {
     "\\.(jpg|jpeg|png|gif|eot|otf|webp|svg|ttf|woff|woff2|mp4|webm|wav|mp3|m4a|aac|oga)$":
       "<rootDir>/tests/__mocks__/fileMock.js",
@@ -20,44 +22,9 @@ const sharedProjectConfig = {
   // files that no test imports — otherwise they are silently omitted from the
   // report rather than counted as uncovered, and the coverage percentage rises
   // while real coverage is unchanged.
-  roots: ["<rootDir>/src"],
-};
-
-module.exports = {
+  roots: ["<rootDir>/src", "<rootDir>/tests/jest"],
+  testMatch: ["<rootDir>/tests/jest/**/*.test.{js,jsx,ts,tsx}"],
   coverageDirectory: "coverage/jest",
   coverageReporters: ["lcov", "text-summary"],
-  // Must live at the top level, not inside `projects`: jest reads
-  // `collectCoverageFrom` off the *global* config. Set it only per-project and it
-  // is silently undefined globally, so every executed file gets instrumented —
-  // including test helpers and everything under `tests/`.
-  collectCoverageFrom: [
-    "src/**/*.{ts,tsx}",
-    "!src/**/__tests__/**",
-    "!src/**/*.d.ts",
-  ],
-  projects: [
-    {
-      ...sharedProjectConfig,
-      displayName: "unit",
-      roots: [...sharedProjectConfig.roots, "<rootDir>/tests/jest"],
-      testMatch: ["<rootDir>/tests/jest/**/*.test.{js,jsx,ts,tsx}"],
-      // `jest-fixed-jsdom` restores Node's fetch/Response globals so MSW can
-      // intercept them.
-      testEnvironment: "jest-fixed-jsdom",
-      setupFilesAfterEnv: ["<rootDir>/tests/jest/jest-setup.ts"],
-    },
-    {
-      // Legacy enzyme + chai + sinon tests, still in `src/**/__tests__/`. They are
-      // being rewritten as React Testing Library tests under `tests/jest/`; when the
-      // last one is gone, delete this project, its environment, and its setup file.
-      //
-      // These cannot use `jest-fixed-jsdom`: it swaps in undici's `FormData`, which
-      // rejects `new FormData(formElement)` and crashes any test that submits a form.
-      ...sharedProjectConfig,
-      displayName: "legacy",
-      testMatch: ["<rootDir>/src/**/__tests__/*-test.{ts,tsx}"],
-      testEnvironment: "<rootDir>/tests/jest/legacyJsdomEnvironment.js",
-      setupFilesAfterEnv: ["<rootDir>/tests/jest/legacy-setup.ts"],
-    },
-  ],
+  collectCoverageFrom: ["src/**/*.{ts,tsx}", "!src/**/*.d.ts"],
 };

package/package.json

@@ -84,13 +84,10 @@
     "@types/react-router": "^3.0.28",
     "@types/recharts": "^1.8.28",
     "@types/redux-mock-store": "^1.0.3",
-    "chai": "4.2.0",
     "clean-webpack-plugin": "^2.0.1",
     "copy-webpack-plugin": "14.0.0",
     "css-loader": "^7.1.4",
     "dotenv-cli": "^11.0.0",
-    "enzyme": "^3.11.0",
-    "enzyme-adapter-react-16": "^1.15.6",
     "eslint": "^9.39.4",
     "eslint-config-prettier": "^10.1.8",
     "eslint-plugin-jsx-a11y": "^6.10.2",
@@ -119,7 +116,6 @@
     "sass-lint": "^1.13.1",
     "sass-loader": "^17.0.0",
     "selenium-standalone": "^10.0.2",
-    "sinon": "22.0.0",
     "style-loader": "^4.0.0",
     "terser-webpack-plugin": "^5.3.6",
     "ts-jest": "^29.2.6",
@@ -143,5 +139,5 @@
     "*.{js,jsx,ts,tsx,css,md}": "prettier --write",
     "*.{js,css,md}": "prettier --write"
   },
-  "version": "1.43.0-post.18"
+  "version": "1.43.0-post.20"
 }

package/tests/jest/components/BookEditor.test.tsx

@@ -7,7 +7,6 @@ import {
   PER_LIBRARY_UNSUPPRESS_REL,
 } from "../../../src/features/book/bookEditorSlice";
 import { BookDetailsEditor } from "../../../src/components/BookDetailsEditor";
-import { expect } from "chai";
 import { store } from "../../../src/store";
 import * as fetchMock from "fetch-mock-jest";
 
@@ -93,8 +92,8 @@ describe("BookDetails", () => {
     const hideButton = queryByRole("button", { name: "Hide" });
     const restoreButton = queryByRole("button", { name: "Restore" });
 
-    expect(hideButton).to.be.null;
-    expect(restoreButton).to.be.null;
+    expect(hideButton).toBeNull();
+    expect(restoreButton).toBeNull();
   });
 
   it("don't show restore button if not a library's admin", () => {
@@ -110,8 +109,8 @@ describe("BookDetails", () => {
     const hideButton = queryByRole("button", { name: "Hide" });
     const restoreButton = queryByRole("button", { name: "Restore" });
 
-    expect(hideButton).to.be.null;
-    expect(restoreButton).to.be.null;
+    expect(hideButton).toBeNull();
+    expect(restoreButton).toBeNull();
   });
 
   it("uses modal for suppress book confirmation", async () => {
@@ -149,8 +148,8 @@ describe("BookDetails", () => {
     await user.click(cancelButton);
     confirmButton = queryByRole("button", { name: "Suppress Availability" });
     cancelButton = queryByRole("button", { name: "Cancel" });
-    expect(confirmButton).to.be.null;
-    expect(cancelButton).to.be.null;
+    expect(confirmButton).toBeNull();
+    expect(cancelButton).toBeNull();
 
     // Clicking `Hide` again should show the modal again.
     await user.click(hideButton);
@@ -163,15 +162,15 @@ describe("BookDetails", () => {
     getByRole("button", { name: "Dismiss" });
 
     // Check that the API was invoked.
-    expect(suppressBook.mock.calls.length).to.equal(1);
-    expect(suppressBook.mock.calls[0][0]).to.equal("/suppress/href");
+    expect(suppressBook.mock.calls.length).toBe(1);
+    expect(suppressBook.mock.calls[0][0]).toBe("/suppress/href");
     const fetchCalls = fetchMock.calls();
-    expect(fetchCalls.length).to.equal(1);
+    expect(fetchCalls.length).toBe(1);
     const fetchCall = fetchCalls[0];
     const fetchOptions = fetchCalls[0][1];
-    expect(fetchCall[0]).to.equal("/suppress/href");
-    expect(fetchOptions["headers"]["X-CSRF-Token"]).to.contain("token");
-    expect(fetchOptions["method"]).to.equal("POST");
+    expect(fetchCall[0]).toBe("/suppress/href");
+    expect(fetchOptions["headers"]["X-CSRF-Token"]).toContain("token");
+    expect(fetchOptions["method"]).toBe("POST");
   });
   it("uses modal for unsuppress book confirmation", async () => {
     // Configure standard constructors so that RTK Query works in tests with FetchMockJest
@@ -213,8 +212,8 @@ describe("BookDetails", () => {
     await user.click(cancelButton);
     confirmButton = queryByRole("button", { name: "Restore Availability" });
     cancelButton = queryByRole("button", { name: "Cancel" });
-    expect(confirmButton).to.be.null;
-    expect(cancelButton).to.be.null;
+    expect(confirmButton).toBeNull();
+    expect(cancelButton).toBeNull();
 
     // Clicking `Restore` again should show the modal again.
     await user.click(restoreButton);
@@ -227,14 +226,14 @@ describe("BookDetails", () => {
     getByRole("button", { name: "Dismiss" });
 
     // Check that the API was invoked.
-    expect(unsuppressBook.mock.calls.length).to.equal(1);
-    expect(unsuppressBook.mock.calls[0][0]).to.equal("/unsuppress/href");
+    expect(unsuppressBook.mock.calls.length).toBe(1);
+    expect(unsuppressBook.mock.calls[0][0]).toBe("/unsuppress/href");
     const fetchCalls = fetchMock.calls();
-    expect(fetchCalls.length).to.equal(1);
+    expect(fetchCalls.length).toBe(1);
     const fetchCall = fetchCalls[0];
     const fetchOptions = fetchCalls[0][1];
-    expect(fetchCall[0]).to.equal("/unsuppress/href");
-    expect(fetchOptions["headers"]["X-CSRF-Token"]).to.contain("token");
-    expect(fetchOptions["method"]).to.equal("DELETE");
+    expect(fetchCall[0]).toBe("/unsuppress/href");
+    expect(fetchOptions["headers"]["X-CSRF-Token"]).toContain("token");
+    expect(fetchOptions["method"]).toBe("DELETE");
   });
 });

package/tests/jest/components/Classifications.test.tsx

@@ -4,8 +4,8 @@ import userEvent from "@testing-library/user-event";
 
 import { renderWithProviders } from "../testUtils/withProviders";
 import buildStore from "../../../src/store";
-import genreData from "../../../src/components/__tests__/genreData";
-import classificationsData from "../../../src/components/__tests__/classificationsData";
+import genreData from "../fixtures/genreData";
+import classificationsData from "../fixtures/classificationsData";
 // Render the CONNECTED default export so that mapStateToProps / mapDispatchToProps
 // are exercised. The genre tree and classifications are fetched on mount and fed
 // back in through the Redux store; `book` and `refreshCatalog` are own props.

package/tests/jest/components/ClassificationsForm.test.tsx

@@ -4,7 +4,7 @@ import userEvent from "@testing-library/user-event";
 
 import ClassificationsForm from "../../../src/components/ClassificationsForm";
 import { BookData } from "../../../src/interfaces";
-import genreData from "../../../src/components/__tests__/genreData";
+import genreData from "../fixtures/genreData";
 
 // ClassificationsForm is a plain (unconnected) class component, so a bare
 // `render` is enough — no store/context/router needed.

package/tests/jest/components/ClassificationsTable.test.tsx

@@ -2,7 +2,7 @@ import * as React from "react";
 import { render, screen } from "@testing-library/react";
 
 import ClassificationsTable from "../../../src/components/ClassificationsTable";
-import classificationsData from "../../../src/components/__tests__/classificationsData";
+import classificationsData from "../fixtures/classificationsData";
 
 // Mirrors ClassificationsTable's own `readableType`: it strips the genres URL
 // prefix so the displayed type is human-readable.

package/tests/jest/components/CustomListsForBook.test.tsx

@@ -313,7 +313,7 @@ describe("CustomListsForBook", () => {
     expect(screen.getByRole("button", { name: "Add" })).toBeDisabled();
   });
 
-  it("resyncs its lists when the book url changes", () => {
+  it("shows the new book's lists when it is rerendered for another book", () => {
     const commonProps = {
       csrfToken: "token",
       library: "library",
@@ -331,10 +331,12 @@ describe("CustomListsForBook", () => {
         customListsForBook={listsForBook as any}
       />
     );
-    // list 2 (the book's only list) is in the current column.
-    expect(screen.getByText("list 2")).toBeInTheDocument();
+    // list 2 (the book's only list) is in the current column, which renders
+    // each list as a link; the add-menu renders options instead.
+    expect(screen.getByRole("link", { name: "list 2" })).toBeInTheDocument();
 
-    // A new book url re-syncs the current lists from the new customListsForBook.
+    // The current column renders straight from customListsForBook, so a new
+    // book's lists replace the old ones on rerender.
     rerender(
       <UnconnectedCustomListsForBook
         {...commonProps}
@@ -343,6 +345,9 @@ describe("CustomListsForBook", () => {
         customListsForBook={[allCustomLists[0]] as any}
       />
     );
-    expect(screen.getByText("list 1")).toBeInTheDocument();
+    expect(screen.getByRole("link", { name: "list 1" })).toBeInTheDocument();
+    expect(
+      screen.queryByRole("link", { name: "list 2" })
+    ).not.toBeInTheDocument();
   });
 });

package/tests/jest/components/GenreForm.test.tsx

@@ -3,7 +3,7 @@ import { render, screen, fireEvent } from "@testing-library/react";
 import userEvent from "@testing-library/user-event";
 
 import GenreForm from "../../../src/components/GenreForm";
-import genreData from "../../../src/components/__tests__/genreData";
+import genreData from "../fixtures/genreData";
 
 const genreOptions = Object.keys(genreData["Fiction"]).map(
   (name) => genreData["Fiction"][name]

package/tests/jest/components/Lane.test.tsx

@@ -1,7 +1,6 @@
 import * as React from "react";
 import { render, screen } from "@testing-library/react";
 import userEvent from "@testing-library/user-event";
-import { stub } from "sinon";
 import { LaneData } from "../../../src/interfaces";
 import Lane from "../../../src/components/Lane";
 
@@ -20,8 +19,8 @@ jest.mock("react-router", () => ({
   ),
 }));
 
-const renderLanes = stub();
-const toggleLaneVisibility = stub();
+const renderLanes = jest.fn();
+const toggleLaneVisibility = jest.fn();
 
 function createLaneData(displayName: string, isAutomated: boolean): LaneData {
   return {

package/tests/jest/components/LaneEditor.test.tsx

@@ -1,7 +1,6 @@
 import * as React from "react";
 import { render, screen, waitFor, within } from "@testing-library/react";
 import userEvent from "@testing-library/user-event";
-import { stub } from "sinon";
 import { LaneData } from "../../../src/interfaces";
 import LaneEditor from "../../../src/components/LaneEditor";
 import * as navigate from "../../../src/utils/navigate";
@@ -62,11 +61,15 @@ const customListsData = [
   { id: 1, name: "list 1", entries: [], is_owner: true, is_shared: false },
 ];
 
-const editLane = stub().returns(new Promise<void>((resolve) => resolve()));
+const editLane = jest
+  .fn()
+  .mockReturnValue(new Promise<void>((resolve) => resolve()));
 
-const deleteLane = stub().returns(new Promise<void>((resolve) => resolve()));
+const deleteLane = jest
+  .fn()
+  .mockReturnValue(new Promise<void>((resolve) => resolve()));
 
-const toggleLaneVisibility = stub();
+const toggleLaneVisibility = jest.fn();
 
 function createLaneData(displayName: string, isAutomated: boolean): LaneData {
   return {
@@ -104,7 +107,7 @@ describe("LaneEditor", () => {
           editOrCreate="edit"
           editLane={editLane}
           deleteLane={deleteLane}
-          findParentOfLane={stub().returns(laneData)}
+          findParentOfLane={jest.fn().mockReturnValue(laneData)}
           toggleLaneVisibility={toggleLaneVisibility}
         />
       );
@@ -139,7 +142,7 @@ describe("LaneEditor", () => {
           editOrCreate="edit"
           editLane={editLane}
           deleteLane={deleteLane}
-          findParentOfLane={stub().returns(laneData)}
+          findParentOfLane={jest.fn().mockReturnValue(laneData)}
           toggleLaneVisibility={toggleLaneVisibility}
         />
       );
@@ -177,7 +180,7 @@ describe("LaneEditor", () => {
         editOrCreate="edit"
         editLane={editLane}
         deleteLane={deleteLane}
-        findParentOfLane={stub().returns(laneData)}
+        findParentOfLane={jest.fn().mockReturnValue(laneData)}
         toggleLaneVisibility={toggleLaneVisibility}
       />
     );

package/tests/jest/components/Stats.test.tsx

@@ -28,13 +28,6 @@ const normalizedData = normalizeStatistics(statisticsApiResponseData);
 global.ResizeObserver = require("resize-observer-polyfill");
 
 describe("Dashboard Statistics", () => {
-  // NB: This adds test to the already existing tests in:
-  // - `src/components/__tests__/LibraryStats-test.tsx`.
-  // - `src/components/__tests__/SingleStatListItem-test.tsx`.
-  //
-  // Those tests should eventually be migrated here and
-  // adapted to the Jest/React Testing Library paradigm.
-
   // Configure standard constructors so that RTK Query works in tests with FetchMockJest
   Object.assign(fetchMock.config, {
     fetch,

package/tests/jest/features/book.test.ts

@@ -3,7 +3,6 @@ import reducer, {
   getBookData,
   submitBookData,
 } from "../../../src/features/book/bookEditorSlice";
-import { expect } from "chai";
 import * as fetchMock from "fetch-mock-jest";
 import { BookData } from "@thepalaceproject/web-opds-client/lib/interfaces";
 import { RequestError } from "@thepalaceproject/web-opds-client/lib/DataFetcher";
@@ -57,12 +56,12 @@ describe("Redux bookEditorSlice...", () => {
 
   describe("reducers...", () => {
     it("should return the initial state from undefined, if no action is passed", () => {
-      expect(reducer(undefined, { type: "unknown" })).to.deep.equal(
+      expect(reducer(undefined, { type: "unknown" })).toStrictEqual(
         initialState
       );
     });
     it("should return the initial state from initialState, if no action is passed", () => {
-      expect(reducer(initialState, { type: "unknown" })).to.deep.equal(
+      expect(reducer(initialState, { type: "unknown" })).toStrictEqual(
         initialState
       );
     });
@@ -70,7 +69,7 @@ describe("Redux bookEditorSlice...", () => {
       // This is dispatched by `web-opds`client`, but we need to handle it, too.
       const action = { type: ActionCreator.BOOK_CLEAR };
 
-      expect(reducer(fetchedState, action)).to.deep.equal(initialState);
+      expect(reducer(fetchedState, action)).toStrictEqual(initialState);
     });
 
     it("should handle getBookData.pending", () => {
@@ -81,11 +80,11 @@ describe("Redux bookEditorSlice...", () => {
       const previousState = { ...initialState, url: null, isFetching: false };
       const state = reducer(previousState, action);
 
-      expect(state.url).to.equal("https://example.com/book");
-      expect(state.data).to.be.null;
-      expect(state.isFetching).to.equal(true);
-      expect(state.fetchError).to.be.null;
-      expect(state.editError).to.be.null;
+      expect(state.url).toBe("https://example.com/book");
+      expect(state.data).toBeNull();
+      expect(state.isFetching).toBe(true);
+      expect(state.fetchError).toBeNull();
+      expect(state.editError).toBeNull();
     });
     it("should handle getBookData.fulfilled", () => {
       const action = {
@@ -101,11 +100,11 @@ describe("Redux bookEditorSlice...", () => {
       };
       const state = reducer(previousState, action);
 
-      expect(state.url).to.equal("https://example.com/book");
-      expect(state.data).to.deep.equal(bookData);
-      expect(state.isFetching).to.equal(false);
-      expect(state.fetchError).to.be.null;
-      expect(state.editError).to.be.null;
+      expect(state.url).toBe("https://example.com/book");
+      expect(state.data).toStrictEqual(bookData);
+      expect(state.isFetching).toBe(false);
+      expect(state.fetchError).toBeNull();
+      expect(state.editError).toBeNull();
     });
     it("should handle getBookData.rejected", () => {
       const errorObject = { error: "some error object" };
@@ -122,11 +121,11 @@ describe("Redux bookEditorSlice...", () => {
       };
       const state = reducer(previousState, action);
 
-      expect(state.url).to.equal("https://example.com/book");
-      expect(state.data).to.be.null;
-      expect(state.isFetching).to.equal(false);
-      expect(state.fetchError).to.deep.equal(errorObject);
-      expect(state.editError).to.be.null;
+      expect(state.url).toBe("https://example.com/book");
+      expect(state.data).toBeNull();
+      expect(state.isFetching).toBe(false);
+      expect(state.fetchError).toStrictEqual(errorObject);
+      expect(state.editError).toBeNull();
     });
 
     it("should handle submitBookData.pending", () => {
@@ -134,7 +133,7 @@ describe("Redux bookEditorSlice...", () => {
       const previousState = { ...fetchedState, isFetching: false };
       const state = reducer(previousState, action);
 
-      expect(state).to.deep.equal({ ...fetchedState, isFetching: true });
+      expect(state).toStrictEqual({ ...fetchedState, isFetching: true });
     });
     it("should handle submitBookData.fulfilled", () => {
       const action = {
@@ -144,7 +143,7 @@ describe("Redux bookEditorSlice...", () => {
       const previousState = { ...fetchedState, isFetching: true };
       const state = reducer(previousState, action);
 
-      expect(state).to.deep.equal({
+      expect(state).toStrictEqual({
         ...fetchedState,
         isFetching: false,
         editError: null,
@@ -158,7 +157,7 @@ describe("Redux bookEditorSlice...", () => {
       const previousState = { ...fetchedState, isFetching: true };
       const state = reducer(previousState, action);
 
-      expect(state).to.deep.equal({
+      expect(state).toStrictEqual({
         ...fetchedState,
         isFetching: false,
         editError: "some value",
@@ -200,22 +199,22 @@ describe("Redux bookEditorSlice...", () => {
         const dispatchCalls = dispatch.mock.calls;
 
         const payload = result.payload as BookData;
-        expect(payload.id).to.equal(
+        expect(payload.id).toBe(
           "urn:uuid:1cca9468-c447-4303-bc5a-c57470b85cb1"
         );
-        expect(payload.title).to.equal(
+        expect(payload.title).toBe(
           "Tea-Cup Reading and Fortune-Telling by Tea Leaves"
         );
 
-        expect(dispatchCalls.length).to.equal(2);
-        expect(dispatchCalls[0][0].type).to.equal(getBookData.pending.type);
-        expect(dispatchCalls[0][0].payload).to.equal(undefined);
-        expect(dispatchCalls[0][0].meta.arg).to.deep.equal({
+        expect(dispatchCalls.length).toBe(2);
+        expect(dispatchCalls[0][0].type).toBe(getBookData.pending.type);
+        expect(dispatchCalls[0][0].payload).toBe(undefined);
+        expect(dispatchCalls[0][0].meta.arg).toStrictEqual({
           url: goodBookUrl,
         });
-        expect(dispatchCalls[1][0].type).to.equal(getBookData.fulfilled.type);
-        expect(dispatchCalls[1][0].payload).to.deep.equal(payload);
-        expect(dispatchCalls[1][0].meta.arg).to.deep.equal({
+        expect(dispatchCalls[1][0].type).toBe(getBookData.fulfilled.type);
+        expect(dispatchCalls[1][0].payload).toStrictEqual(payload);
+        expect(dispatchCalls[1][0].meta.arg).toStrictEqual({
           url: goodBookUrl,
         });
       });
@@ -226,18 +225,18 @@ describe("Redux bookEditorSlice...", () => {
         const dispatchCalls = dispatch.mock.calls;
 
         const payload = result.payload as RequestError;
-        expect(payload.response).to.equal(FETCH_OPDS_PARSE_ERROR_MESSAGE);
-        expect(payload.url).to.equal(brokenBookUrl);
+        expect(payload.response).toBe(FETCH_OPDS_PARSE_ERROR_MESSAGE);
+        expect(payload.url).toBe(brokenBookUrl);
 
-        expect(dispatchCalls.length).to.equal(2);
-        expect(dispatchCalls[0][0].type).to.equal(getBookData.pending.type);
-        expect(dispatchCalls[0][0].payload).to.equal(undefined);
-        expect(dispatchCalls[0][0].meta.arg).to.deep.equal({
+        expect(dispatchCalls.length).toBe(2);
+        expect(dispatchCalls[0][0].type).toBe(getBookData.pending.type);
+        expect(dispatchCalls[0][0].payload).toBe(undefined);
+        expect(dispatchCalls[0][0].meta.arg).toStrictEqual({
           url: brokenBookUrl,
         });
-        expect(dispatchCalls[1][0].type).to.equal(getBookData.rejected.type);
-        expect(dispatchCalls[1][0].payload).to.deep.equal(payload);
-        expect(dispatchCalls[1][0].meta.arg).to.deep.equal({
+        expect(dispatchCalls[1][0].type).toBe(getBookData.rejected.type);
+        expect(dispatchCalls[1][0].payload).toStrictEqual(payload);
+        expect(dispatchCalls[1][0].meta.arg).toStrictEqual({
           url: brokenBookUrl,
         });
       });
@@ -249,20 +248,20 @@ describe("Redux bookEditorSlice...", () => {
 
         const payload = result.payload as RequestError;
 
-        expect(result.type).to.equal(getBookData.rejected.type);
-        expect(result.meta.arg).to.deep.equal({ url: errorBookUrl });
-        expect(payload.response).to.equal("Internal server error");
-        expect(payload.url).to.equal(errorBookUrl);
+        expect(result.type).toBe(getBookData.rejected.type);
+        expect(result.meta.arg).toStrictEqual({ url: errorBookUrl });
+        expect(payload.response).toBe("Internal server error");
+        expect(payload.url).toBe(errorBookUrl);
 
-        expect(dispatchCalls.length).to.equal(2);
-        expect(dispatchCalls[0][0].type).to.equal(getBookData.pending.type);
-        expect(dispatchCalls[0][0].payload).to.equal(undefined);
-        expect(dispatchCalls[0][0].meta.arg).to.deep.equal({
+        expect(dispatchCalls.length).toBe(2);
+        expect(dispatchCalls[0][0].type).toBe(getBookData.pending.type);
+        expect(dispatchCalls[0][0].payload).toBe(undefined);
+        expect(dispatchCalls[0][0].meta.arg).toStrictEqual({
           url: errorBookUrl,
         });
-        expect(dispatchCalls[1][0].type).to.equal(getBookData.rejected.type);
-        expect(dispatchCalls[1][0].payload).to.deep.equal(payload);
-        expect(dispatchCalls[1][0].meta.arg).to.deep.equal({
+        expect(dispatchCalls[1][0].type).toBe(getBookData.rejected.type);
+        expect(dispatchCalls[1][0].payload).toStrictEqual(payload);
+        expect(dispatchCalls[1][0].meta.arg).toStrictEqual({
           url: errorBookUrl,
         });
       });
@@ -327,27 +326,23 @@ describe("Redux bookEditorSlice...", () => {
         const dispatchCalls = dispatch.mock.calls;
         const fetchCalls = fetchMock.calls();
 
-        expect(fetchCalls.length).to.equal(1);
-        expect(fetchCalls[0].identifier).to.equal("valid-csrf-token-post");
-        expect(
-          (fetchCalls[0][1].headers as Headers).get(csrfTokenHeader)
-        ).to.equal(validCsrfToken);
+        expect(fetchCalls.length).toBe(1);
+        expect(fetchCalls[0].identifier).toBe("valid-csrf-token-post");
+        expect((fetchCalls[0][1].headers as Headers).get(csrfTokenHeader)).toBe(
+          validCsrfToken
+        );
 
-        expect(fetchCalls[0][0]).to.equal(editBookUrl);
-        expect(fetchCalls[0][1].method).to.equal("POST");
-        expect(fetchCalls[0][1].body).to.equal(formData);
+        expect(fetchCalls[0][0]).toBe(editBookUrl);
+        expect(fetchCalls[0][1].method).toBe("POST");
+        expect(fetchCalls[0][1].body).toBe(formData);
 
-        expect(dispatchCalls.length).to.equal(3);
-        expect(dispatchCalls[0][0].type).to.equal(submitBookData.pending.type);
-        expect(dispatchCalls[0][0].payload).to.equal(undefined);
+        expect(dispatchCalls.length).toBe(3);
+        expect(dispatchCalls[0][0].type).toBe(submitBookData.pending.type);
+        expect(dispatchCalls[0][0].payload).toBe(undefined);
         // On a successful update, the second dispatch is to re-fetch the updated book data.
         // The third dispatch is for the fulfilled action.
-        expect(dispatchCalls[2][0].type).to.equal(
-          submitBookData.fulfilled.type
-        );
-        expect(dispatchCalls[2][0].payload.body.toString()).to.equal(
-          "Success!"
-        );
+        expect(dispatchCalls[2][0].type).toBe(submitBookData.fulfilled.type);
+        expect(dispatchCalls[2][0].payload.body.toString()).toBe("Success!");
       });
       it("should fail, if the user is unauthorized", async () => {
         const csrfToken = "invalid-token";
@@ -365,24 +360,24 @@ describe("Redux bookEditorSlice...", () => {
         const dispatchCalls = dispatch.mock.calls;
         const fetchCalls = fetchMock.calls();
 
-        expect(fetchCalls.length).to.equal(1);
-        expect(fetchCalls[0].identifier).to.equal("invalid-csrf-token-post");
+        expect(fetchCalls.length).toBe(1);
+        expect(fetchCalls[0].identifier).toBe("invalid-csrf-token-post");
         expect(
           (fetchCalls[0][1].headers as Headers).get(csrfTokenHeader)
-        ).not.to.equal(validCsrfToken);
+        ).not.toBe(validCsrfToken);
 
-        expect(fetchCalls[0][0]).to.equal(editBookUrl);
-        expect(fetchCalls[0][1].method).to.equal("POST");
-        expect(fetchCalls[0][1].body).to.equal(formData);
+        expect(fetchCalls[0][0]).toBe(editBookUrl);
+        expect(fetchCalls[0][1].method).toBe("POST");
+        expect(fetchCalls[0][1].body).toBe(formData);
 
-        expect(dispatchCalls.length).to.equal(2);
-        expect(dispatchCalls[0][0].type).to.equal(submitBookData.pending.type);
-        expect(dispatchCalls[0][0].payload).to.equal(undefined);
+        expect(dispatchCalls.length).toBe(2);
+        expect(dispatchCalls[0][0].type).toBe(submitBookData.pending.type);
+        expect(dispatchCalls[0][0].payload).toBe(undefined);
         // There is no re-fetch on a failed request, ...
         // ...so the second dispatch is for the rejected action.
-        expect(dispatchCalls[1][0].type).to.equal(submitBookData.rejected.type);
-        expect(dispatchCalls[1][0].payload.status).to.equal(400);
-        expect(dispatchCalls[1][0].payload.response).to.equal(
+        expect(dispatchCalls[1][0].type).toBe(submitBookData.rejected.type);
+        expect(dispatchCalls[1][0].payload.status).toBe(400);
+        expect(dispatchCalls[1][0].payload.response).toBe(
           "There was an error saving your changes."
         );
       });