@thepalaceproject/circulation-admin 1.43.0-post.14 → 1.43.0-post.16

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

@@ -1,19 +1,18 @@
 import * as React from "react";
-import { fireEvent } from "@testing-library/react";
+import { installFormDataShim } from "../testUtils/formDataShim";
+import { fireEvent, screen, waitFor } from "@testing-library/react";
+import userEvent from "@testing-library/user-event";
 import { Collections } from "../../../src/components/Collections";
+import CollectionsConnected from "../../../src/components/Collections";
 import renderWithContext from "../testUtils/renderWithContext";
+import { renderWithProviders } from "../testUtils/withProviders";
+import buildStore from "../../../src/store";
 import {
   CollectionsData,
   ConfigurationSettings,
 } from "../../../src/interfaces";
 import { defaultFeatureFlags } from "../../../src/utils/featureFlags";
 
-// NB: This adds tests to the already existing tests in:
-// - `src/components/__tests__/Collections-test.tsx`.
-//
-// Those tests should eventually be migrated here and
-// adapted to the Jest/React Testing Library paradigm.
-
 describe("Collections - associated library disclosure", () => {
   // ── Shared fixtures ───────────────────────────────────────────────────────
 
@@ -219,3 +218,332 @@ describe("Collections - associated library disclosure", () => {
     );
   });
 });
