From 2dba4e563ee76196f7143b26a9bca4a4617f1468 Mon Sep 17 00:00:00 2001
From: Tim DiLauro <tdilauro@users.noreply.github.com>
Date: Thu, 26 Sep 2024 20:21:13 -0400
Subject: [PATCH] Logo-only header for QuickSight page. (PP-1720) (#136)

* Add assett and style mocks to jest, so we can import our logo in tests.

* Add terms of service context to our providers test helper.

* Enable a logo-only header and use it for QuickSight dashboard.
---
 jest.config.js                                |   5 +
 src/components/Header.tsx                     | 153 ++++++++++--------
 src/components/QuicksightDashboardPage.tsx    |   2 +-
 src/components/TOSContext.tsx                 |   7 +-
 tests/__mocks__/fileMock.js                   |   1 +
 tests/__mocks__/styleMock.js                  |   1 +
 .../components/QuicksightDashboard.test.tsx   |  29 ++--
 tests/jest/testUtils/withProviders.tsx        |  22 ++-
 8 files changed, 139 insertions(+), 81 deletions(-)
 create mode 100644 tests/__mocks__/fileMock.js
 create mode 100644 tests/__mocks__/styleMock.js

diff --git a/jest.config.js b/jest.config.js
index 39fc0af80..8ed13af42 100644
--- a/jest.config.js
+++ b/jest.config.js
@@ -1,5 +1,10 @@
 /** @type {import('ts-jest').JestConfigWithTsJest} */
 module.exports = {
+  moduleNameMapper: {
+    "\\.(jpg|jpeg|png|gif|eot|otf|webp|svg|ttf|woff|woff2|mp4|webm|wav|mp3|m4a|aac|oga)$":
+      "<rootDir>/tests/__mocks__/fileMock.js",
+    "\\.(css|less)$": "<rootDir>/tests/__mocks__/styleMock.js",
+  },
   preset: "ts-jest",
   testEnvironment: "jsdom",
   testEnvironmentOptions: {
diff --git a/src/components/Header.tsx b/src/components/Header.tsx
index b5d14d3df..5f5a6ae68 100644
--- a/src/components/Header.tsx
+++ b/src/components/Header.tsx
@@ -29,6 +29,7 @@ export interface HeaderDispatchProps {
 
 export interface HeaderOwnProps {
   store?: Store<RootState>;
+  logoOnly?: boolean;
 }
 
 export interface HeaderProps
@@ -130,83 +131,90 @@ export class Header extends React.Component<HeaderProps, HeaderState> {
       },
     ];
     const accountLink = { label: "Change password", href: "account/" };
+    const logoOnly = this.props.logoOnly ?? false;
 
     return (
       <Navbar fluid={true}>
         <Navbar.Header>
           <img src={palaceLogoUrl} alt={title()} />
-          {this.props.libraries && this.props.libraries.length > 0 && (
-            <EditableInput
-              elementType="select"
-              ref={this.libraryRef}
-              value={currentLibrary}
-              onChange={this.changeLibrary}
-              aria-label="Select a library"
-            >
-              {(!this.context.library || !currentLibrary) && (
-                <option aria-selected={false}>Select a library</option>
-              )}
-              {this.props.libraries.map((library) => (
-                <option
-                  key={library.short_name}
-                  value={library.short_name}
-                  aria-selected={currentLibrary === library.short_name}
+          {!logoOnly && (
+            <>
+              {this.props.libraries && this.props.libraries.length > 0 && (
+                <EditableInput
+                  elementType="select"
+                  ref={this.libraryRef}
+                  value={currentLibrary}
+                  onChange={this.changeLibrary}
+                  aria-label="Select a library"
                 >
-                  {library.name || library.short_name}
-                </option>
-              ))}
-            </EditableInput>
+                  {(!this.context.library || !currentLibrary) && (
+                    <option aria-selected={false}>Select a library</option>
+                  )}
+                  {this.props.libraries.map((library) => (
+                    <option
+                      key={library.short_name}
+                      value={library.short_name}
+                      aria-selected={currentLibrary === library.short_name}
+                    >
+                      {library.name || library.short_name}
+                    </option>
+                  ))}
+                </EditableInput>
+              )}
+              <Navbar.Toggle />
+            </>
           )}
-          <Navbar.Toggle />
         </Navbar.Header>
 
-        <Navbar.Collapse className="menu">
-          {currentLibrary && (
-            <Nav>
-              {this.renderLinkItem(
-                dashboardLinkItem,
-                currentPathname,
-                currentLibrary
-              )}
-              {libraryNavItems.map((item) =>
-                this.renderNavItem(item, currentPathname, currentLibrary)
+        {!logoOnly && (
+          <Navbar.Collapse className="menu">
+            {currentLibrary && (
+              <Nav>
+                {this.renderLinkItem(
+                  dashboardLinkItem,
+                  currentPathname,
+                  currentLibrary
+                )}
+                {libraryNavItems.map((item) =>
+                  this.renderNavItem(item, currentPathname, currentLibrary)
+                )}
+                {libraryLinkItems.map((item) =>
+                  this.renderLinkItem(item, currentPathname, currentLibrary)
+                )}
+              </Nav>
+            )}
+            <Nav className="pull-right">
+              {sitewideLinkItems.map((item) =>
+                this.renderLinkItem(item, currentPathname)
               )}
-              {libraryLinkItems.map((item) =>
-                this.renderLinkItem(item, currentPathname, currentLibrary)
+              {this.context.admin.email && (
+                <li className="dropdown">
+                  <Button
+                    className="account-dropdown-toggle transparent"
+                    type="button"
+                    aria-haspopup="true"
+                    aria-expanded={this.state.showAccountDropdown}
+                    callback={this.toggleAccountDropdown}
+                    content={
+                      <span>
+                        {this.context.admin.email} <GenericWedgeIcon />
+                      </span>
+                    }
+                  />
+                  {this.state.showAccountDropdown && (
+                    <ul className="dropdown-menu">
+                      {this.displayPermissions(isSystemAdmin, isLibraryManager)}
+                      {this.renderLinkItem(accountLink, currentPathname)}
+                      <li>
+                        <a href="/admin/sign_out">Sign out</a>
+                      </li>
+                    </ul>
+                  )}
+                </li>
               )}
             </Nav>
-          )}
-          <Nav className="pull-right">
-            {sitewideLinkItems.map((item) =>
-              this.renderLinkItem(item, currentPathname)
-            )}
-            {this.context.admin.email && (
-              <li className="dropdown">
-                <Button
-                  className="account-dropdown-toggle transparent"
-                  type="button"
-                  aria-haspopup="true"
-                  aria-expanded={this.state.showAccountDropdown}
-                  callback={this.toggleAccountDropdown}
-                  content={
-                    <span>
-                      {this.context.admin.email} <GenericWedgeIcon />
-                    </span>
-                  }
-                />
-                {this.state.showAccountDropdown && (
-                  <ul className="dropdown-menu">
-                    {this.displayPermissions(isSystemAdmin, isLibraryManager)}
-                    {this.renderLinkItem(accountLink, currentPathname)}
-                    <li>
-                      <a href="/admin/sign_out">Sign out</a>
-                    </li>
-                  </ul>
-                )}
-              </li>
-            )}
-          </Nav>
-        </Navbar.Collapse>
+          </Navbar.Collapse>
+        )}
       </Navbar>
     );
   }
@@ -331,7 +339,13 @@ const ConnectedHeader = connect<
 
 /** HeaderWithStore is a wrapper component to pass the store as a prop to the
     ConnectedHeader, since it's not in the regular place in the context. */
-export default class HeaderWithStore extends React.Component<{}, {}> {
+type HeaderWithStoreProps = {
+  logoOnly?: boolean;
+};
+
+export default class HeaderWithStore extends React.Component<
+  HeaderWithStoreProps
+> {
   context: { editorStore: Store<RootState> };
 
   static contextTypes = {
@@ -339,6 +353,11 @@ export default class HeaderWithStore extends React.Component<{}, {}> {
   };
 
   render(): JSX.Element {
-    return <ConnectedHeader store={this.context.editorStore} />;
+    return (
+      <ConnectedHeader
+        store={this.context.editorStore}
+        logoOnly={this.props.logoOnly}
+      />
+    );
   }
 }
diff --git a/src/components/QuicksightDashboardPage.tsx b/src/components/QuicksightDashboardPage.tsx
index b36c37f5d..cb7c470a5 100644
--- a/src/components/QuicksightDashboardPage.tsx
+++ b/src/components/QuicksightDashboardPage.tsx
@@ -42,7 +42,7 @@ export default class QuicksightDashboardPage extends React.Component<
     const { library } = this.props.params;
     return (
       <div className="quicksight-dashboard">
-        <Header />
+        <Header logoOnly={true} />
         <main className="body">
           <QuicksightDashboard dashboardId="library" />
         </main>
diff --git a/src/components/TOSContext.tsx b/src/components/TOSContext.tsx
index d553ad668..22166e73a 100644
--- a/src/components/TOSContext.tsx
+++ b/src/components/TOSContext.tsx
@@ -1,3 +1,8 @@
 import * as React from "react";
-export const TOSContext = React.createContext(null);
+
+export type TOSContextProviderProps = {
+  [key: number]: string;
+};
+
+export const TOSContext = React.createContext<TOSContextProviderProps>(null);
 export const TOSContextProvider = TOSContext.Provider;
diff --git a/tests/__mocks__/fileMock.js b/tests/__mocks__/fileMock.js
new file mode 100644
index 000000000..0a445d060
--- /dev/null
+++ b/tests/__mocks__/fileMock.js
@@ -0,0 +1 @@
+module.exports = "test-file-stub";
diff --git a/tests/__mocks__/styleMock.js b/tests/__mocks__/styleMock.js
new file mode 100644
index 000000000..f053ebf79
--- /dev/null
+++ b/tests/__mocks__/styleMock.js
@@ -0,0 +1 @@
+module.exports = {};
diff --git a/tests/jest/components/QuicksightDashboard.test.tsx b/tests/jest/components/QuicksightDashboard.test.tsx
index 1e4cd17d6..926a24790 100644
--- a/tests/jest/components/QuicksightDashboard.test.tsx
+++ b/tests/jest/components/QuicksightDashboard.test.tsx
@@ -7,6 +7,8 @@ import buildStore from "../../../src/store";
 import { setupServer } from "msw/node";
 import { http, HttpResponse } from "msw";
 import renderWithContext from "../testUtils/renderWithContext";
+import { renderWithProviders } from "../testUtils/withProviders";
+import QuicksightDashboardPage from "../../../src/components/QuicksightDashboardPage";
 
 const libraries: LibrariesData = { libraries: [{ uuid: "my-uuid" }] };
 const dashboardId = "test";
@@ -35,15 +37,8 @@ describe("QuicksightDashboard", () => {
   });
 
   it("embed url is retrieved and set in iframe", async () => {
-    const contextProviderProps = {
-      csrfToken: "",
-      featureFlags: {},
-      roles: [{ role: "system" }],
-    };
-
-    renderWithContext(
-      <QuicksightDashboard dashboardId={dashboardId} store={buildStore()} />,
-      contextProviderProps
+    renderWithProviders(
+      <QuicksightDashboard dashboardId={dashboardId} store={buildStore()} />
     );
 
     await waitFor(() => {
@@ -53,4 +48,20 @@ describe("QuicksightDashboard", () => {
       );
     });
   });
+
+  it("header renders without navigation links ", () => {
+    renderWithProviders(<QuicksightDashboardPage params={{ library: null }} />);
+
+    // Make sure we see the QuicksSight iFrame.
+    expect(screen.getByTitle("Library Dashboard")).toBeInTheDocument();
+    // Make sure we have the branding image.
+    expect(
+      screen.getByAltText("Palace Collection Manager")
+    ).toBeInTheDocument();
+
+    // Make sure we do not see other navigation links.
+    ["Dashboard", "System Configuration"].forEach((name) => {
+      expect(screen.queryByText(name)).not.toBeInTheDocument();
+    });
+  });
 });
diff --git a/tests/jest/testUtils/withProviders.tsx b/tests/jest/testUtils/withProviders.tsx
index 3e7a153f5..5a5ef928c 100644
--- a/tests/jest/testUtils/withProviders.tsx
+++ b/tests/jest/testUtils/withProviders.tsx
@@ -7,10 +7,15 @@ 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 & {
@@ -21,6 +26,13 @@ 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 = {
@@ -35,6 +47,7 @@ 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
  */
@@ -46,6 +59,7 @@ export const componentWithProviders = ({
     csrfToken: "",
     featureFlags: defaultFeatureFlags,
   },
+  tosContextProviderProps = requiredTOSContextProviderProps,
   queryClient = new QueryClient(),
 }: TestProviderWrapperOptions = {}): React.FunctionComponent => {
   const effectiveContextProviderProps = {
@@ -56,9 +70,11 @@ export const componentWithProviders = ({
   const wrapper = ({ children }) => (
     <Provider {...reduxProviderProps}>
       <ContextProvider {...effectiveContextProviderProps}>
-        <QueryClientProvider client={queryClient}>
-          {children}
-        </QueryClientProvider>
+        <TOSContextProvider value={tosContextProviderProps}>
+          <QueryClientProvider client={queryClient}>
+            {children}
+          </QueryClientProvider>
+        </TOSContextProvider>
       </ContextProvider>
     </Provider>
   );