🧪 test(ui): close coverage gaps in composables, preferences, services, and views

- Add new specs for useOperationDisplayHold and views/containers/useContainerPolicy
- Extend coverage on useContainerFilters, preferences/migrate, services/logs,
  services/system-log-stream, and utils/container-update
- Drop unreachable short-circuit in useOperationDisplayHold.removeHeldOperation
  and scheduleHeldOperationRelease existing-record guard
- Simplify preferences/migrate pruneResponsiveDashboardLayouts early return to
  undefined on the missing-key path

Author: s-b-e-n-s-o-n

ui/src/composables/useOperationDisplayHold.ts

@@ -26,9 +26,6 @@ function setHeldOperation(operationId: string, hold: OperationDisplayHoldRecord)
 }
 
 function removeHeldOperation(operationId: string) {
-  if (!heldOperations.value.has(operationId)) {
-    return;
-  }
   const next = new Map(heldOperations.value);
   next.delete(operationId);
   heldOperations.value = next;
@@ -179,12 +176,7 @@ function scheduleHeldOperationRelease(args: {
   let scheduled = false;
 
   for (const operationId of operationIds) {
-    const existing = heldOperations.value.get(operationId);
-    if (!existing) {
-      continue;
-    }
-
-    const nextHold = updateHoldTargets(existing, args);
+    const nextHold = updateHoldTargets(heldOperations.value.get(operationId)!, args);
     setHeldOperation(operationId, nextHold);
 
     const remaining = nextHold.displayUntil - (args.now ?? Date.now());

ui/src/preferences/migrate.ts

@@ -58,7 +58,7 @@ function pruneResponsiveDashboardLayouts(
   dashboard: Record<string, unknown>,
 ): Record<string, unknown> | undefined {
   if (!('gridLayouts' in dashboard)) {
-    return isRecord(dashboard.gridLayouts) ? dashboard.gridLayouts : undefined;
+    return undefined;
   }
 
   if (!isRecord(dashboard.gridLayouts)) {

ui/tests/composables/useContainerFilters.spec.ts

@@ -193,6 +193,22 @@ describe('useContainerFilters', () => {
         'postgres',
       ]);
     });
+
+    it('should filter blocked containers only when they have a newTag', async () => {
+      const blockedWithTag = makeContainer({
+        id: 'c4',
+        name: 'api',
+        image: 'api:latest',
+        bouncer: 'blocked',
+        newTag: '2.0',
+      });
+      const mod = await import('@/composables/useContainerFilters');
+      const localContainers = ref([...containers.value, blockedWithTag]);
+      const filters = mod.useContainerFilters(localContainers);
+      filters.filterKind.value = 'blocked';
+
+      expect(filters.filteredContainers.value.map((c) => c.name)).toEqual(['api']);
+    });
   });
 
   describe('activeFilterCount', () => {