remove code and file columns from alerts table

Author: kantord

src/components/AlertsTable.tsx

@@ -168,8 +168,6 @@ export function AlertsTable() {
                 Type
               </Column>
               <Column width={300}>Trigger Token</Column>
-              <Column width={150}>File</Column>
-              <Column width={250}>Code</Column>
             </Row>
           </TableHeader>
           <TableBody>
@@ -185,18 +183,6 @@ export function AlertsTable() {
                   <Cell className="overflow-auto whitespace-nowrap max-w-80">
                     {wrapObjectOutput(alert.trigger_string)}
                   </Cell>
-                  <Cell className="truncate">
-                    {alert.code_snippet?.filepath || "N/A"}
-                  </Cell>
-                  <Cell className="overflow-auto whitespace-nowrap max-w-80">
-                    {alert.code_snippet?.code ? (
-                      <pre className="max-h-40 overflow-auto bg-gray-100 p-2 whitespace-pre-wrap">
-                        <code>{alert.code_snippet.code}</code>
-                      </pre>
-                    ) : (
-                      "N/A"
-                    )}
-                  </Cell>
                 </Row>
               ))}
           </TableBody>

src/routes/__tests__/route-dashboard.test.tsx

@@ -150,16 +150,6 @@ describe("Dashboard", () => {
       }),
     ).toBeVisible();
 
-    expect(
-      screen.getByRole("columnheader", {
-        name: /file/i,
-      }),
-    ).toBeVisible();
-    expect(
-      screen.getByRole("columnheader", {
-        name: /code/i,
-      }),
-    ).toBeVisible();
     expect(
       screen.getByRole("columnheader", {
         name: /time/i,
@@ -176,17 +166,9 @@ describe("Dashboard", () => {
     const firstRow = within(screen.getByTestId("alerts-table")).getAllByRole(
       "row",
     )[1] as HTMLElement;
-    const secondRow = within(screen.getByTestId("alerts-table")).getAllByRole(
-      "row",
-    )[2] as HTMLElement;
 
-    expect(within(firstRow).getByText(/ghp_token/i)).toBeVisible();
     expect(within(firstRow).getByText(/codegate-secrets/i)).toBeVisible();
-    expect(within(firstRow).getAllByText(/n\/a/i).length).toEqual(2);
     expect(within(firstRow).getByText(/[0-9]+.*ago/i)).toBeVisible();
-
-    // check trigger_string null
-    expect(within(secondRow).getAllByText(/n\/a/i).length).toEqual(3);
   });
 
   it("should render malicious pkg", async () => {
@@ -294,7 +276,6 @@ describe("Dashboard", () => {
     const row = within(screen.getByTestId("alerts-table")).getAllByRole(
       "row",
     )[1] as HTMLElement;
-    expect(within(row).getByText(/ghp_token/i)).toBeVisible();
     expect(within(row).getByText(/codegate-secrets/i)).toBeVisible();
   });