From f029dfe9710e525a5ec2299b419119ced3f9c942 Mon Sep 17 00:00:00 2001
From: Bruce Schultz <bschultz013@gmail.com>
Date: Wed, 24 Jun 2026 13:17:40 +0200
Subject: [PATCH 1/5] feat(logs): optional debug log filter

---
 .../analysis/logs/AnalysisLogCardContent.vue  | 55 ++++++++++++++-----
 .../analysis/logs/ContainerLogs.vue           | 15 ++++-
 2 files changed, 55 insertions(+), 15 deletions(-)

diff --git a/app/components/analysis/logs/AnalysisLogCardContent.vue b/app/components/analysis/logs/AnalysisLogCardContent.vue
index c411edb..487e90b 100644
--- a/app/components/analysis/logs/AnalysisLogCardContent.vue
+++ b/app/components/analysis/logs/AnalysisLogCardContent.vue
@@ -1,6 +1,6 @@
 <script lang="ts" setup>
 import { Card, VirtualScroller } from "primevue";
-import { nextTick, onMounted, ref, watch } from "vue";
+import { computed, nextTick, onMounted, ref, watch } from "vue";
 import { useToast } from "primevue/usetoast";
 import Toast from "primevue/toast";
 import type { FlatLogLine } from "~/types/logs";
