@thepalaceproject/circulation-admin 1.28.1 → 1.29.0-post.3

package/dist/circulation-admin.js.LICENSE.txt

@@ -29,7 +29,7 @@ object-assign
  * @license  MIT
  */
 
-/*! @license DOMPurify | (c) Cure53 and other contributors | Released under the Apache license 2.0 and Mozilla Public License 2.0 | github.com/cure53/DOMPurify/blob/2.2.2/LICENSE */
+/*! @license DOMPurify 3.2.4 | (c) Cure53 and other contributors | Released under the Apache license 2.0 and Mozilla Public License 2.0 | github.com/cure53/DOMPurify/blob/3.2.4/LICENSE */
 
 /*! @preserve
  * numeral.js

package/package.json

@@ -38,7 +38,7 @@
     "@nypl/dgx-svg-icons": "0.3.4",
     "@reduxjs/toolkit": "^2.2.5",
     "@tanstack/react-query": "^4.36.1",
-    "@thepalaceproject/web-opds-client": "^1.0.1",
+    "@thepalaceproject/web-opds-client": "^1.1.0",
     "bootstrap": "^3.3.6",
     "classnames": "^2.3.1",
     "draft-convert": "^2.1.5",
@@ -151,5 +151,5 @@
     "*.{js,jsx,ts,tsx,css,md}": "prettier --write",
     "*.{js,css,md}": "prettier --write"
   },
-  "version": "1.28.1"
+  "version": "1.29.0-post.3"
 }

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

@@ -0,0 +1,66 @@
+import * as React from "react";
+import { screen, fireEvent } from "@testing-library/react";
+import "@testing-library/jest-dom";
+
+import CirculationEventsDownload from "../../../src/components/CirculationEventsDownload";
+import { ContextProviderProps } from "../../../src/components/ContextProvider";
+import { FeatureFlags } from "../../../src/interfaces";
+import { defaultFeatureFlags } from "../../../src/utils/featureFlags";
+import { renderWithProviders } from "../testUtils/withProviders";
+
+describe("CirculationEventsDownload", () => {
+  it("renders nothing if showCircEventsDownload feature flag is false", () => {
+    const featureFlags: FeatureFlags = {
+      ...defaultFeatureFlags,
+      showCircEventsDownload: false,
+    };
+    const contextProviderProps: Partial<ContextProviderProps> = {
+      featureFlags,
+    };
+    const { container } = renderWithProviders(
+      <CirculationEventsDownload library="testlib" />,
+      {
+        contextProviderProps,
+      }
+    );
+    expect(container).toBeEmptyDOMElement();
+  });
+
+  describe("when showCircEventsDownload feature flag is true", () => {
+    const featureFlags: FeatureFlags = {
+      ...defaultFeatureFlags,
+      showCircEventsDownload: true,
+    };
+    const libraryProp = "testlib";
+    const contextProviderProps: Partial<ContextProviderProps> = {
+      featureFlags,
+    };
+
+    beforeEach(() => {
+      renderWithProviders(<CirculationEventsDownload library={libraryProp} />, {
+        contextProviderProps,
+      });
+    });
+
+    it("renders a download button", () => {
+      expect(
+        screen.getByRole("button", { name: "Download CSV" })
+      ).toBeInTheDocument();
+    });
+
+    it("shows the form when the download button is clicked", () => {
+      const downloadButton = screen.getByRole("button", {
+        name: "Download CSV",
+      });
+      fireEvent.click(downloadButton);
+      expect(
+        screen.getByRole("heading", { name: "Download CSV" })
+      ).toBeInTheDocument();
+      expect(
+        screen.getByRole("button", { name: "Download" })
+      ).toBeInTheDocument();
+      expect(screen.getByText("Start Date")).toBeInTheDocument();
+      expect(screen.getByText("End Date")).toBeInTheDocument();
+    });
+  });
+});

package/tests/jest/context/AppContext.test.tsx

@@ -5,6 +5,7 @@ import {
   useAppEmail,
   useAppFeatureFlags,
   useCsrfToken,
+  useTermsOfService,
 } from "../../../src/context/appContext";
 import { componentWithProviders } from "../testUtils/withProviders";
 import { ContextProviderProps } from "../../../src/components/ContextProvider";
@@ -25,12 +26,18 @@ describe("AppContext", () => {
     testFalse: false,
   };
   const expectedRoles = [{ role: "system" }];
+  const expectedTermsOfService = {
+    text: "Terms of Service",
+    href: "/terms-of-service",
+  };
 
   const contextProviderProps: ContextProviderProps = {
     csrfToken: expectedCsrfToken,
     featureFlags: expectedFeatureFlags,
     roles: expectedRoles,
     email: expectedEmail,
+    tos_link_text: expectedTermsOfService.text,
+    tos_link_href: expectedTermsOfService.href,
   };
   const wrapper = componentWithProviders({ contextProviderProps });
 
@@ -69,4 +76,15 @@ describe("AppContext", () => {
     const flags = result.current;
     expect(flags).toEqual(expectedFeatureFlags);
   });
+
+  it("provides useTermsOfService context hook", () => {
+    const { result } = renderHook(() => useTermsOfService(), {
+      wrapper,
+    });
+    const tosLink = result.current;
+    const { text, href } = tosLink;
+    expect(tosLink).toEqual(expectedTermsOfService);
+    expect(text).toEqual(expectedTermsOfService.text);
+    expect(href).toEqual(expectedTermsOfService.href);
+  });
 });

package/tests/jest/testUtils/withProviders.tsx

@@ -7,15 +7,10 @@ import { QueryClient, QueryClientProvider } from "@tanstack/react-query";
 import { render, RenderOptions, RenderResult } from "@testing-library/react";
 import { defaultFeatureFlags } from "../../../src/utils/featureFlags";
 import { store } from "../../../src/store";
-import {
-  TOSContextProvider,
-  TOSContextProviderProps,
-} from "../../../src/components/TOSContext";
 
 export type TestProviderWrapperOptions = {
   reduxProviderProps?: ProviderProps;
   contextProviderProps?: Partial<ContextProviderProps>;
-  tosContextProviderProps?: TOSContextProviderProps;
   queryClient?: QueryClient;
 };
 export type TestRenderWrapperOptions = TestProviderWrapperOptions & {
@@ -26,13 +21,6 @@ export type TestRenderWrapperOptions = TestProviderWrapperOptions & {
 // be the same for both the Redux Provider and the ContextProvider.
 const defaultReduxStore = store;
 
-// Setup default TOSContext provider props.
-const tosText = "Sample terms of service.";
-const tosHref = "http://example.com/terms-of-service";
-const requiredTOSContextProviderProps: TOSContextProviderProps = {
-  ...[tosText, tosHref],
-};
-
 // The `csrfToken` context provider prop is required, so we provide
 // a default value here, so it can be easily merged with other props.
 const requiredContextProviderProps: ContextProviderProps = {
@@ -47,7 +35,6 @@ const requiredContextProviderProps: ContextProviderProps = {
  * @param {TestProviderWrapperOptions} options
  * @param options.reduxProviderProps Props to pass to the Redux `Provider` wrapper
  * @param {ContextProviderProps} options.contextProviderProps Props to pass to the ContextProvider wrapper
- * @param {TOSContextProviderProps} options.tosContextProviderProps Props to pass to the TOSContextProvider wrapper
  * @param {QueryClient} options.queryClient A `tanstack/react-query` QueryClient
  * @returns {React.FunctionComponent} A React component that wraps children with our providers
  */
