datajunction-ui 0.0.30 → 0.0.34

package/src/app/components/__tests__/__snapshots__/NamespaceHeader.test.jsx.snap

@@ -1,47 +1,152 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
 exports[`<NamespaceHeader /> should render and match the snapshot 1`] = `
-<ol
-  className="breadcrumb breadcrumb-chevron p-3 bg-body-tertiary rounded-3"
+<div
+  style={
+    Object {
+      "alignItems": "center",
+      "background": "#ffffff",
+      "borderBottom": "1px solid #e2e8f0",
+      "borderTop": "1px solid #e2e8f0",
+      "display": "flex",
+      "justifyContent": "space-between",
+      "marginBottom": "16px",
+      "padding": "12px 12px 12px 20px",
+    }
+  }
 >
-  <li
-    className="breadcrumb-item"
+  <div
+    style={
+      Object {
+        "alignItems": "center",
+        "display": "flex",
+        "gap": "8px",
+      }
+    }
   >
     <a
       href="/"
+      style={
+        Object {
+          "alignItems": "center",
+          "display": "flex",
+        }
+      }
     >
-      <HorizontalHierarchyIcon />
+      <svg
+        fill="currentColor"
+        height="16"
+        viewBox="0 0 16 16"
+        width="16"
+        xmlns="http://www.w3.org/2000/svg"
+      >
+        <path
+          d="M8.186 1.113a.5.5 0 0 0-.372 0L1.846 3.5 8 5.961 14.154 3.5 8.186 1.113zM15 4.239l-6.5 2.6v7.922l6.5-2.6V4.24zM7.5 14.762V6.838L1 4.239v7.923l6.5 2.6zM7.443.184a1.5 1.5 0 0 1 1.114 0l7.129 2.852A.5.5 0 0 1 16 3.5v8.662a1 1 0 0 1-.629.928l-7.185 2.874a.5.5 0 0 1-.372 0L.63 13.09a1 1 0 0 1-.63-.928V3.5a.5.5 0 0 1 .314-.464L7.443.184z"
+        />
+      </svg>
     </a>
-  </li>
-  <li
-    className="breadcrumb-item"
-  >
-    <a
-      className="link-body-emphasis"
-      href="/namespaces/shared"
+    <svg
+      fill="#6c757d"
+      height="12"
+      viewBox="0 0 16 16"
+      width="12"
+      xmlns="http://www.w3.org/2000/svg"
     >
-      shared
-    </a>
-  </li>
-  <li
-    className="breadcrumb-item"
-  >
-    <a
-      className="link-body-emphasis"
-      href="/namespaces/shared.dimensions"
+      <path
+        d="M4.646 1.646a.5.5 0 0 1 .708 0l6 6a.5.5 0 0 1 0 .708l-6 6a.5.5 0 0 1-.708-.708L10.293 8 4.646 2.354a.5.5 0 0 1 0-.708z"
+        fillRule="evenodd"
+      />
+    </svg>
+    <span
+      style={
+        Object {
+          "alignItems": "center",
+          "display": "flex",
+          "gap": "8px",
+        }
+      }
     >
-      dimensions
-    </a>
-  </li>
-  <li
-    className="breadcrumb-item"
-  >
-    <a
-      className="link-body-emphasis"
-      href="/namespaces/shared.dimensions.accounts"
+      <a
+        href="/namespaces/shared"
+        style={
+          Object {
+            "color": "#1e293b",
+            "fontWeight": "400",
+            "textDecoration": "none",
+          }
+        }
+      >
+        shared
+      </a>
+      <svg
+        fill="#94a3b8"
+        height="12"
+        viewBox="0 0 16 16"
+        width="12"
+        xmlns="http://www.w3.org/2000/svg"
+      >
+        <path
+          d="M4.646 1.646a.5.5 0 0 1 .708 0l6 6a.5.5 0 0 1 0 .708l-6 6a.5.5 0 0 1-.708-.708L10.293 8 4.646 2.354a.5.5 0 0 1 0-.708z"
+          fillRule="evenodd"
+        />
+      </svg>
+    </span>
+    <span
+      style={
+        Object {
+          "alignItems": "center",
+          "display": "flex",
+          "gap": "8px",
+        }
+      }
     >
-      accounts
-    </a>
-  </li>
-</ol>
+      <a
+        href="/namespaces/shared.dimensions"
+        style={
+          Object {
+            "color": "#1e293b",
+            "fontWeight": "400",
+            "textDecoration": "none",
+          }
+        }
+      >
+        dimensions
+      </a>
+      <svg
+        fill="#94a3b8"
+        height="12"
+        viewBox="0 0 16 16"
+        width="12"
+        xmlns="http://www.w3.org/2000/svg"
+      >
+        <path
+          d="M4.646 1.646a.5.5 0 0 1 .708 0l6 6a.5.5 0 0 1 0 .708l-6 6a.5.5 0 0 1-.708-.708L10.293 8 4.646 2.354a.5.5 0 0 1 0-.708z"
+          fillRule="evenodd"
+        />
+      </svg>
+    </span>
+    <span
+      style={
+        Object {
+          "alignItems": "center",
+          "display": "flex",
+          "gap": "8px",
+        }
+      }
+    >
+      <a
+        href="/namespaces/shared.dimensions.accounts"
+        style={
+          Object {
+            "color": "#1e293b",
+            "fontWeight": "400",
+            "textDecoration": "none",
+          }
+        }
+      >
+        accounts
+      </a>
+    </span>
+  </div>
+</div>
 `;

