@truedat/core 8.1.1 → 8.1.4

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

@@ -0,0 +1,103 @@
+import { render } from "@truedat/test/render";
+import { StatusPill, ResponseCell } from "../UploadJobParser";
+
+describe("<StatusPill />", () => {
+  it("renders with correct color based on status", async () => {
+    const statuses = ["COMPLETED", "FAILED", "ERROR", "INFO", "OTHER"];
+    const expectedColors = ["green", "red", "red", "blue", "grey"];
+
+    statuses.forEach((status, index) => {
+      const rendered = render(<StatusPill status={status} />);
+      const label = rendered.container.querySelector(".ui.label");
+      expect(label).toHaveClass(expectedColors[index]);
+      expect(
+        rendered.getByText(
+          new RegExp(`uploadJob.parser.event.status.${status}`, "i")
+        )
+      ).toBeInTheDocument();
+    });
+  });
+});
+
+describe("<ResponseCell />", () => {
+  it("returns null when response is empty", () => {
+    const rendered = render(
+      <ResponseCell response={null} status="COMPLETED" />
+    );
+    expect(rendered.container.firstChild).toBeNull();
+
+    const rendered2 = render(<ResponseCell response={{}} status="COMPLETED" />);
+    expect(rendered2.container.firstChild).toBeNull();
+  });
+
+  it("renders FAILED status with message", () => {
+    const response = { message: "missing_required_headers" };
+    const rendered = render(
+      <ResponseCell response={response} status="FAILED" />
+    );
+    expect(
+      rendered.getByText(/missing_required_headers/i)
+    ).toBeInTheDocument();
+  });
+
+  it("renders ERROR status with details", () => {
+    const response = {
+      type: "missing_required_headers",
+      details: {
+        missing_headers: ["header1", "header2"],
+      },
+    };
+    const rendered = render(
+      <ResponseCell response={response} status="ERROR" />
+    );
+    expect(
+      rendered.getByText(
+        /uploadJob.parser.error.missing_required_headers/i
+      )
+    ).toBeInTheDocument();
+    expect(rendered.getByText(/header1, header2/i)).toBeInTheDocument();
+  });
+
+  it("renders COMPLETED status with summary", () => {
+    const response = {
+      insert_count: 2,
+      update_count: 1,
+      error_count: 0,
+    };
+    const rendered = render(
+      <ResponseCell response={response} status="COMPLETED" />
+    );
+    expect(
+      rendered.getByText(/uploadJob.parser.result.summary.created/i)
+    ).toBeInTheDocument();
+    expect(
+      rendered.getByText(/uploadJob.parser.result.summary.updated/i)
+    ).toBeInTheDocument();
+  });
+
+  it("renders INFO status with implementation details", () => {
+    const response = {
+      type: "implementation_updated",
+      details: {
+        id: 123,
+        changes: {
+          name: "new name",
+        },
+      },
+    };
+    const rendered = render(<ResponseCell response={response} status="INFO" />);
+    expect(
+      rendered.getByText(
+        /uploadJob.parser.info.implementation_updated/i
+      )
+    ).toBeInTheDocument();
+  });
+
+  it("renders default case for unknown status", () => {
+    const response = "some text";
+    const rendered = render(
+      <ResponseCell response={response} status="UNKNOWN" />
+    );
+    expect(rendered.getByText(/some text/i)).toBeInTheDocument();
+  });
+});

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

@@ -0,0 +1,60 @@
+import { render, waitForLoad } from "@truedat/test/render";
+import { useUploadJobs } from "../../hooks/useUploadJobs";
+import UploadJobs from "../UploadJobs";
+
+jest.mock("../../hooks/useUploadJobs", () => ({
+  useUploadJobs: jest.fn(),
+}));
+
+describe("<UploadJobs />", () => {
+  it("renders loading state", async () => {
+    useUploadJobs.mockReturnValue({
+      loading: true,
+      data: null,
+    });
+
+    const rendered = render(<UploadJobs />);
+
+    expect(rendered.container.querySelector(".ui.loader")).toBeInTheDocument();
+  });
+
+  it("renders empty state", async () => {
+    useUploadJobs.mockReturnValue({
+      loading: false,
+      data: {
+        data: [],
+      },
+    });
+
+    const rendered = render(<UploadJobs />);
+    await waitForLoad(rendered);
+
+    expect(rendered.getByText(/No jobs found./i)).toBeInTheDocument();
+  });
+
+  it("renders jobs list", async () => {
+    const mockJobs = [
+      {
+        id: 1,
+        filename: "test.csv",
+        latest_status: "completed",
+        latest_event_response: "Success",
+        latest_event_at: "2023-01-01T00:00:00Z",
+      },
+    ];
+
+    useUploadJobs.mockReturnValue({
+      loading: false,
+      data: {
+        data: mockJobs,
+      },
+    });
+
+    const rendered = render(<UploadJobs />);
+    await waitForLoad(rendered);
+
+    expect(rendered.getByText(/test.csv/i)).toBeInTheDocument();
+    expect(rendered.getByText(/completed/i)).toBeInTheDocument();
+    expect(rendered.getByText(/success/i)).toBeInTheDocument();
+  });
+});

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

