link to specific logs page

Author: CamDavidsonPilon

frontend/src/__tests__/ErrorSnackbar.test.jsx

@@ -118,6 +118,25 @@ describe("ErrorSnackbar", () => {
     );
   });
 
+  test("links system log alerts to the unit-specific system logs page", () => {
+    renderErrorSnackbar();
+    const handler = mockSubscribeToTopic.mock.calls[0][1];
+
+    publishLog(handler, {
+      unit: "xr1",
+      experiment: "$experiment",
+      message: "stirring is already running (job_id=2062). Skipping.",
+    });
+
+    const firstCallOptions = mockEnqueueSnackbar.mock.calls[0][1];
+    render(<MemoryRouter>{firstCallOptions.content("snackbar-1")}</MemoryRouter>);
+
+    expect(screen.getByRole("link", { name: "View System Logs" })).toHaveAttribute(
+      "href",
+      "/system-logs/xr1",
+    );
+  });
+
   test("updates repeated alerts from the same unit without replacing the snackbar", () => {
     const { unmount } = renderErrorSnackbar();
     const handler = mockSubscribeToTopic.mock.calls[0][1];

frontend/src/components/ErrorSnackbar.jsx

@@ -87,15 +87,13 @@ function getDedupeKey({ unit, experiment, task, level, message }) {
 }
 
 function getLogsRoute(unit, experiment) {
-  if (experiment === "$experiment") {
-    return "/system-logs";
-  }
+  const logsRoute = experiment === "$experiment" ? "/system-logs" : "/logs";
 
   if (unit === "$broadcast") {
-    return "/logs";
+    return logsRoute;
   }
 
-  return `/logs/${encodeURIComponent(unit)}`;
+  return `${logsRoute}/${encodeURIComponent(unit)}`;
 }
 
 const LogAlertContent = React.forwardRef(function LogAlertContent({