@thepalaceproject/circulation-admin 1.45.0 → 1.46.0-post.1

package/package.json

@@ -139,5 +139,5 @@
     "*.{js,jsx,ts,tsx,css,md}": "prettier --write",
     "*.{js,css,md}": "prettier --write"
   },
-  "version": "1.45.0"
+  "version": "1.46.0-post.1"
 }

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

@@ -364,4 +364,19 @@ describe("CustomListEditor", () => {
       container.querySelector(".sharing-info button")
     ).not.toBeInTheDocument();
   });
+
+  it("does not render a share button once the list is already shared, and shows the shared-state copy", () => {
+    const { container } = renderEditor({ isShared: true });
+
+    expect(
+      container.querySelector(".sharing-info button")
+    ).not.toBeInTheDocument();
+
+    expect(
+      screen.getByText(
+        "This list has been shared with other libraries, and will automatically be made available to any libraries registered in this Collection Manager, including libraries registered in the future.",
+        { exact: false }
+      )
+    ).toBeInTheDocument();
+  });
 });

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

@@ -258,6 +258,92 @@ describe("PatronAuthServiceEditForm – serialization", () => {
   });
 });
 
+describe("PatronAuthServiceEditForm – per-library settings", () => {
+  const LIB_SETTING_KEY = "lib_setting";
+  const protocolWithLibrarySettings = {
+    ...SIP2_PROTOCOL_DATA,
+    library_settings: [{ key: LIB_SETTING_KEY, label: "Library setting" }],
+  };
+  const dataWithLibrarySettings: PatronAuthServicesData = {
+    ...servicesData,
+    protocols: [protocolWithLibrarySettings, OTHER_PROTOCOL_DATA],
+  };
+  const settingInput = (root: ParentNode) =>
+    root.querySelector(`input[name="${LIB_SETTING_KEY}"]`) as HTMLInputElement;
+
+  it("keeps a library setting entered when adding a new library", async () => {
+    const user = userEvent.setup();
+    const item = {
+      id: 3,
+      protocol: SIP2_PROTOCOL,
+      settings: {},
+      libraries: [],
+    };
+    const { container } = renderForm(
+      <PatronAuthServiceEditForm
+        {...baseProps}
+        data={dataWithLibrarySettings}
+        item={item}
+      />
+    );
+
+    await expandLibrariesPanel(user);
+    await user.selectOptions(
+      screen.getByRole("combobox", { name: /Add Library/i }),
+      LIBRARY_SHORT_NAME
+    );
+    await user.type(settingInput(container), "added value");
+    await user.click(screen.getByRole("button", { name: /Add Library/i }));
+
+    // Reopen the new library's settings; the value entered on add is there.
+    await user.click(screen.getByRole("button", { name: /Edit/i }));
+    const editSection = container.querySelector(
+      ".edit-library-settings"
+    ) as HTMLElement;
+    expect(settingInput(editSection).value).toBe("added value");
+  });
+
+  it("keeps a library setting changed when editing an existing library", async () => {
+    const user = userEvent.setup();
+    const item = {
+      ...buildSIP2Item(),
+      libraries: [
+        { short_name: LIBRARY_SHORT_NAME, [LIB_SETTING_KEY]: "old value" },
+      ],
+    };
+    const { container } = renderForm(
+      <PatronAuthServiceEditForm
+        {...baseProps}
+        data={dataWithLibrarySettings}
+        item={item}
+      />
+    );
+
+    await expandLibrariesPanel(user);
+    const clickEdit = () =>
+      user.click(screen.getByRole("button", { name: /Edit/i }));
+    await clickEdit();
+    let editSection = container.querySelector(
+      ".edit-library-settings"
+    ) as HTMLElement;
+    expect(settingInput(editSection).value).toBe("old value");
+
+    await user.clear(settingInput(editSection));
+    await user.type(settingInput(editSection), "new value");
+    await user.click(
+      editSection.querySelector("button.edit-library") as HTMLElement
+    );
+
+    // Saving collapses the library; reopening shows the saved value.
+    expect(container.querySelector(".edit-library-settings")).toBeNull();
+    await clickEdit();
+    editSection = container.querySelector(
+      ".edit-library-settings"
+    ) as HTMLElement;
+    expect(settingInput(editSection).value).toBe("new value");
+  });
+});
+
 describe("PatronAuthServiceEditForm – save button gating", () => {
   it("disables the per-library Save button immediately when a new rule is added", async () => {
     const user = userEvent.setup();

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

@@ -896,6 +896,27 @@ describe("ServiceEditForm", () => {
       expect(libraries).toHaveLength(0);
     });
 
+    it("removes an expanded library and closes its settings", async () => {
+      const user = userEvent.setup();
+      const { container } = renderForm({ item: serviceData });
+      const libraryItem = container.querySelector(
+        ".with-remove-button"
+      ) as HTMLElement;
+
+      await user.click(
+        libraryItem.querySelector("button.edit-btn") as HTMLElement
+      );
+      expect(
+        container.querySelector(".edit-library-settings")
+      ).toBeInTheDocument();
+
+      await user.click(
+        libraryItem.querySelector("button.remove-btn") as HTMLElement
+      );
+      expect(container.querySelectorAll(".with-remove-button")).toHaveLength(0);
+      expect(container.querySelector(".edit-library-settings")).toBeNull();
+    });
+
     it("edits a library", async () => {
       const user = userEvent.setup();
       const { container } = renderForm({ item: serviceData });
@@ -1059,6 +1080,28 @@ describe("ServiceEditForm", () => {
       expect(nameInput().value).toBe("");
     });
 
+    it("clears every associated library after a successful save (PP-5153)", () => {
+      const threeLibraries = {
+        ...serviceData,
+        libraries: [
+          { short_name: "nypl" },
+          { short_name: "bpl" },
+          { short_name: "qpl" },
+        ],
+      };
+      const { container, rerender } = renderForm({ item: threeLibraries });
+      expect(container.querySelectorAll(".with-remove-button")).toHaveLength(3);
+
+      // clearForm fires onRemove once per library inside one React batch.
+      // Each removal must build on the previous one, otherwise only the last
+      // one survives and the other libraries stay in the list.
+      rerenderForm(rerender, {
+        item: threeLibraries,
+        responseBody: "new service",
+      });
+      expect(container.querySelectorAll(".with-remove-button")).toHaveLength(0);
+    });
+
     it("doesn't clear the form if there's an error message", async () => {
       const user = userEvent.setup();
       const { container, rerender } = renderForm();