@truedat/se 7.5.9 → 7.5.11

package/src/components/__tests__/SearchInput.spec.js

@@ -1,25 +1,20 @@
-import React from "react";
-import { shallow } from "enzyme";
-import { intl } from "@truedat/test/intl-stub";
+import { render, waitForLoad } from "@truedat/test/render";
 import { SearchInput } from "../SearchInput";
 
-// workaround for enzyme issue with React.useContext
-// see https://github.com/airbnb/enzyme/issues/2176#issuecomment-532361526
-jest.spyOn(React, "useContext").mockImplementation(() => intl);
-
 describe("<SearchInput />", () => {
   const query = "My Q";
   const fetchSearch = jest.fn();
   const loading = false;
 
-  it("it matches the last snapshot", () => {
+  it("it matches the last snapshot", async () => {
     const props = {
       query,
       fetchSearch,
-      loading
+      loading,
     };
-    const wrapper = shallow(<SearchInput {...props} />);
-    expect(wrapper).toMatchSnapshot();
-    expect(wrapper.find("Input").length).toBe(1);
+    const rendered = render(<SearchInput {...props} />);
+    await waitForLoad(rendered);
+    expect(rendered.container).toMatchSnapshot();
+    expect(rendered.container.querySelector("input")).toBeInTheDocument();
   });
 });

package/src/components/__tests__/SearchLoader.spec.js

@@ -1,86 +1,78 @@
-import React from "react";
-import { shallow } from "enzyme";
-import { SearchLoader } from "../SearchLoader";
+import { render, waitForLoad } from "@truedat/test/render";
+import SearchLoader from "../SearchLoader";
+import { clearSearch, fetchSearch, updateSearchPath } from "../../routines";
 
 describe("<SearchLoader />", () => {
-  const searchLoading = false;
-  const fetchSearch = jest.fn();
-  const clearSearch = jest.fn();
-  const updateSearchPath = jest.fn();
-  const match = { path: "my_path" };
+  beforeEach(() => {
+    jest.clearAllMocks();
+  });
+
+  it("it matches the last snapshot", async () => {
+    const mockDispatch = jest.fn();
+    const rendered = render(<SearchLoader />, {
+      state: { searchLoading: true },
+      dispatch: mockDispatch,
+    });
 
-  it("it matches the last snapshot", () => {
-    const searchLoading = true;
-    const props = {
-      clearSearch,
-      fetchSearch,
-      updateSearchPath,
-      match,
-      searchLoading
-    };
-    const wrapper = shallow(<SearchLoader {...props} />);
-    expect(wrapper).toMatchSnapshot();
+    expect(rendered.container).toMatchSnapshot();
   });
 
-  it("renders a loader if searchLoading is true", () => {
-    const searchLoading = true;
-    const props = {
-      clearSearch,
-      fetchSearch,
-      updateSearchPath,
-      match,
-      searchLoading
-    };
-    const wrapper = shallow(<SearchLoader {...props} />);
-    expect(wrapper.find("Loading").length).toBe(1);
+  it("renders a loader if searchLoading is true", async () => {
+    const mockDispatch = jest.fn();
+    const rendered = render(<SearchLoader />, {
+      state: { searchLoading: true },
+      dispatch: mockDispatch,
+    });
+    expect(rendered.container.querySelector(".loader")).toBeInTheDocument();
   });
 
-  it("renders null if searchLoading is false", () => {
-    const props = {
-      clearSearch,
-      fetchSearch,
-      updateSearchPath,
-      match,
-      searchLoading
-    };
-    const wrapper = shallow(<SearchLoader {...props} />);
-    expect(wrapper.getElement()).toBeNull();
+  it("renders null if searchLoading is false", async () => {
+    const mockDispatch = jest.fn();
+    const rendered = render(<SearchLoader />, {
+      state: { searchLoading: false },
+      dispatch: mockDispatch,
+    });
+
+    await waitForLoad(rendered);
+    expect(rendered.container.firstChild).toBeNull();
   });
 
-  it("calls fetchSearch when component mounts but not when it unmounts", () => {
-    const fetchSearch = jest.fn();
-    const updateSearchPath = jest.fn();
+  it("calls fetchSearch when component mounts but not when it unmounts", async () => {
+    const mockDispatch = jest.fn();
+    const rendered = render(<SearchLoader />, {
+      state: { searchLoading: false },
+      dispatch: mockDispatch,
+    });
+
+    await waitForLoad(rendered);
+
+    // Verify updateSearchPath and fetchSearch were called on mount
+    expect(mockDispatch).toHaveBeenCalledWith(
+      updateSearchPath(expect.objectContaining({ path: expect.any(String) }))
+    );
+    expect(mockDispatch).toHaveBeenCalledWith(fetchSearch({}));
 
-    const props = {
-      clearSearch,
-      fetchSearch,
-      updateSearchPath,
-      match,
-      searchLoading
-    };
-    jest.spyOn(SearchLoader.prototype, "componentDidMount");
-    const wrapper = shallow(<SearchLoader {...props} />);
-    expect(SearchLoader.prototype.componentDidMount.mock.calls.length).toBe(1);
-    expect(fetchSearch.mock.calls.length).toBe(1);
-    expect(updateSearchPath.mock.calls.length).toBe(1);
-    wrapper.unmount();
-    expect(fetchSearch.mock.calls.length).toBe(1);
-    expect(updateSearchPath.mock.calls.length).toBe(1);
+    const dispatchCallCount = mockDispatch.mock.calls.length;
+    rendered.unmount();
+
+    // Verify no additional calls to fetchSearch or updateSearchPath
+    expect(mockDispatch.mock.calls.length).toBe(dispatchCallCount + 1); // +1 for clearSearch
   });
 
-  it("calls clearSearch when component mounts but not when it unmounts", () => {
-    const clearSearch = jest.fn();
-    const props = {
-      clearSearch,
-      fetchSearch,
-      updateSearchPath,
-      match,
-      searchLoading
-    };
-    jest.spyOn(SearchLoader.prototype, "componentWillUnmount");
-    const wrapper = shallow(<SearchLoader {...props} />);
-    expect(clearSearch.mock.calls.length).toBe(0);
-    wrapper.unmount();
-    expect(clearSearch.mock.calls.length).toBe(1);
+  it("calls clearSearch when component unmounts", async () => {
+    const mockDispatch = jest.fn();
+    const rendered = render(<SearchLoader />, {
+      state: { searchLoading: false },
+      dispatch: mockDispatch,
+    });
+
+    await waitForLoad(rendered);
+
+    mockDispatch.mockClear(); // Clear previous calls
+    rendered.unmount();
+
+    // Verify clearSearch was called on unmount
+    expect(mockDispatch).toHaveBeenCalledTimes(1);
+    expect(mockDispatch).toHaveBeenCalledWith(clearSearch());
   });
 });

package/src/components/__tests__/SearchPagination.spec.js

@@ -1,4 +1,3 @@
-import React from "react";
 import { render } from "@truedat/test/render";
 import SearchPagination from "../SearchPagination";
 

package/src/components/__tests__/SearchResults.spec.js

@@ -1,4 +1,3 @@
-import React from "react";
 import { render } from "@truedat/test/render";
 import { SearchResults } from "../SearchResults";
 

package/src/components/__tests__/SearchRoutes.spec.js

@@ -1,14 +1,114 @@
-import React from "react";
-import { shallow } from "enzyme";
+import { render, waitForLoad } from "@truedat/test/render";
+import { useAuthorized } from "@truedat/core/hooks/useAuthorized";
+import {
+  SEARCH,
+  SEARCH_CONCEPTS,
+  SEARCH_INGESTS,
+  SEARCH_RESULTS,
+  SEARCH_STRUCTURES,
+  ELASTICINDEXES,
+} from "@truedat/core/routes";
 import { SearchRoutes } from "../SearchRoutes";
 
-jest.mock("@truedat/core/hooks", () => ({
-  useAuthorized: jest.fn(() => true),
+// Mock the hooks module
+jest.mock("@truedat/core/hooks/useAuthorized");
+
+// Mock child components
+jest.mock(
+  "../SearchLoader",
+  () =>
+    function SearchLoader() {
+      return <div>SearchLoader</div>;
+    }
+);
+jest.mock(
+  "../SearchHome",
+  () =>
+    function SearchHome() {
+      return <div>SearchHome</div>;
+    }
+);
+jest.mock(
+  "../Search",
+  () =>
+    function Search() {
+      return <div>Search</div>;
+    }
+);
+jest.mock(
+  "../ElasticIndexes",
+  () =>
+    function ElasticIndexes() {
+      return <div>ElasticIndexes</div>;
+    }
+);
+jest.mock("@truedat/core/components", () => ({
+  Unauthorized: () => <div>Unauthorized</div>,
+  ...jest.requireActual("@truedat/core/components"),
 }));
 
 describe("<SearchRoutes />", () => {
-  it("matches the latest snapshot", () => {
-    const wrapper = shallow(<SearchRoutes />);
-    expect(wrapper).toMatchSnapshot();
+  it("renders correctly with default route", async () => {
+    const rendered = render(<SearchRoutes />);
+    await waitForLoad(rendered);
+    expect(rendered.container).toMatchSnapshot();
+  });
+
+  it("renders correctly with SEARCH route", async () => {
+    const rendered = render(<SearchRoutes />, {
+      routes: [SEARCH],
+    });
+    await waitForLoad(rendered);
+    expect(rendered.container).toMatchSnapshot();
+  });
+
+  it("renders correctly with SEARCH_RESULTS route", async () => {
+    const rendered = render(<SearchRoutes />, {
+      routes: [SEARCH_RESULTS],
+    });
+    await waitForLoad(rendered);
+    expect(rendered.container).toMatchSnapshot();
+  });
+
+  it("renders correctly with SEARCH_CONCEPTS route", async () => {
+    const rendered = render(<SearchRoutes />, {
+      routes: [SEARCH_CONCEPTS],
+    });
+    await waitForLoad(rendered);
+    expect(rendered.container).toMatchSnapshot();
+  });
+
+  it("renders correctly with SEARCH_INGESTS route", async () => {
+    const rendered = render(<SearchRoutes />, {
+      routes: [SEARCH_INGESTS],
+    });
+    await waitForLoad(rendered);
+    expect(rendered.container).toMatchSnapshot();
+  });
+
+  it("renders correctly with SEARCH_STRUCTURES route", async () => {
+    const rendered = render(<SearchRoutes />, {
+      routes: [SEARCH_STRUCTURES],
+    });
+    await waitForLoad(rendered);
+    expect(rendered.container).toMatchSnapshot();
+  });
+
+  it("renders correctly with ELASTICINDEXES route", async () => {
+    const rendered = render(<SearchRoutes />, {
+      routes: [ELASTICINDEXES],
+    });
+    await waitForLoad(rendered);
+    expect(rendered.container).toMatchSnapshot();
+  });
+
+  it("renders unauthorized component when not authorized", async () => {
+    useAuthorized.mockReturnValueOnce(false);
+
+    const rendered = render(<SearchRoutes />, {
+      routes: [ELASTICINDEXES],
+    });
+    await waitForLoad(rendered);
+    expect(rendered.container).toMatchSnapshot();
   });
 });

package/src/components/__tests__/SearchTabs.spec.js

@@ -1,11 +1,12 @@
-import React from "react";
 import { render } from "@truedat/test/render";
 import { waitFor } from "@testing-library/react";
 import { SearchTabs } from "../SearchTabs";
 
 jest.mock("@truedat/core/hooks", () => ({
   useAuthorized: jest.fn(() => true),
-  useActiveRoute: jest.fn(param => (param === "/search/results" ? true : false))
+  useActiveRoute: jest.fn((param) =>
+    param === "/search/results" ? true : false
+  ),
 }));
 
 describe("<SearchTabs />", () => {
@@ -16,9 +17,9 @@ describe("<SearchTabs />", () => {
         "tabs.se.all": "All",
         "tabs.se.concepts": "Concepts",
         "tabs.se.structures": "Structures",
-        "tabs.se.ingests": "Ingests"
-      }
-    }
+        "tabs.se.ingests": "Ingests",
+      },
+    },
   };
 
   it("matches the latest snapshot", async () => {

package/src/components/__tests__/SearchTabsHide.spec.js

@@ -1,40 +1,24 @@
-import React from "react";
-import { render } from "@truedat/test/render";
-import { waitFor } from "@testing-library/react";
+import { render, waitForLoad } from "@truedat/test/render";
 import { SearchTabs } from "../SearchTabs";
 
 jest.mock("@truedat/core/hooks", () => ({
   useAuthorized: jest.fn(
-    params =>
+    (params) =>
       params.includes("all") || params.includes("business_glossary_view")
   ),
-  useActiveRoute: jest.fn(params => params === "/search/results")
+  useActiveRoute: jest.fn((params) => params === "/search/results"),
 }));
 
 describe("<SearchTabs />", () => {
   const props = { indices: ["concepts", "ingests", "structures"] };
-  const renderOpts = {
-    messages: {
-      en: {
-        "tabs.se.all": "All",
-        "tabs.se.concepts": "Concepts",
-        "tabs.se.structures": "Structures",
-        "tabs.se.ingests": "Ingests"
-      }
-    }
-  };
 
   it("show all and Concepts tabs, structures and Ingest tabs are removed", async () => {
-    const { getByText, findAllByText } = render(
-      <SearchTabs {...props} />,
-      renderOpts
-    );
+    const rendered = render(<SearchTabs {...props} />);
+    await waitForLoad(rendered);
 
-    await waitFor(
-      () => expect(getByText(/All/)).not.toBeNull(),
-      expect(getByText(/Concepts/)).not.toBeNull(),
-      expect(findAllByText(/Structures/)).toMatchObject({}),
-      expect(findAllByText(/Ingests/)).toMatchObject({})
-    );
+    expect(rendered.getByText(/all/i)).toBeInTheDocument();
+    expect(rendered.getByText(/concepts/i)).toBeInTheDocument();
+    expect(rendered.queryByText(/structures/i)).not.toBeInTheDocument();
+    expect(rendered.queryByText(/ingests/i)).not.toBeInTheDocument();
   });
 });

package/src/components/__tests__/__snapshots__/CardResult.spec.js.snap

@@ -1,28 +1,35 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
 exports[`<CardResult /> it matches the last snapshot 1`] = `
-<Card
-  className="search result"
-  description="My description"
-  header={
-    <div>
-      <Link
-        to="/foo"
+<div>
+  <div
+    class="ui card search result"
+  >
+    <div
+      class="content"
+    >
+      <div>
+        <a
+          data-discover="true"
+          href="/foo"
+        >
+          My header
+        </a>
+        <p>
+          business_concept.search.description
+        </p>
+      </div>
+      <div
+        class="meta"
       >
-        My header
-      </Link>
-      <p>
-        <Memo(MemoizedFormattedMessage)
-          id="business_concept.search.description"
-          values={
-            {
-              "header": "My header",
-            }
-          }
-        />
-      </p>
+        levelOne &gt; levelTwo &gt; levelThree
+      </div>
+      <div
+        class="description"
+      >
+        My description
+      </div>
     </div>
-  }
-  meta="levelOne > levelTwo > levelThree"
-/>
+  </div>
+</div>
 `;

package/src/components/__tests__/__snapshots__/ElasticIndexes.spec.js.snap

@@ -1,6 +1,6 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
-exports[`<ElasticIndexes /> matches the last snapshot 1`] = `
+exports[`<ElasticIndexes /> matches the latest snapshot 1`] = `
 <div>
   <div
     class="ui segment"
@@ -26,17 +26,6 @@ exports[`<ElasticIndexes /> matches the last snapshot 1`] = `
     <div
       class="dimmable"
     >
-      <div
-        class="ui inverted dimmer"
-      >
-        <div
-          class="content"
-        >
-          <div
-            class="ui loader"
-          />
-        </div>
-      </div>
       <table
         class="ui table"
       >

package/src/components/__tests__/__snapshots__/Search.spec.js.snap

@@ -9,9 +9,8 @@ exports[`<Search /> it matches the last snapshot 1`] = `
       class="ui icon input custom"
     >
       <input
-        placeholder="Search"
+        placeholder="search.input.placeholder"
         type="text"
-        value=""
       />
       <i
         aria-hidden="true"
@@ -23,9 +22,10 @@ exports[`<Search /> it matches the last snapshot 1`] = `
     >
       <a
         class="item"
+        data-discover="true"
         href="/search/results"
       >
-        All
+        tabs.se.all
       </a>
     </div>
     <div
@@ -41,13 +41,14 @@ exports[`<Search /> it matches the last snapshot 1`] = `
         <div
           class="header"
         >
-          No results found
+          search.not_found.header
         </div>
         <a
           class="link pointer"
+          data-discover="true"
           href="/search"
         >
-          Try another search
+          search.not_found.body
         </a>
       </div>
     </div>

package/src/components/__tests__/__snapshots__/SearchInput.spec.js.snap

@@ -1,18 +1,19 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
 exports[`<SearchInput /> it matches the last snapshot 1`] = `
-<Input
-  className="custom"
-  icon={
-    {
-      "link": true,
-      "name": "search",
-    }
-  }
-  loading={false}
-  onChange={[Function]}
-  placeholder="search.input.placeholder"
-  type="text"
-  value="My Q"
-/>
+<div>
+  <div
+    class="ui icon input custom"
+  >
+    <input
+      placeholder="search.input.placeholder"
+      type="text"
+      value="My Q"
+    />
+    <i
+      aria-hidden="true"
+      class="search link icon"
+    />
+  </div>
+</div>
 `;

package/src/components/__tests__/__snapshots__/SearchLoader.spec.js.snap

@@ -1,3 +1,9 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
-exports[`<SearchLoader /> it matches the last snapshot 1`] = `<Loading />`;
+exports[`<SearchLoader /> it matches the last snapshot 1`] = `
+<div>
+  <div
+    class="ui active loader"
+  />
+</div>
+`;

package/src/components/__tests__/__snapshots__/SearchResults.spec.js.snap

@@ -15,13 +15,14 @@ exports[`<SearchResults /> it matches the last snapshot 1`] = `
       <div
         class="header"
       >
-        No results found
+        search.not_found.header
       </div>
       <a
         class="link pointer"
+        data-discover="true"
         href="/search"
       >
-        Try another search
+        search.not_found.body
       </a>
     </div>
   </div>