@@ -32,6 +32,7 @@ exports[`<AdminMenu /> matches the latest snapshot 1`] = `
           class="divider"
         />
         <a
+          aria-checked="false"
           class="item"
           data-discover="true"
           href="/templates"
@@ -45,6 +46,7 @@ exports[`<AdminMenu /> matches the latest snapshot 1`] = `
           </span>
         </a>
         <a
+          aria-checked="false"
           class="item"
           data-discover="true"
           href="/hierarchies"
@@ -58,6 +60,7 @@ exports[`<AdminMenu /> matches the latest snapshot 1`] = `
           </span>
         </a>
         <a
+          aria-checked="false"
           class="item"
           data-discover="true"
           href="/relationTags"
@@ -71,6 +74,7 @@ exports[`<AdminMenu /> matches the latest snapshot 1`] = `
           </span>
         </a>
         <a
+          aria-checked="false"
           class="item"
           data-discover="true"
           href="/subscriptions"
@@ -84,6 +88,7 @@ exports[`<AdminMenu /> matches the latest snapshot 1`] = `
           </span>
         </a>
         <a
+          aria-checked="false"
           class="item"
           data-discover="true"
           href="/sources"
@@ -97,6 +102,7 @@ exports[`<AdminMenu /> matches the latest snapshot 1`] = `
           </span>
         </a>
         <a
+          aria-checked="false"
           class="item"
           data-discover="true"
           href="/jobs"
@@ -110,6 +116,7 @@ exports[`<AdminMenu /> matches the latest snapshot 1`] = `
           </span>
         </a>
         <a
+          aria-checked="false"
           class="item"
           data-discover="true"
           href="/configurations"
@@ -123,6 +130,7 @@ exports[`<AdminMenu /> matches the latest snapshot 1`] = `
           </span>
         </a>
         <a
+          aria-checked="false"
           class="item"
           data-discover="true"
           href="/i18n/messages"
@@ -136,6 +144,7 @@ exports[`<AdminMenu /> matches the latest snapshot 1`] = `
           </span>
         </a>
         <a
+          aria-checked="false"
           class="item"
           data-discover="true"
           href="/reindex"
@@ -149,6 +158,7 @@ exports[`<AdminMenu /> matches the latest snapshot 1`] = `
           </span>
         </a>
         <a
+          aria-checked="false"
           class="item"
           data-discover="true"
           href="/search/elasticIndexes"

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

@@ -56,7 +56,7 @@ exports[`<CatalogMenu /> matches the latest snapshot 1`] = `
       <a
         class="link item"
         data-discover="true"
-        href="/bulkUpdateTemplateContentEvents"
+        href="/structureNotes/uploadJobs"
       >
         structures_upload_events
       </a>

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

@@ -192,7 +192,7 @@ exports[`<SideMenu /> matches the latest snapshot 1`] = `
           <a
             class="link item"
             data-discover="true"
-            href="/bulkUpdateTemplateContentEvents"
+            href="/structureNotes/uploadJobs"
           >
             structures_upload_events
           </a>

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

@@ -36,3 +36,40 @@ exports[`<Submenu /> matches the latest snapshot 1`] = `
   </div>
 </div>
 `;
+
+exports[`<Submenu /> should handle subscope path detection correctly 1`] = `
+<div>
+  <div
+    class="active item selectable"
+  >
+    <a
+      data-discover="true"
+      href="/implementations"
+    >
+      <i
+        aria-hidden="true"
+        class="large icon"
+      />
+      quality
+    </a>
+    <div
+      class="menu"
+    >
+      <a
+        class="link item"
+        data-discover="true"
+        href="/implementations"
+      >
+        implementations
+      </a>
+      <a
+        class="active link item"
+        data-discover="true"
+        href="/implementations/subscope/subscope1"
+      >
+        subscope1
+      </a>
+    </div>
+  </div>
+</div>
+`;

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