@@ -59,7 +46,6 @@ export const componentWithProviders = ({
     csrfToken: "",
     featureFlags: defaultFeatureFlags,
   },
-  tosContextProviderProps = requiredTOSContextProviderProps,
   queryClient = new QueryClient(),
 }: TestProviderWrapperOptions = {}): React.FunctionComponent => {
   const effectiveContextProviderProps = {
@@ -70,11 +56,9 @@ export const componentWithProviders = ({
   const wrapper = ({ children }) => (
     <Provider {...reduxProviderProps}>
       <ContextProvider {...effectiveContextProviderProps}>
-        <TOSContextProvider value={tosContextProviderProps}>
-          <QueryClientProvider client={queryClient}>
-            {children}
-          </QueryClientProvider>
-        </TOSContextProvider>
+        <QueryClientProvider client={queryClient}>
+          {children}
+        </QueryClientProvider>
       </ContextProvider>
     </Provider>
   );

package/tests/jest/utils/NoCacheDataFetcher.test.ts

@@ -0,0 +1,75 @@
+// Need to manually mock DataFetcher before importing NoCacheDataFetcher
+const mockSuperFetch = jest.fn();
+
+// Mock the DataFetcher module
+jest.mock("@thepalaceproject/web-opds-client/lib/DataFetcher", () => ({
+  __esModule: true,
+  default: class MockDataFetcher {
+    fetch(url: string, options?: any) {
+      return mockSuperFetch(url, options);
+    }
+  },
+}));
+
+// Import after mocking
+import NoCacheDataFetcher from "../../../src/utils/NoCacheDataFetcher";
+
+const TEST_URL = "http://example.com";
+
+describe("NoCacheDataFetcher", () => {
+  let fetcher: NoCacheDataFetcher;
+
+  beforeEach(() => {
+    // Clear mocks between tests
+    mockSuperFetch.mockClear();
+
+    // Create instance of class under test
+    fetcher = new NoCacheDataFetcher();
+  });
+
+  it("adds no-cache headers to fetch requests", async () => {
+    await fetcher.fetch(TEST_URL);
+
+    expect(mockSuperFetch).toHaveBeenCalledWith(
+      TEST_URL,
+      expect.objectContaining({
+        headers: { "Cache-Control": "no-cache" },
+        cache: "no-cache",
+      })
+    );
+  });
+
+  it("preserves existing headers when adding no-cache headers", async () => {
+    const options = {
+      headers: { "Content-Type": "application/json" },
+    };
+
+    await fetcher.fetch(TEST_URL, options);
+
+    expect(mockSuperFetch).toHaveBeenCalledWith(
+      TEST_URL,
+      expect.objectContaining({
+        headers: {
+          "Cache-Control": "no-cache",
+          "Content-Type": "application/json",
+        },
+        cache: "no-cache",
+      })
+    );
+  });
+
+  it("passes through other options to super.fetch", async () => {
+    await fetcher.fetch(TEST_URL, {
+      credentials: "include",
+    });
+
+    expect(mockSuperFetch).toHaveBeenCalledWith(
+      TEST_URL,
+      expect.objectContaining({
+        headers: { "Cache-Control": "no-cache" },
+        cache: "no-cache",
+        credentials: "include",
+      })
+    );
+  });
+});