+
+// These exercise behavior the disclosure tests above do not: the connect()
+// wiring (mapStateToProps / mapDispatchToProps, which only run when the CONNECTED
+// default export mounts), the marked-for-deletion list item, the
+// confirm-before-disassociating-libraries behavior, the
+// getChildContext().registerLibrary closure, the on-mount registrations fetch,
+// and the delete flow.
+
+describe("Collections - connected wiring, deletion, and registration", () => {
+  const allLibraries = [
+    { short_name: "nypl", name: "NYPL", uuid: "uuid-nypl" },
+  ];
+
+  const sysAdminConfig: Partial<ConfigurationSettings> = {
+    csrfToken: "",
+    featureFlags: defaultFeatureFlags,
+    roles: [{ role: "system" }],
+  };
+
+  afterEach(() => {
+    jest.restoreAllMocks();
+  });
+
+  describe("connected default export", () => {
+    // A loaded `libraries` slice persists through the on-mount collections fetch
+    // (the libraries reducer ignores that action), so mapStateToProps takes its
+    // `data.allLibraries` branch on every render.
+    const loadedLibrariesState = {
+      data: { libraries: allLibraries },
+      isFetching: false,
+      isEditing: false,
+      fetchError: null,
+      formError: null,
+      isLoaded: true,
+      responseBody: null,
+      successMessage: null,
+    };
+
+    const stubFetch = (body: unknown) =>
+      jest.spyOn(globalThis, "fetch").mockImplementation(
+        async () =>
+          new Response(JSON.stringify(body), {
+            status: 200,
+            headers: { "Content-Type": "application/json" },
+          })
+      );
+
+    it("wires mapStateToProps/mapDispatchToProps and marks a collection for deletion", async () => {
+      const listData = {
+        collections: [
+          { id: 2, protocol: "test protocol", name: "ODL" },
+          {
+            id: 3,
+            protocol: "test protocol",
+            name: "Enki",
+            marked_for_deletion: true,
+          },
+        ],
+        protocols: [{ name: "test protocol", label: "TP", settings: [] }],
+      };
+      stubFetch(listData);
+      const store = buildStore({
+        editor: { libraries: loadedLibrariesState },
+      } as any);
+
+      const { container } = renderWithProviders(
+        <CollectionsConnected csrfToken="token" />,
+        {
+          reduxProviderProps: { store },
+          appConfigSettings: sysAdminConfig,
+        }
+      );
+
+      // The list appears once the connected component's on-mount fetch resolves
+      // and mapStateToProps feeds the fetched data back in as props.
+      expect(
+        await screen.findByRole("heading", { level: 3, name: "ODL" })
+      ).toBeInTheDocument();
+
+      // The non-deleted collection has edit and delete controls.
+      const editLink = container.querySelector("a.edit-item");
+      expect(editLink.getAttribute("href")).toBe(
+        "/admin/web/config/collections/edit/2"
+      );
+
+      // The deleted collection renders a special item with no edit/delete
+      // controls and an explanatory message.
+      const deleted = container.querySelector(".deleted-collection");
+      expect(deleted).not.toBeNull();
+      expect(deleted).toHaveTextContent("Enki");
+      expect(deleted).toHaveTextContent(/cannot be edited/);
+      expect(deleted.querySelector("a.edit-item")).toBeNull();
+      expect(deleted.querySelector("button.delete-item")).toBeNull();
+    });
+  });
+
+  describe("edit mode: registration and additional content", () => {
+    // The reusable-components `Form` builds `new FormData(formElement)` on
+    // submit, which the unit jsdom env's undici FormData rejects; install the
+    // shared shim that reads the form's successful controls.
+    installFormDataShim();
+    it("fetches registrations on mount, renders task controls, and registers a library on click", async () => {
+      const user = userEvent.setup();
+      const registerLibrary = jest.fn().mockResolvedValue(undefined);
+      const fetchLibraryRegistrations = jest.fn().mockResolvedValue(undefined);
+      renderWithContext(
+        <Collections
+          data={
+            {
+              collections: [
+                {
+                  id: 7,
+                  protocol: "OPDS Import",
+                  name: "An OPDS Collection",
+                  libraries: [],
+                },
+              ],
+              protocols: [
+                {
+                  name: "OPDS Import",
+                  supports_registration: true,
+                  supports_import: true,
+                  settings: [],
+                  library_settings: [],
+                },
+              ],
+              allLibraries,
+            } as any
+          }
+          editOrCreate="edit"
+          identifier="7"
+          fetchData={jest.fn()}
+          editItem={jest.fn().mockResolvedValue(undefined)}
+          deleteItem={jest.fn().mockResolvedValue(undefined)}
+          registerLibrary={registerLibrary}
+          fetchLibraryRegistrations={fetchLibraryRegistrations}
+          importCollection={jest.fn().mockResolvedValue(undefined)}
+          reapCollection={jest.fn().mockResolvedValue(undefined)}
+          csrfToken="token"
+          isFetching={false}
+        />,
+        sysAdminConfig
+      );
+
+      // componentDidMount fetches the registrations once. (CollectionEditForm's
+      // renderAdditionalContent also runs here, since the edited collection has
+      // an id, injecting the import/reap task panels into the form.)
+      expect(fetchLibraryRegistrations).toHaveBeenCalledTimes(1);
+
+      // Registering a library goes through getChildContext().registerLibrary.
+      expect(registerLibrary).not.toHaveBeenCalled();
+      await user.click(screen.getByRole("button", { name: "Register" }));
+
+      expect(registerLibrary).toHaveBeenCalledTimes(1);
+      const formData = registerLibrary.mock.calls[0][0];
+      expect(formData.get("library_short_name")).toBe("nypl");
+      expect(formData.get("collection_id")).toBe("7");
+
+      // After a successful registration, the child context refetches.
+      await waitFor(() =>
+        expect(fetchLibraryRegistrations).toHaveBeenCalledTimes(2)
+      );
+    });
+  });
+
+  describe("confirm before disassociating libraries", () => {
+    const initialLibraries = [
+      { short_name: "palace", name: "Palace" },
+      { short_name: "another-library", name: "Another Library" },
+    ];
+
+    const renderEditForm = (
+      libraries: Array<{ short_name: string }>,
+      libraryMeta: Array<{ short_name: string; name?: string }>
+    ) =>
+      renderWithContext(
+        <Collections
+          data={
+            {
+              collections: [
+                {
+                  id: 7,
+                  protocol: "OPDS Import",
+                  name: "An OPDS Collection",
+                  libraries,
+                },
+              ],
+              protocols: [
+                { name: "OPDS Import", settings: [], library_settings: [] },
+              ],
+              allLibraries: libraryMeta,
+            } as any
+          }
+          editOrCreate="edit"
+          identifier="7"
+          fetchData={jest.fn()}
+          editItem={jest.fn().mockResolvedValue(undefined)}
+          deleteItem={jest.fn().mockResolvedValue(undefined)}
+          registerLibrary={jest.fn().mockResolvedValue(undefined)}
+          importCollection={jest.fn().mockResolvedValue(undefined)}
+          reapCollection={jest.fn().mockResolvedValue(undefined)}
+          csrfToken="token"
+          isFetching={false}
+        />,
+        sysAdminConfig
+      );
+
+    it("removes the library when the confirmation is accepted", () => {
+      const confirmSpy = jest.spyOn(window, "confirm").mockReturnValue(true);
+      const { container } = renderEditForm(
+        [...initialLibraries],
+        [...initialLibraries]
+      );
+
+      // One remove button per associated library.
+      expect(container.querySelectorAll("button.remove-btn")).toHaveLength(2);
+
+      fireEvent.click(container.querySelectorAll("button.remove-btn")[0]);
+
+      // The prompt names the (full) library being removed.
+      expect(confirmSpy).toHaveBeenCalledTimes(1);
+      expect(confirmSpy.mock.calls[0][0]).toBe(
+        'Disassociating library "Palace" from this collection will ' +
+          "remove all loans and holds for its patrons. Do you wish to continue?"
+      );
+
+      // The association (and its remove button) is gone.
+      expect(container.querySelectorAll("button.remove-btn")).toHaveLength(1);
+    });
+
+    it("keeps the library when the confirmation is canceled", () => {
+      jest.spyOn(window, "confirm").mockReturnValue(false);
+      const { container } = renderEditForm(
+        [...initialLibraries],
+        [...initialLibraries]
+      );
+
+      expect(container.querySelectorAll("button.remove-btn")).toHaveLength(2);
+      fireEvent.click(container.querySelectorAll("button.remove-btn")[0]);
+      // Nothing was removed.
+      expect(container.querySelectorAll("button.remove-btn")).toHaveLength(2);
+    });
+
+    it("uses the library short_name in the prompt when the full name is unavailable", () => {
+      const confirmSpy = jest.spyOn(window, "confirm").mockReturnValue(false);
+      // The library is not present in allLibraries, so getLibrary() returns null.
+      const { container } = renderEditForm([{ short_name: "orphan" }], []);
+
+      fireEvent.click(container.querySelector("button.remove-btn"));
+
+      expect(confirmSpy).toHaveBeenCalledTimes(1);
+      expect(confirmSpy.mock.calls[0][0]).toBe(
+        'Disassociating library "orphan" from this collection will ' +
+          "remove all loans and holds for its patrons. Do you wish to continue?"
+      );
+    });
+  });
+
+  describe("create mode", () => {
+    it("renders the create form without import/reap task panels", () => {
+      // In create mode there is no saved collection, so CollectionEditForm's
+      // renderAdditionalContent short-circuits and injects no task panels.
+      renderWithContext(
+        <Collections
+          data={
+            {
+              collections: [],
+              protocols: [
+                { name: "OPDS Import", settings: [], library_settings: [] },
+              ],
+              allLibraries,
+            } as any
+          }
+          editOrCreate="create"
+          fetchData={jest.fn()}
+          editItem={jest.fn().mockResolvedValue(undefined)}
+          deleteItem={jest.fn().mockResolvedValue(undefined)}
+          registerLibrary={jest.fn().mockResolvedValue(undefined)}
+          importCollection={jest.fn().mockResolvedValue(undefined)}
+          reapCollection={jest.fn().mockResolvedValue(undefined)}
+          csrfToken="token"
+          isFetching={false}
+        />,
+        sysAdminConfig
+      );
+
+      expect(
+        screen.getByRole("heading", { name: "Create a new collection" })
+      ).toBeInTheDocument();
+    });
+  });
+
+  describe("deletion", () => {
+    it("deletes a collection and refetches after confirmation", async () => {
+      const deleteItem = jest.fn().mockResolvedValue(undefined);
+      const fetchData = jest.fn();
+      const confirmSpy = jest.spyOn(window, "confirm").mockReturnValue(true);
+
+      const { container } = renderWithContext(
+        <Collections
+          data={
+            {
+              collections: [{ id: 1, protocol: "p", name: "My Collection" }],
+              protocols: [],
+              allLibraries,
+            } as any
+          }
+          fetchData={fetchData}
+          editItem={jest.fn().mockResolvedValue(undefined)}
+          deleteItem={deleteItem}
+          registerLibrary={jest.fn().mockResolvedValue(undefined)}
+          importCollection={jest.fn().mockResolvedValue(undefined)}
+          reapCollection={jest.fn().mockResolvedValue(undefined)}
+          csrfToken="token"
+          isFetching={false}
+        />,
+        sysAdminConfig
+      );
+
+      fireEvent.click(container.querySelector("button.delete-item"));
+
+      expect(confirmSpy).toHaveBeenCalledTimes(1);
+      await waitFor(() => expect(deleteItem).toHaveBeenCalledWith(1));
+      // fetchData is called once on mount; the post-delete refetch is the second
+      // call, so assert the count rather than mere invocation.
+      await waitFor(() => expect(fetchData).toHaveBeenCalledTimes(2));
+    });
+  });
+});

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