@@ -0,0 +1,42 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`ImplementationUploadJobBreadcrumbs renders breadcrumbs with filename 1`] = `
+<div>
+  <div
+    class="ui breadcrumb"
+  >
+    <a
+      class="section"
+      data-discover="true"
+      href="/implementations/uploadJobs"
+    >
+      uploadJobs.implementations.header
+    </a>
+    <i
+      aria-hidden="true"
+      class="right angle icon divider"
+    />
+    <div
+      class="active section"
+    >
+      test.csv
+    </div>
+  </div>
+</div>
+`;
+
+exports[`ImplementationUploadJobBreadcrumbs renders breadcrumbs without filename 1`] = `
+<div>
+  <div
+    class="ui breadcrumb"
+  >
+    <a
+      class="section"
+      data-discover="true"
+      href="/implementations/uploadJobs"
+    >
+      uploadJobs.implementations.header
+    </a>
+  </div>
+</div>
+`;

package/src/hooks/__tests__/useActiveRoutes.spec.js

@@ -0,0 +1,83 @@
+import { useActiveRoutes } from "../useActiveRoutes";
+
+// Mock the react-router and lodash functions that are used in the hook
+jest.mock("react-router", () => ({
+  useLocation: () => ({ pathname: "/test" }),
+}));
+
+jest.mock("lodash/fp", () => ({
+  prop: jest.fn((key) => (obj) => obj[key]),
+  orderBy: jest.fn((iteratees, orders, collection) => collection.sort()),
+  castArray: jest.fn((val) => Array.isArray(val) ? val : [val]),
+  size: jest.fn((val) => typeof val === 'string' ? val.length : 0),
+  flow: jest.fn((...funcs) => (...args) => funcs.reduceRight((arg, fn) => fn(arg), args)),
+  map: jest.fn((fn) => (arr) => arr.map(fn)),
+  head: jest.fn((arr) => arr[0]),
+}));
+
+jest.mock("@truedat/core/routes", () => ({
+  BUCKETS_VIEW: "/buckets/:propertyPath",
+}));
+
+jest.mock("react-router", () => ({
+  matchPath: jest.fn(() => null),
+}));
+
+describe("hooks: useActiveRoutes", () => {
+  beforeEach(() => {
+    jest.clearAllMocks();
+    
+    // Setup default mocks
+    require("lodash/fp").prop.mockImplementation((key) => (obj) => obj[key]);
+    require("lodash/fp").orderBy.mockImplementation((iteratees, orders, collection) => collection);
+    require("lodash/fp").castArray.mockImplementation((val) => Array.isArray(val) ? val : [val]);
+    require("react-router").useLocation = () => ({ pathname: "/test" });
+  });
+
+  it("should return the most specific route when multiple routes match", () => {
+    const mockUseLocation = jest.spyOn(require("react-router"), "useLocation");
+    mockUseLocation.mockReturnValue({ pathname: "/implementations/subscope/test" });
+
+    const mockRoutes = ["/implementations", "/implementations/subscope/test"];
+    
+    const result = useActiveRoutes(mockRoutes, null);
+    
+    // Should return the more specific route
+    expect(result).toBe("/implementations/subscope/test");
+  });
+
+  it("should handle subscope path detection correctly", () => {
+    const mockUseLocation = jest.spyOn(require("react-router"), "useLocation");
+    mockUseLocation.mockReturnValue({ pathname: "/concepts/subscope/AI_Initiative" });
+
+    const mockRoutes = ["/concepts", "/concepts/subscope/AI_Initiative"];
+    
+    const result = useActiveRoutes(mockRoutes, null);
+    
+    // Should return the more specific route
+    expect(result).toBe("/concepts/subscope/AI_Initiative");
+  });
+
+  it("should return null when no routes match", () => {
+    const mockUseLocation = jest.spyOn(require("react-router"), "useLocation");
+    mockUseLocation.mockReturnValue({ pathname: "/other" });
+
+    const mockRoutes = ["/implementations", "/concepts"];
+    
+    const result = useActiveRoutes(mockRoutes, null);
+    
+    expect(result).toBeNull();
+  });
+
+  it("should handle URL decoding properly", () => {
+    const mockUseLocation = jest.spyOn(require("react-router"), "useLocation");
+    mockUseLocation.mockReturnValue({ pathname: "/concepts/subscope/AI%20Initiative" });
+
+    const mockRoutes = ["/concepts", "/concepts/subscope/AI Initiative"];
+    
+    const result = useActiveRoutes(mockRoutes, null);
+    
+    // Should match the decoded route
+    expect(result).toBe("/concepts/subscope/AI Initiative");
+  });
+});

package/src/hooks/index.js

