#5352: Re-fetch the (server-cached) group list on every instance change

Author: ulischulte

spring-boot-admin-server-ui/src/main/frontend/views/instances/details/details-health.spec.ts

@@ -105,7 +105,7 @@ describe('DetailsHealth', () => {
   });
 
   describe('SSE reactive updates', () => {
-    it('should call fetchCachedHealthGroups once on mount, not on SSE version changes', async () => {
+    it('should re-fetch cached health groups on SSE version changes', async () => {
       const baseApp = applications[0];
       const instance1 = new Application(baseApp).instances[0];
       instance1.fetchCachedHealthGroups = vi
@@ -119,7 +119,8 @@ describe('DetailsHealth', () => {
       await screen.findAllByRole('status');
       expect(instance1.fetchCachedHealthGroups).toHaveBeenCalledTimes(1);
 
-      // Same instance, different version (SSE update) — should NOT call fetchCachedHealthGroups again
+      // Same instance, different version (SSE update) — should re-fetch the
+      // (server-cached) group list so it self-corrects when groups change.
       const instance2 = new Application({
         ...baseApp,
         instances: [
@@ -132,15 +133,17 @@ describe('DetailsHealth', () => {
       }).instances[0];
       instance2.fetchCachedHealthGroups = vi
         .fn()
-        .mockResolvedValue({ data: [] });
+        .mockResolvedValue({ data: ['liveness', 'readiness'] });
 
       await rerender({ instance: instance2 });
 
-      // Original instance's mock should still be 1 (no additional calls)
-      expect(instance1.fetchCachedHealthGroups).toHaveBeenCalledTimes(1);
+      // The new instance's cached groups should have been fetched on the SSE update.
+      await waitFor(() => {
+        expect(instance2.fetchCachedHealthGroups).toHaveBeenCalledTimes(1);
+      });
     });
 
-    it('should reactively update through multiple SSE status changes without extra HTTP calls', async () => {
+    it('should reactively update health status and details through SSE status changes', async () => {
       const baseApp = applications[0];
 
       const instance1 = new Application(baseApp).instances[0];

spring-boot-admin-server-ui/src/main/frontend/views/instances/details/details-health.vue

@@ -175,15 +175,9 @@ export default defineComponent({
         this.healthGroupOpenStatus = {};
         this.healthGroupLoadingMap = {};
         this.healthGroupsError = null;
-        this.fetchHealthGroups();
-      } else {
-        // Same instance, SSE update (e.g. status change) — collapse groups and clear stale data
-        for (const group of this.healthGroups) {
-          group.data = null;
-        }
-        this.healthGroupOpenStatus = {};
-        this.healthGroupLoadingMap = {};
       }
+      // Re-fetch the (server-cached) group list on every instance change
+      this.fetchHealthGroups();
     },
     isHealthGroupOpen(groupName: string) {
       return this.healthGroupOpenStatus[groupName]?.isOpen ?? false;