@@ -1,19 +1,18 @@
 import * as React from "react";
-import { fireEvent } from "@testing-library/react";
+import { installFormDataShim } from "../testUtils/formDataShim";
+import { fireEvent, screen, waitFor } from "@testing-library/react";
+import userEvent from "@testing-library/user-event";
 import { DiscoveryServices } from "../../../src/components/DiscoveryServices";
+import DiscoveryServicesConnected from "../../../src/components/DiscoveryServices";
 import renderWithContext from "../testUtils/renderWithContext";
+import { renderWithProviders } from "../testUtils/withProviders";
+import buildStore from "../../../src/store";
 import {
   ConfigurationSettings,
   DiscoveryServicesData,
 } from "../../../src/interfaces";
 import { defaultFeatureFlags } from "../../../src/utils/featureFlags";
 
-// NB: This adds tests to the already existing tests in:
-// - `src/components/__tests__/DiscoveryServices-test.tsx`.
-//
-// Those tests should eventually be migrated here and
-// adapted to the Jest/React Testing Library paradigm.
-
 describe("DiscoveryServices - registered library disclosure", () => {
   // ── Shared fixtures ───────────────────────────────────────────────────────
 
@@ -543,3 +542,151 @@ describe("DiscoveryServices - registered library disclosure", () => {
     expect(container.querySelectorAll(".associated-items")).toHaveLength(0);
   });
 });