@@ -7,3 +7,4 @@ export * from "./usePath";
 export * from "./useOnScreen";
 export * from "./useOperators";
 export * from "./useTemplate";
+export * from "./useUploadJobs";

package/src/hooks/useActiveRoutes.js

@@ -6,27 +6,33 @@ import { matchPath } from "react-router";
 export const useActiveRoutes = (route, navFilter) => {
   const location = useLocation();
   const pathname = _.prop("pathname")(location);
-  const routes = _.castArray(route);
+  const decodedPathname = decodeURIComponent(pathname);
+  const routes = _.orderBy([_.size], ['desc'], _.castArray(route)); // Sort routes by length in descending order to prioritize more specific routes
 
-  return _.flow(
-    _.map((route) => ({
-      route,
-      filterMatch: matchPath({ path: BUCKETS_VIEW }, route),
-    })),
-    _.reduce(
-      (acc, { route, filterMatch }) => {
-        const { filterMatchRoutes, pathMatchRoutes } = acc;
+  for (const routeObj of _.map((route) => ({
+    route,
+    filterMatch: matchPath({ path: BUCKETS_VIEW }, route),
+  }))(routes)) {
+    const { route, filterMatch } = routeObj;
 
-        return navFilter &&
-          _.includes(filterMatch?.params?.propertyPath, Object.keys(navFilter))
-          ? { ...acc, filterMatchRoutes: route }
-          : route === pathname || _.startsWith(`${route}/`)(pathname)
-            ? { ...acc, pathMatchRoutes: route }
-            : { filterMatchRoutes, pathMatchRoutes };
-      },
-      { filterMatchRoutes: null, pathMatchRoutes: null }
-    ),
-    ({ filterMatchRoutes, pathMatchRoutes }) =>
-      !_.isEmpty(filterMatchRoutes) ? filterMatchRoutes : pathMatchRoutes
-  )(routes);
+    if (navFilter &&
+        filterMatch?.params?.propertyPath &&
+        _.includes(filterMatch?.params?.propertyPath, Object.keys(navFilter))) {
+      return route;
+    }
+
+    if (route === pathname || route === decodedPathname) {
+      return route;
+    }
+
+    if (_.startsWith(`${route}/`)(pathname) || _.startsWith(`${route}/`)(decodedPathname)) {
+      return route;
+    }
+
+    if (matchPath({ path: route }, pathname) || matchPath({ path: route }, decodedPathname)) {
+      return route;
+    }
+  }
+
+  return null;
 };

package/src/hooks/useUploadJobs.js

@@ -0,0 +1,24 @@
+import { compile } from "path-to-regexp";
+import useSWR from "swr";
+import { apiJson } from "@truedat/core/services/api";
+import {
+  API_UPLOAD_JOBS,
+  API_UPLOAD_JOB,
+} from "../api";
+
+export const useUploadJobs = (scope) => {
+  const url = scope ? `${API_UPLOAD_JOBS}?scope=${scope}` : API_UPLOAD_JOBS;
+  const { data, error, mutate } = useSWR(
+    url,
+    apiJson
+  );
+  return { data: data?.data, error, loading: !error && !data, mutate };
+};
+
+export const useUploadJob = (id) => {
+  const url = compile(API_UPLOAD_JOB)({
+    id: `${id}`,
+  });
+  const { data, error, mutate } = useSWR(url, apiJson);
+  return { data: data?.data, error, loading: !error && !data, mutate };
+};

package/src/routes.js

@@ -127,6 +127,7 @@ export const IMPLEMENTATION_MOVE = "/implementations/:implementation_id/move";
 export const IMPLEMENTATION_NEW = "/implementations/new";
 export const IMPLEMENTATION_NEW_BASIC = "/implementations/basic";
 export const IMPLEMENTATION_NEW_RAW = "/implementations/new_raw";
+export const IMPLEMENTATIONS_BY_SUBSCOPE = "/implementations/subscope/:subscope";
 export const IMPLEMENTATION_RESULTS =
   "/implementations/:implementation_id/results";
 export const IMPLEMENTATION_RESULTS_DETAILS =
@@ -233,7 +234,6 @@ export const SOURCE_JOBS_NEW = "/sources/:sourceId/jobs/new";
 export const STRUCTURE = "/structures/:id";
 export const STRUCTURES = "/structures";
 export const STRUCTURES_BULK_UPDATE = "/structures/bulkUpdate";
-export const STRUCTURES_UPLOAD_EVENTS = "/bulkUpdateTemplateContentEvents";
 export const STRUCTURE_CHILDREN = "/structures/:id/children";
 export const STRUCTURE_FIELDS = "/structures/:id/fields";
 export const STRUCTURE_VERSION_FIELDS =