@@ -9,6 +9,7 @@ const props = defineProps<{
   nginxLines: FlatLogLine[];
   analysisLines: FlatLogLine[];
   showTimestamps?: boolean;
+  showDebug?: boolean;
   nginxHasOlder?: boolean;
   analysisHasOlder?: boolean;
   nginxLoading?: boolean;
@@ -19,6 +20,25 @@ const props = defineProps<{
 
 const toast = useToast();
 
+// Hide DEBUG entries unless toggled on
+function filterDebug(lines: FlatLogLine[]): FlatLogLine[] {
+  if (props.showDebug) return lines;
+  const result: FlatLogLine[] = [];
+  let skipping = false;
+  for (const line of lines) {
+    if (line.isStacktrace) {
+      if (!skipping) result.push(line);
+      continue;
+    }
+    skipping = line.level?.toUpperCase() === "DEBUG";
+    if (!skipping) result.push(line);
+  }
+  return result;
+}
+
+const displayedNginxLines = computed(() => filterDebug(props.nginxLines));
+const displayedAnalysisLines = computed(() => filterDebug(props.analysisLines));
+
 function isNearBottom(el: HTMLElement): boolean {
   return el.scrollTop >= el.scrollHeight - el.clientHeight - 50;
 }
@@ -80,7 +100,7 @@ function createScrollState(
 
 const { scrollerRef: nginxScrollerRef, onScroll: onNginxScroll } =
   createScrollState(
-    () => props.nginxLines,
+    () => displayedNginxLines.value,
     () => props.nginxHasOlder,
     () => props.nginxLoading,
     () => props.onLoadOlderNginx,
@@ -88,7 +108,7 @@ const { scrollerRef: nginxScrollerRef, onScroll: onNginxScroll } =
 
 const { scrollerRef: analysisScrollerRef, onScroll: onAnalysisScroll } =
   createScrollState(
-    () => props.analysisLines,
+    () => displayedAnalysisLines.value,
     () => props.analysisHasOlder,
     () => props.analysisLoading,
     () => props.onLoadOlderAnalysis,
@@ -139,7 +159,7 @@ function downloadLogs(isAnalysis: boolean) {
   const prefix = isAnalysis ? "analysis" : "nginx";
   const filename = `${prefix}-logs-${Date.now()}`;
   const logs = formatFlatLines(
-    isAnalysis ? props.analysisLines : props.nginxLines,
+    isAnalysis ? displayedAnalysisLines.value : displayedNginxLines.value,
   );
   if (logs) {
     const blob = new Blob([logs], { type: "text/plain" });
@@ -156,7 +176,7 @@ function downloadLogs(isAnalysis: boolean) {
 
 async function copyToClipboard(isAnalysis: boolean) {
   const logs = formatFlatLines(
-    isAnalysis ? props.analysisLines : props.nginxLines,
+    isAnalysis ? displayedAnalysisLines.value : displayedNginxLines.value,
   );
   if (logs) {
     try {
@@ -172,8 +192,6 @@ async function copyToClipboard(isAnalysis: boolean) {
     }
   }
 }
-
-
 </script>
 
 <template>
@@ -208,9 +226,9 @@ async function copyToClipboard(isAnalysis: boolean) {
       <template #content>
         <div class="card nginx-log-content">
           <VirtualScroller
-            v-if="nginxLines.length"
+            v-if="displayedNginxLines.length"
             ref="nginxScrollerRef"
-            :items="nginxLines"
+            :items="displayedNginxLines"
             :itemSize="20"
             style="height: 30em"
             class="log-scroll-panel"
@@ -232,7 +250,9 @@ async function copyToClipboard(isAnalysis: boolean) {
               </div>
             </template>
           </VirtualScroller>
-          <span v-else>No logs found...</span>
+          <div v-else class="log-entry-empty log-entry">
+            <span>No logs found...</span>
+          </div>
         </div>
       </template>
     </Card>
@@ -264,9 +284,9 @@ async function copyToClipboard(isAnalysis: boolean) {
       </template>
       <template #content>
         <VirtualScroller
-          v-if="analysisLines.length"
+          v-if="displayedAnalysisLines.length"
           ref="analysisScrollerRef"
-          :items="analysisLines"
+          :items="displayedAnalysisLines"
           :itemSize="20"
           style="height: 30em"
           class="log-scroll-panel"
@@ -288,7 +308,9 @@ async function copyToClipboard(isAnalysis: boolean) {
             </div>
           </template>
         </VirtualScroller>
-        <span v-else>No logs found...</span>
+        <div v-else class="log-entry-empty log-entry">
+          <span>No logs found...</span>
+        </div>
       </template>
     </Card>
   </div>
@@ -312,7 +334,8 @@ async function copyToClipboard(isAnalysis: boolean) {
 }
 
 .log-scroll-panel {
-  font-family: ui-monospace, "Cascadia Code", "Source Code Pro", Menlo, Monaco,
+  font-family:
+    ui-monospace, "Cascadia Code", "Source Code Pro", Menlo, Monaco,
     "Courier New", monospace;
   font-size: 0.8125rem;
   height: 30em;
@@ -369,6 +392,10 @@ async function copyToClipboard(isAnalysis: boolean) {
   overflow: hidden;
 }
 
+.log-entry-empty {
+  margin: 2rem 0.25rem;
+}
+
 .log-message {
   display: block;
   white-space: nowrap;
diff --git a/app/components/analysis/logs/ContainerLogs.vue b/app/components/analysis/logs/ContainerLogs.vue
index eb21b4f..c26904a 100644
--- a/app/components/analysis/logs/ContainerLogs.vue
+++ b/app/components/analysis/logs/ContainerLogs.vue
@@ -30,6 +30,7 @@ const analysis = ref<AnalysisNode | null>(null);
 const logChunks = useLogChunks(analysisId);
 const lastFetchedAt = ref<string | null>(null);
 const showTimestamps = ref(false);
+const showDebug = ref(false);
 
 await logChunks.initialize();
 if (logChunks.httpError.value === 403) {
@@ -149,6 +150,14 @@ function onRefreshToggle() {
             offIcon="pi pi-clock"
             class="log-timestamp-toggle"
           />
+          <ToggleButton
+            v-model="showDebug"
+            onLabel="Debug On"
+            offLabel="Debug Off"
+            onIcon="pi pi-eye"
+            offIcon="pi pi-eye-slash"
+            class="log-debug-toggle"
+          />
           <RefreshSwitch
             :disabled="analysis?.execution_status !== ProcessStatus.Executing"
             :startEnabled="
@@ -169,6 +178,7 @@ function onRefreshToggle() {
             :nginxLines="prevLogs[0]!.nginxLines"
             :analysisLines="prevLogs[0]!.analysisLines"
             :showTimestamps="showTimestamps"
+            :showDebug="showDebug"
           />
         </div>
         <AnalysisLogCardContent
@@ -176,6 +186,7 @@ function onRefreshToggle() {
           :nginxLines="logChunks.nginxLines.value"
           :analysisLines="logChunks.analysisLines.value"
           :showTimestamps="showTimestamps"
+          :showDebug="showDebug"
           :nginxHasOlder="logChunks.hasOlder.value"
           :analysisHasOlder="logChunks.hasOlder.value"
           :nginxLoading="logChunks.isLoading.value"
@@ -198,6 +209,7 @@ function onRefreshToggle() {
                 :nginxLines="run.nginxLines"
                 :analysisLines="run.analysisLines"
                 :showTimestamps="showTimestamps"
+                :showDebug="showDebug"
               />
             </Fieldset>
           </div>
@@ -237,7 +249,8 @@ function onRefreshToggle() {
   gap: 0.75em;
 }
 
-.log-timestamp-toggle {
+.log-timestamp-toggle,
+.log-debug-toggle {
   font-size: 0.8em;
 }
 </style>

From d51effb1d9dfefc36b06c156c7fa7e5a6183cd30 Mon Sep 17 00:00:00 2001
From: Bruce Schultz <bschultz013@gmail.com>
Date: Wed, 24 Jun 2026 13:18:03 +0200
Subject: [PATCH 2/5] style(logs): reload icon spins when refreshing logs

---
 app/components/analysis/logs/RefreshSwitch.vue | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/components/analysis/logs/RefreshSwitch.vue b/app/components/analysis/logs/RefreshSwitch.vue
index a7bba3d..0ce6bbb 100644
--- a/app/components/analysis/logs/RefreshSwitch.vue
+++ b/app/components/analysis/logs/RefreshSwitch.vue
@@ -20,7 +20,7 @@ const tooltip = props.disabled
     :disabled="props.disabled"
     onLabel="Refresh Logs On"
     offLabel="Refresh Logs Off"
-    onIcon="pi pi-refresh"
+    onIcon="pi pi-spin pi-refresh"
     offIcon="pi pi-refresh"
     class="log-timestamp-toggle"
     v-tooltip.top="tooltip"

From 165f165e0395c84151fb9d31c025e02a61c9d424 Mon Sep 17 00:00:00 2001
From: Bruce Schultz <bschultz013@gmail.com>
Date: Wed, 24 Jun 2026 13:18:33 +0200
Subject: [PATCH 3/5] style(analysis): hide log button situationally

---
 .../analysis/AnalysisControlButtons.vue          | 16 ++++++++++------
 1 file changed, 10 insertions(+), 6 deletions(-)

diff --git a/app/components/analysis/AnalysisControlButtons.vue b/app/components/analysis/AnalysisControlButtons.vue
index d29a134..db1a65a 100644
--- a/app/components/analysis/AnalysisControlButtons.vue
+++ b/app/components/analysis/AnalysisControlButtons.vue
@@ -17,6 +17,7 @@ interface ButtonStates {
   rerunActive: boolean;
   stopActive: boolean;
   deleteActive: boolean;
+  logsActive: boolean;
 }
 
 const props = defineProps({
@@ -121,26 +122,28 @@ const playButtonActiveStates = [null, "", undefined];
 const rerunButtonActiveStates: Array<string | null | undefined> = [
   PodStatus.Failed,
   PodStatus.Executed,
-  PodStatus.Finished, // Deprecated
   PodStatus.Stopped,
-  PodStatus.Stopping,
 ];
 const stopButtonActiveStates: Array<string | null | undefined> = [
   PodStatus.Executing,
-  PodStatus.Running, // Deprecated
   PodStatus.Starting,
   PodStatus.Started,
   PodStatus.Stopping,
 ];
 const deleteButtonActiveStates: Array<string | null | undefined> = [
-  PodStatus.Failed,
   PodStatus.Stopped,
   PodStatus.Stopping,
   PodStatus.Executing,
-  PodStatus.Running, // Deprecated
   PodStatus.Starting,
   PodStatus.Started,
 ];
+const logsButtonActiveStates: Array<string | null | undefined> = [
+  PodStatus.Failed,
+  PodStatus.Stopped,
+  PodStatus.Stopping,
+  PodStatus.Executing,
+  PodStatus.Executed,
+];
 
 function getButtonStatuses(podStatus: string | null | undefined) {
   return {
@@ -148,6 +151,7 @@ function getButtonStatuses(podStatus: string | null | undefined) {
     rerunActive: rerunButtonActiveStates.includes(podStatus),
     stopActive: stopButtonActiveStates.includes(podStatus),
     deleteActive: deleteButtonActiveStates.includes(podStatus),
+    logsActive: logsButtonActiveStates.includes(podStatus),
   };
 }
 
@@ -445,6 +449,7 @@ async function onDeleteAnalysis(silent: boolean = false) {
       @click="onDeleteAnalysis()"
     />
     <NuxtLink
+      v-show="buttonStatuses.logsActive"
       :to="{
         name: 'analyses-id',
         params: { id: props.analysisId },
@@ -454,7 +459,6 @@ async function onDeleteAnalysis(silent: boolean = false) {
     >
       <Button
         v-tooltip.top="'View the logs'"
-        :disabled="buttonStatuses.playActive"
         aria-label="Logs"
         class="analysis-btn logs-analysis-btn"
         icon="pi pi-bars"

From c43d112421a6c2bb53b8e84541e541fd59214259 Mon Sep 17 00:00:00 2001
From: Bruce Schultz <bschultz013@gmail.com>
Date: Thu, 25 Jun 2026 08:20:04 +0200
Subject: [PATCH 4/5] fix!: remove deprecated statuses

---
 app/components/analysis/AnalysesTable.vue             |  6 ++----
 app/components/analysis/ContainerCounter.vue          |  8 --------
 app/services/Api.ts                                   |  2 --
 app/utils/count-analyses.ts                           |  2 --
 app/utils/status-tag-severity.ts                      |  2 --
 test/components/analysis/AnalysisTable.spec.ts        |  2 +-
 test/components/analysis/AnalysisUpdateButton.spec.ts |  2 +-
 test/components/analysis/ContainerCounter.spec.ts     |  7 +------
 test/mockapi/handlers.ts                              |  4 ++--
 test/utils/count-analyses.test.ts                     | 10 ++++------
 test/utils/status-tag-severity.test.ts                |  2 --
 11 files changed, 11 insertions(+), 36 deletions(-)

diff --git a/app/components/analysis/AnalysesTable.vue b/app/components/analysis/AnalysesTable.vue
index 3ca79dc..bb7e8d1 100644
--- a/app/components/analysis/AnalysesTable.vue
+++ b/app/components/analysis/AnalysesTable.vue
@@ -822,15 +822,13 @@ const onCloseNavToast = () => {
             <template #body="{ data }">
               <ProgressBar
                 :mode="
-                  (data.execution_status === PodStatus.Executing ||
-                    data.execution_status === PodStatus.Running) &&
+                  data.execution_status === PodStatus.Executing &&
                   !data.execution_progress
                     ? 'indeterminate'
                     : 'determinate'
                 "
                 :style="
-                  (data.execution_status === PodStatus.Executing ||
-                    data.execution_status === PodStatus.Running) &&
+                  data.execution_status === PodStatus.Executing &&
                   !data.execution_progress
                     ? {}
                     : determineProgressBarColor(data.execution_progress)
diff --git a/app/components/analysis/ContainerCounter.vue b/app/components/analysis/ContainerCounter.vue
index 20c00ad..9d7cd74 100644
--- a/app/components/analysis/ContainerCounter.vue
+++ b/app/components/analysis/ContainerCounter.vue
@@ -24,14 +24,6 @@ const emit = defineEmits(["applyExecutionStatusFilter"]);
 
 function onApplyExecutionStatusFilter(executionStatus: PodStatus) {
   emit("applyExecutionStatusFilter", executionStatus);
-
-  // Also emit the PO alternatives
-  if (executionStatus === PodStatus.Executing) {
-    emit("applyExecutionStatusFilter", PodStatus.Running);
-  }
-  if (executionStatus === PodStatus.Executed) {
-    emit("applyExecutionStatusFilter", PodStatus.Finished);
-  }
 }
 </script>
 
diff --git a/app/services/Api.ts b/app/services/Api.ts
index 3a48709..bb80cd9 100644
--- a/app/services/Api.ts
+++ b/app/services/Api.ts
@@ -51,8 +51,6 @@ export enum PodStatus {
   Started = "started",
   Stopping = "stopping",
   Stopped = "stopped",
-  Running = "running",
-  Finished = "finished",
   Executing = "executing",
   Executed = "executed",
   Stuck = "stuck",
diff --git a/app/utils/count-analyses.ts b/app/utils/count-analyses.ts
index 4fb4d53..a515bdc 100644
--- a/app/utils/count-analyses.ts
+++ b/app/utils/count-analyses.ts
@@ -29,7 +29,6 @@ export function countAnalysisContainers(
           break;
 
         case PodStatus.Executing:
-        case PodStatus.Running:
           counts.executing++;
           break;
 
@@ -43,7 +42,6 @@ export function countAnalysisContainers(
           break;
 
         case PodStatus.Executed:
-        case PodStatus.Finished:
           counts.executed++;
           break;
 
diff --git a/app/utils/status-tag-severity.ts b/app/utils/status-tag-severity.ts
index a2caa89..51985bd 100644
--- a/app/utils/status-tag-severity.ts
+++ b/app/utils/status-tag-severity.ts
@@ -40,7 +40,6 @@ export const getExecutionStatusSeverity = (status: PodStatus) => {
       return "info";
 
     case PodStatus.Executing:
-    case PodStatus.Running:
       return "contrast";
 
     case PodStatus.Stopping:
@@ -51,7 +50,6 @@ export const getExecutionStatusSeverity = (status: PodStatus) => {
       return "danger";
 
     case PodStatus.Executed:
-    case PodStatus.Finished:
       return "success";
   }
 };
diff --git a/test/components/analysis/AnalysisTable.spec.ts b/test/components/analysis/AnalysisTable.spec.ts
index 1c99929..6bad3c3 100644
--- a/test/components/analysis/AnalysisTable.spec.ts
+++ b/test/components/analysis/AnalysisTable.spec.ts
@@ -377,7 +377,7 @@ describe("AnalysesTable.vue - PO calls and Progress", () => {
     const runStatusCell = wrapper.findAll("tbody tr")[0]!.findAll("td")[
       RUN_STATUS_COL
     ]!;
-    expect(runStatusCell.text()).toBe(PodStatus.Running);
+    expect(runStatusCell.text()).toBe(PodStatus.Executing);
   });
 
   test("Non-terminal execution_status is cleared when analysis is absent from PodOrc", async () => {
diff --git a/test/components/analysis/AnalysisUpdateButton.spec.ts b/test/components/analysis/AnalysisUpdateButton.spec.ts
index 6329cee..de44aca 100644
--- a/test/components/analysis/AnalysisUpdateButton.spec.ts
+++ b/test/components/analysis/AnalysisUpdateButton.spec.ts
@@ -63,7 +63,7 @@ describe("AnalysisUpdateButton.vue", () => {
     );
     expect(wrapper.emitted("updateAnalysisRunStatus")).toHaveLength(1);
     expect(wrapper.emitted("updateAnalysisRunStatus")![0]).toEqual([
-      PodStatus.Running,
+      PodStatus.Executing,
       undefined,
     ]);
   });
diff --git a/test/components/analysis/ContainerCounter.spec.ts b/test/components/analysis/ContainerCounter.spec.ts
index 27d7586..3d9554e 100644
--- a/test/components/analysis/ContainerCounter.spec.ts
+++ b/test/components/analysis/ContainerCounter.spec.ts
@@ -70,12 +70,7 @@ describe("ContainerCounter.vue", () => {
           expect(counterDiv.attributes("class")).toContain("opaque-badge");
         }
 
-        if (executionStatus === "Executing" || executionStatus === "Executed") {
-          // Emit extra for running and finished
-          emitCounts += 2;
-        } else {
-          emitCounts++;
-        }
+        emitCounts++;
         // Clicking on badge emits correctly
         badgeDiv.trigger("click");
         expect(wrapper.emitted()).toHaveProperty("applyExecutionStatusFilter");
diff --git a/test/mockapi/handlers.ts b/test/mockapi/handlers.ts
index 3603921..9222c3e 100644
--- a/test/mockapi/handlers.ts
+++ b/test/mockapi/handlers.ts
@@ -148,7 +148,7 @@ export const handlers = [
   http.get(`/po/status/${fakeAnalysisId}`, () => {
     return HttpResponse.json({
       [fakeAnalysisId]: {
-        status: PodStatus.Running,
+        status: PodStatus.Executing,
         progress: undefined,
       },
     });
@@ -157,7 +157,7 @@ export const handlers = [
   http.get(`/po/status`, () => {
     return HttpResponse.json({
       [fakeAnalysisId]: {
-        status: PodStatus.Running,
+        status: PodStatus.Executing,
         progress: undefined,
       },
     });
diff --git a/test/utils/count-analyses.test.ts b/test/utils/count-analyses.test.ts
index 422fb52..b31437c 100644
--- a/test/utils/count-analyses.test.ts
+++ b/test/utils/count-analyses.test.ts
@@ -29,12 +29,11 @@ describe("countAnalysisContainers", () => {
     expect(result.started).toBe(2);
   });
 
-  it("counts Executing and Running as executing", () => {
+  it("counts Executing as executing", () => {
     const result = countAnalysisContainers([
       makeNode(PodStatus.Executing),
-      makeNode(PodStatus.Running),
     ]);
-    expect(result.executing).toBe(2);
+    expect(result.executing).toBe(1);
   });
 
   it("counts Stopping and Stopped as stopped", () => {
@@ -50,12 +49,11 @@ describe("countAnalysisContainers", () => {
     expect(result.failed).toBe(1);
   });
 
-  it("counts Executed and Finished as executed", () => {
+  it("counts Executed as executed", () => {
     const result = countAnalysisContainers([
       makeNode(PodStatus.Executed),
-      makeNode(PodStatus.Finished),
     ]);
-    expect(result.executed).toBe(2);
+    expect(result.executed).toBe(1);
   });
 
   it("skips nodes with null or undefined execution_status", () => {
diff --git a/test/utils/status-tag-severity.test.ts b/test/utils/status-tag-severity.test.ts
index 416ae64..52bdfbb 100644
--- a/test/utils/status-tag-severity.test.ts
+++ b/test/utils/status-tag-severity.test.ts
@@ -42,12 +42,10 @@ test("Analysis run status severity tag", () => {
   const expectations = {
     starting: "info",
     started: "info",
-    running: "contrast", // deprecated
     executing: "contrast",
     stopping: "warning",
     stopped: "warning",
     executed: "success",
-    finished: "success", // deprecated
     failed: "danger",
   };
 

From cff3d41b07cb44629dc3e516fb45ba3ab5bd0cf8 Mon Sep 17 00:00:00 2001
From: Bruce Schultz <bschultz013@gmail.com>
Date: Thu, 25 Jun 2026 09:49:06 +0200
Subject: [PATCH 5/5] test(analysis): fix analysis button tests

---
 .../analysis/AnalysisControlButtons.spec.ts   | 84 +++++++++----------
 1 file changed, 42 insertions(+), 42 deletions(-)

diff --git a/test/components/analysis/AnalysisControlButtons.spec.ts b/test/components/analysis/AnalysisControlButtons.spec.ts
index eaa1f1f..1fdabc1 100644
--- a/test/components/analysis/AnalysisControlButtons.spec.ts
+++ b/test/components/analysis/AnalysisControlButtons.spec.ts
@@ -16,6 +16,7 @@ interface ButtonStates {
   rerunActive: boolean;
   stopActive: boolean;
   deleteActive: boolean;
+  logsActive: boolean;
 }
 
 function isButtonVisible(wrapper: VueWrapper, selector: string): boolean {
@@ -26,6 +27,17 @@ function isButtonVisible(wrapper: VueWrapper, selector: string): boolean {
   return !style || !style.includes("display: none");
 }
 
+// The logs button is wrapped in a NuxtLink that carries the v-show, so the
+// `display: none` lands on the anchor rather than the button itself.
+function isLogsBtnVisible(wrapper: VueWrapper): boolean {
+  const btn = wrapper.find(".logs-analysis-btn");
+  if (!btn.exists()) return false;
+
+  const anchor = btn.element.closest("a");
+  const style = anchor?.getAttribute("style");
+  return !style || !style.includes("display: none");
+}
+
 describe("AnalysisControlButtons.vue", () => {
   let spy;
   let mockToast;
@@ -105,6 +117,7 @@ describe("AnalysisControlButtons.vue", () => {
         rerunActive: false,
         stopActive: true,
         deleteActive: true,
+        logsActive: false,
       },
       "",
       PodStatus.Started,
@@ -130,6 +143,7 @@ describe("AnalysisControlButtons.vue", () => {
         rerunActive: false,
         stopActive: false,
         deleteActive: false,
+        logsActive: false,
       },
       "",
       "",
@@ -148,6 +162,7 @@ describe("AnalysisControlButtons.vue", () => {
         rerunActive: false,
         stopActive: false,
         deleteActive: false,
+        logsActive: false,
       },
       "",
       "",
@@ -168,6 +183,7 @@ describe("AnalysisControlButtons.vue", () => {
         rerunActive: true,
         stopActive: false,
         deleteActive: true,
+        logsActive: true,
       },
       PodStatus.Executing,
       PodStatus.Stopped,
@@ -186,6 +202,7 @@ describe("AnalysisControlButtons.vue", () => {
         rerunActive: true,
         stopActive: false,
         deleteActive: true,
+        logsActive: true,
       },
       PodStatus.Executing,
       PodStatus.Stopped,
@@ -204,6 +221,7 @@ describe("AnalysisControlButtons.vue", () => {
         rerunActive: false,
         stopActive: true,
         deleteActive: true,
+        logsActive: true,
       },
       PodStatus.Executing,
       PodStatus.Executing,
@@ -222,6 +240,7 @@ describe("AnalysisControlButtons.vue", () => {
         rerunActive: false,
         stopActive: false,
         deleteActive: false,
+        logsActive: false,
       },
       PodStatus.Executing,
       "",
@@ -240,6 +259,7 @@ describe("AnalysisControlButtons.vue", () => {
         rerunActive: false,
         stopActive: false,
         deleteActive: false,
+        logsActive: false,
       },
     );
   });
@@ -256,6 +276,7 @@ describe("AnalysisControlButtons.vue", () => {
         rerunActive: false,
         stopActive: true,
         deleteActive: true,
+        logsActive: true,
       },
       PodStatus.Executing,
       PodStatus.Executing,
@@ -274,58 +295,37 @@ describe("AnalysisControlButtons.vue", () => {
         rerunActive: false,
         stopActive: false,
         deleteActive: false,
+        logsActive: false,
       },
       "",
     );
   });
 
-  it("Log btn check", async () => {
-    const wrapper = mount(AnalysisControlButtons, {
-      props: {
-        analysisBuildStatus: ProcessStatus.Executed,
-        analysisExecutionStatus: null,
-        analysisNodeId: "8003eefe-e39b-4bd4-aec4-78046c63b39b",
-        analysisDistributionStatus: ProcessStatus.Executed,
-        analysisId: fakeAnalysisId,
-        approvalStatus: "approved",
-        projectId: "7f2f3b59-3b6d-4fb6-a900-2a4d5c2ea483",
-        nodeId: "e3b89572-327f-4936-8cf0-fbfbcc6336b7",
-        datastore: true,
-        requireDatastore: true,
-      },
-    });
+  it("Log btn - hidden until the pod has logs to show", async () => {
+    const wrapper = mountControls(null);
 
     expect(AnalysisControlButtons).toBeTruthy();
 
-    // Success check
-    const logBtn = wrapper.find(".logs-analysis-btn");
-    expect(logBtn.attributes("data-p-disabled")).toBe("true");
-
-    await wrapper.setProps({
-      analysisExecutionStatus: PodStatus.Started,
-    });
-    expect(logBtn.attributes("data-p-disabled")).toBe("false");
-
-    await wrapper.setProps({
-      analysisExecutionStatus: PodStatus.Executing,
-    });
-    expect(logBtn.attributes("data-p-disabled")).toBe("false");
-
-    await wrapper.setProps({
-      analysisExecutionStatus: "",
-    });
-    expect(logBtn.attributes("data-p-disabled")).toBe("true");
+    // No run yet, so there is nothing to view
+    expect(isLogsBtnVisible(wrapper)).toBe(false);
 
-    await wrapper.setProps({
-      analysisExecutionStatus: PodStatus.Failed,
-    });
-    expect(logBtn.attributes("data-p-disabled")).toBe("false");
+    // Shown for statuses that have produced logs
+    for (const status of [
+      PodStatus.Failed,
+      PodStatus.Stopped,
+      PodStatus.Stopping,
+      PodStatus.Executing,
+      PodStatus.Executed,
+    ]) {
+      await wrapper.setProps({ analysisExecutionStatus: status });
+      expect(isLogsBtnVisible(wrapper)).toBe(true);
+    }
 
-    await wrapper.setProps({
-      analysisExecutionStatus: PodStatus.Executed,
-    });
-    expect(logBtn.attributes("data-p-disabled")).toBe("false");
-    //
+    // Hidden again for statuses without logs
+    for (const status of [PodStatus.Starting, PodStatus.Started, ""]) {
+      await wrapper.setProps({ analysisExecutionStatus: status });
+      expect(isLogsBtnVisible(wrapper)).toBe(false);
+    }
   });
 
   function mountControls(analysisExecutionStatus: string | null) {