+
+// These exercise behavior the disclosure tests above do not: the connect()
+// wiring (mapStateToProps / mapDispatchToProps, which only run when the CONNECTED
+// default export mounts) and the getChildContext().registerLibrary closure
+// (which only runs when a library is registered from the edit form).
+
+describe("DiscoveryServices - connected wiring and registration", () => {
+  const allLibraries = [
+    { short_name: "nypl", name: "NYPL", uuid: "uuid-nypl" },
+  ];
+
+  const sysAdminConfig: Partial<ConfigurationSettings> = {
+    csrfToken: "",
+    featureFlags: defaultFeatureFlags,
+    roles: [{ role: "system" }],
+  };
+
+  // A loaded `libraries` slice persists through the on-mount discovery fetches
+  // (the libraries reducer ignores those actions), so mapStateToProps takes its
+  // `data.allLibraries` branch on every render.
+  const loadedLibrariesState = {
+    data: { libraries: allLibraries },
+    isFetching: false,
+    isEditing: false,
+    fetchError: null,
+    formError: null,
+    isLoaded: true,
+    responseBody: null,
+    successMessage: null,
+  };
+
+  /**
+   * Stub `fetch` so the panel's on-mount fetches resolve with `body`. A fresh
+   * Response per call avoids "body already used" when several fetches run.
+   */
+  const stubFetch = (body: unknown) =>
+    jest.spyOn(globalThis, "fetch").mockImplementation(
+      async () =>
+        new Response(JSON.stringify(body), {
+          status: 200,
+          headers: { "Content-Type": "application/json" },
+        })
+    );
+
+  afterEach(() => {
+    jest.restoreAllMocks();
+  });
+
+  describe("connected default export", () => {
+    it("wires mapStateToProps/mapDispatchToProps: fetches on mount and renders the list", async () => {
+      const listData = {
+        discovery_services: [
+          { id: 2, protocol: "test protocol", name: "Test Discovery" },
+        ],
+        protocols: [{ name: "test protocol", label: "TP", settings: [] }],
+      };
+      stubFetch(listData);
+      // A fresh store, preloaded with a loaded libraries slice, isolates this
+      // test and exercises mapStateToProps' allLibraries branch.
+      const store = buildStore({
+        editor: { libraries: loadedLibrariesState },
+      } as any);
+
+      const { container } = renderWithProviders(
+        <DiscoveryServicesConnected csrfToken="token" />,
+        { reduxProviderProps: { store } }
+      );
+
+      // The list appears once the connected component's on-mount fetch resolves
+      // and mapStateToProps feeds the fetched data back in as props.
+      expect(
+        await screen.findByRole("heading", { level: 3, name: "Test Discovery" })
+      ).toBeInTheDocument();
+
+      const editLink = container.querySelector("a.edit-item");
+      expect(editLink).not.toBeNull();
+      expect(editLink.getAttribute("href")).toBe(
+        "/admin/web/config/discovery/edit/2"
+      );
+    });
+  });
+
+  describe("registerLibrary child context (edit mode)", () => {
+    // The reusable-components `Form` builds `new FormData(formElement)` on
+    // submit, which the unit jsdom env's undici FormData rejects; install the
+    // shared shim that reads the form's successful controls.
+    installFormDataShim();
+    const renderEditMode = (
+      registerLibrary: jest.Mock,
+      fetchLibraryRegistrations: jest.Mock
+    ) =>
+      renderWithContext(
+        <DiscoveryServices
+          data={
+            {
+              discovery_services: [
+                { id: "2", protocol: "test protocol", name: "Test Discovery" },
+              ],
+              protocols: [
+                {
+                  name: "test protocol",
+                  supports_registration: true,
+                  settings: [],
+                  library_settings: [],
+                },
+              ],
+              allLibraries,
+            } as any
+          }
+          editOrCreate="edit"
+          identifier="2"
+          fetchData={jest.fn()}
+          editItem={jest.fn().mockResolvedValue(undefined)}
+          deleteItem={jest.fn().mockResolvedValue(undefined)}
+          registerLibrary={registerLibrary}
+          fetchLibraryRegistrations={fetchLibraryRegistrations}
+          csrfToken="token"
+          isFetching={false}
+        />,
+        sysAdminConfig
+      );
+
+    it("fetches library registrations on mount, and registers a library (building the right FormData) and refetches on click", async () => {
+      const user = userEvent.setup();
+      const registerLibrary = jest.fn().mockResolvedValue(undefined);
+      const fetchLibraryRegistrations = jest.fn().mockResolvedValue(undefined);
+      renderEditMode(registerLibrary, fetchLibraryRegistrations);
+
+      // getChildContext supplies registerLibrary to the edit form; the edit form
+      // surfaces a "Register" button for each library.
+      expect(fetchLibraryRegistrations).toHaveBeenCalledTimes(1);
+      expect(registerLibrary).not.toHaveBeenCalled();
+
+      await user.click(screen.getByRole("button", { name: "Register" }));
+
+      expect(registerLibrary).toHaveBeenCalledTimes(1);
+      const formData = registerLibrary.mock.calls[0][0];
+      expect(formData.get("library_short_name")).toBe("nypl");
+      expect(formData.get("integration_id")).toBe("2");
+      expect(formData.get("registration_stage")).toBe("testing");
+
+      // After a successful registration, the child context refetches.
+      await waitFor(() =>
+        expect(fetchLibraryRegistrations).toHaveBeenCalledTimes(2)
+      );
+    });
+  });
+});