@@ -253,6 +253,8 @@ export const STRUCTURE_MEMBERS_NEW = "/structures/:id/members/new";
 export const STRUCTURE_METADATA = "/structures/:id/metadata";
 export const STRUCTURE_NOTES = "/structures/:id/notes";
 export const STRUCTURE_NOTES_EDIT = "/structures/:id/notes/edit";
+export const STRUCTURE_NOTES_UPLOAD_JOBS = "/structureNotes/uploadJobs";
+export const STRUCTURE_NOTES_UPLOAD_JOB = "/structureNotes/uploadJobs/:id";
 export const STRUCTURE_PARENTS = "/structures/:id/parents";
 export const STRUCTURE_PROFILE = "/structures/:id/profile";
 export const STRUCTURE_RULES = "/structures/:id/rules";
@@ -385,6 +387,7 @@ const routes = {
   IMPLEMENTATION_NEW,
   IMPLEMENTATION_NEW_BASIC,
   IMPLEMENTATION_NEW_RAW,
+  IMPLEMENTATIONS_BY_SUBSCOPE,
   IMPLEMENTATION_RESULTS,
   IMPLEMENTATION_RESULTS_DETAILS,
   IMPLEMENTATION_RESULT_DETAILS,
@@ -471,7 +474,6 @@ const routes = {
   STRUCTURE,
   STRUCTURES,
   STRUCTURES_BULK_UPDATE,
-  STRUCTURES_UPLOAD_EVENTS,
   STRUCTURE_CHILDREN,
   STRUCTURE_FIELDS,
   STRUCTURE_VERSION_FIELDS,
@@ -489,6 +491,8 @@ const routes = {
   STRUCTURE_METADATA,
   STRUCTURE_NOTES_EDIT,
   STRUCTURE_NOTES,
+  STRUCTURE_NOTES_UPLOAD_JOBS,
+  STRUCTURE_NOTES_UPLOAD_JOB,
   STRUCTURE_PARENTS,
   STRUCTURE_PROFILE,
   STRUCTURE_RULES,

package/src/selectors/__tests__/getRiSubscopes.spec.js

@@ -0,0 +1,53 @@
+import { getRiSubscopes } from "../getRiSubscopes";
+
+describe("selectors: getRiSubscopes", () => {
+  it("should return subscopes for ri scope", () => {
+    const allTemplates = [
+      { scope: "ri", subscope: "subscope1" },
+      { scope: "ri", subscope: "subscope2" },
+      { scope: "ri", subscope: "subscope1" }, // duplicate
+      { scope: "ri", subscope: null }, // should be filtered out
+      { scope: "bg", subscope: "other" }, // different scope
+    ];
+
+    const state = { allTemplates };
+
+    const result = getRiSubscopes(state);
+
+    expect(result).toEqual(["subscope1", "subscope2"]);
+  });
+
+  it("should return empty array when no templates match", () => {
+    const allTemplates = [
+      { scope: "bg", subscope: "subscope1" },
+      { scope: "dq", subscope: "subscope2" },
+    ];
+
+    const state = { allTemplates };
+
+    const result = getRiSubscopes(state);
+
+    expect(result).toEqual([]);
+  });
+
+  it("should return empty array when no subscopes exist", () => {
+    const allTemplates = [
+      { scope: "ri", subscope: null },
+      { scope: "ri", subscope: undefined },
+    ];
+
+    const state = { allTemplates };
+
+    const result = getRiSubscopes(state);
+
+    expect(result).toEqual([]);
+  });
+
+  it("should return empty array when allTemplates is empty", () => {
+    const state = { allTemplates: [] };
+
+    const result = getRiSubscopes(state);
+
+    expect(result).toEqual([]);
+  });
+});

package/src/selectors/getRiSubscopes.js

@@ -0,0 +1,8 @@
+import _ from "lodash/fp";
+import { createSelector } from "reselect";
+import { getSubscopes } from "./subscopedTemplates";
+
+export const getRiSubscopes = createSelector(
+  _.prop("allTemplates"),
+  (allTemplates) => getSubscopes(allTemplates, "ri")
+);

package/src/selectors/index.js

@@ -5,6 +5,7 @@ export * from "./makeSearchQuerySelector";
 export * from "./getDashboardConfig";
 export * from "./getRecipients";
 export * from "./getConceptSubscope";
+export * from "./getRiSubscopes";
 export * from "./getSidemenuGlossarySubscopes";
 export * from "./subscopedTemplates";
 export * from "./taxonomy";