package/src/app/components/djgraph/__tests__/Collapse.test.jsx

@@ -35,10 +35,13 @@ describe('<Collapse />', () => {
         <Collapse
           {...defaultProps}
           data={{
+            name: 'test.transform',
             type: 'transform',
-            column_names: Array(11).fill(idx => {
-              return `column-${idx}`;
-            }),
+            column_names: Array.from({ length: 11 }, (_, idx) => ({
+              name: `column_${idx}`,
+              type: 'string',
+              order: idx,
+            })),
             primary_key: [],
           }}
         />

package/src/app/pages/AddEditNodePage/index.jsx

@@ -53,7 +53,7 @@ export function AddEditNodePage({ extensions = {} }) {
     primary_key: '',
     mode: 'published',
     owners: [],
-    custom_metadata: null,
+    custom_metadata: '',
   };
 
   const validator = values => {

package/src/app/pages/AddEditTagPage/index.jsx

@@ -90,7 +90,7 @@ export function AddEditTagPage() {
                             name="display_name"
                             id="display_name"
                             placeholder="Display Name"
-                            class="FullNameField"
+                            className="FullNameField"
                           />
                         </div>
                         <br />

package/src/app/pages/CubeBuilderPage/__tests__/index.test.jsx

@@ -185,30 +185,48 @@ describe('CubeBuilderPage', () => {
     jest.clearAllMocks();
   });
 
-  it('renders without crashing', () => {
+  it('renders without crashing', async () => {
     render(
       <DJClientContext.Provider value={{ DataJunctionAPI: mockDjClient }}>
         <CubeBuilderPage />
       </DJClientContext.Provider>,
     );
+
+    // Wait for async effects to complete
+    await waitFor(() => {
+      expect(mockDjClient.metrics).toHaveBeenCalled();
+    });
+
     expect(screen.getByText('Cube')).toBeInTheDocument();
   });
 
-  it('renders the Metrics section', () => {
+  it('renders the Metrics section', async () => {
     render(
       <DJClientContext.Provider value={{ DataJunctionAPI: mockDjClient }}>
         <CubeBuilderPage />
       </DJClientContext.Provider>,
     );
+
+    // Wait for async effects to complete
+    await waitFor(() => {
+      expect(mockDjClient.metrics).toHaveBeenCalled();
+    });
+
     expect(screen.getByText('Metrics *')).toBeInTheDocument();
   });
 
-  it('renders the Dimensions section', () => {
+  it('renders the Dimensions section', async () => {
     render(
       <DJClientContext.Provider value={{ DataJunctionAPI: mockDjClient }}>
         <CubeBuilderPage />
       </DJClientContext.Provider>,
     );
+
+    // Wait for async effects to complete
+    await waitFor(() => {
+      expect(mockDjClient.metrics).toHaveBeenCalled();
+    });
+
     expect(screen.getByText('Dimensions *')).toBeInTheDocument();
   });
 
@@ -237,22 +255,31 @@ describe('CubeBuilderPage', () => {
     }
     fireEvent.click(screen.getAllByText('Dimensions *')[0]);
 
-    expect(mockDjClient.commonDimensions).toHaveBeenCalled();
+    // Wait for commonDimensions to be called and state to update
+    await waitFor(() => {
+      expect(mockDjClient.commonDimensions).toHaveBeenCalled();
+    });
 
     const selectDimensions = screen.getAllByTestId('select-dimensions')[0];
     expect(selectDimensions).toBeDefined();
     expect(selectDimensions).not.toBeNull();
-    expect(
-      screen.getByText(
-        'default.repair_order_details.repair_order_id → default.repair_order.hard_hat_id → default.hard_hat.birth_date',
-      ),
-    ).toBeInTheDocument();
+
+    await waitFor(() => {
+      expect(
+        screen.getByText(
+          'default.repair_order_details.repair_order_id → default.repair_order.hard_hat_id → default.hard_hat.birth_date',
+        ),
+      ).toBeInTheDocument();
+    });
 
     const selectDimensionsDate = screen.getAllByTestId(
       'dimensions-default.date_dim',
     )[0];
 
     fireEvent.keyDown(selectDimensionsDate.firstChild, { key: 'ArrowDown' });
+    await waitFor(() => {
+      expect(screen.getByText('Day')).toBeInTheDocument();
+    });
     fireEvent.click(screen.getByText('Day'));
     fireEvent.click(screen.getByText('Month'));
     fireEvent.click(screen.getByText('Year'));
@@ -264,9 +291,8 @@ describe('CubeBuilderPage', () => {
     })[0];
     expect(createCube).toBeInTheDocument();
 
-    await waitFor(() => {
-      fireEvent.click(createCube);
-    });
+    fireEvent.click(createCube);
+
     await waitFor(() => {
       expect(mockDjClient.createCube).toHaveBeenCalledWith(
         '',
@@ -322,22 +348,31 @@ describe('CubeBuilderPage', () => {
 
     fireEvent.click(screen.getAllByText('Dimensions *')[0]);
 
-    expect(mockDjClient.commonDimensions).toHaveBeenCalled();
+    // Wait for commonDimensions to be called and state to update
+    await waitFor(() => {
+      expect(mockDjClient.commonDimensions).toHaveBeenCalled();
+    });
 
     const selectDimensions = screen.getAllByTestId('select-dimensions')[0];
     expect(selectDimensions).toBeDefined();
     expect(selectDimensions).not.toBeNull();
-    expect(
-      screen.getByText(
-        'default.repair_order_details.repair_order_id → default.repair_order.hard_hat_id → default.hard_hat.birth_date',
-      ),
-    ).toBeInTheDocument();
+
+    await waitFor(() => {
+      expect(
+        screen.getByText(
+          'default.repair_order_details.repair_order_id → default.repair_order.hard_hat_id → default.hard_hat.birth_date',
+        ),
+      ).toBeInTheDocument();
+    });
 
     const selectDimensionsDate = screen.getAllByTestId(
       'dimensions-default.date_dim',
     )[0];
 
     fireEvent.keyDown(selectDimensionsDate.firstChild, { key: 'ArrowDown' });
+    await waitFor(() => {
+      expect(screen.getByText('Day')).toBeInTheDocument();
+    });
     fireEvent.click(screen.getByText('Day'));
     fireEvent.click(screen.getByText('Month'));
     fireEvent.click(screen.getByText('Year'));
@@ -349,9 +384,8 @@ describe('CubeBuilderPage', () => {
     })[0];
     expect(createCube).toBeInTheDocument();
 
-    await waitFor(() => {
-      fireEvent.click(createCube);
-    });
+    fireEvent.click(createCube);
+
     await waitFor(() => {
       expect(mockDjClient.patchCube).toHaveBeenCalledWith(
         'default.repair_orders_cube',