Stop polling idle agent update controls

internal/server/http/fleet_update_test.go

@@ -21,7 +21,7 @@ import (
 	"gitea.dcglab.co.uk/steve/restic-manager/internal/version"
 )
 
-func TestFleetUpdatePagePreservesFiltersWithoutTypedConfirmation(t *testing.T) {
+func TestFleetUpdateIdlePageDoesNotPollOrRequireTypedConfirmation(t *testing.T) {
 	t.Parallel()
 	_, baseURL, st := newTestServerWithUI(t)
 	cookie := loginAsAdmin(t, st)
@@ -44,9 +44,6 @@ func TestFleetUpdatePagePreservesFiltersWithoutTypedConfirmation(t *testing.T) {
 		`id="fleet-filter-name"`,
 		`id="fleet-filter-version"`,
 		`id="fleet-filter-state"`,
-		"htmx:beforeSwap",
-		"fleetCaptureUIState()",
-		"fleetRestoreUIState()",
 		"r.style.display=",
 	} {
 		if !strings.Contains(body, want) {
@@ -56,6 +53,33 @@ func TestFleetUpdatePagePreservesFiltersWithoutTypedConfirmation(t *testing.T) {
 	if strings.Contains(body, "Type selected count to confirm") || strings.Contains(body, `id="fleet-update-confirm"`) {
 		t.Error("page still renders the typed-count confirmation")
 	}
+	if strings.Contains(body, `hx-trigger="every 3s`) {
+		t.Error("idle selection page must not poll and replace operator input")
+	}
+}
+
+func TestFleetUpdateRunningPagePollsForProgress(t *testing.T) {
+	t.Parallel()
+	_, baseURL, st := newTestServerWithUI(t)
+	cookie, userID := loginAsAdminWithID(t, st)
+	hostID := makeHost(t, st, "fleet-running-host")
+	if err := st.CreateFleetUpdate(context.Background(), store.FleetUpdate{
+		ID: ulid.Make().String(), StartedByUserID: userID, TargetVersion: version.Version,
+	}, []string{hostID}); err != nil {
+		t.Fatalf("create fleet update: %v", err)
+	}
+
+	req, _ := stdhttp.NewRequest("GET", baseURL+"/settings/fleet-update", nil)
+	req.AddCookie(cookie)
+	res, err := stdhttp.DefaultClient.Do(req)
+	if err != nil {
+		t.Fatalf("get fleet update page: %v", err)
+	}
+	defer res.Body.Close()
+	raw, _ := io.ReadAll(res.Body)
+	if body := string(raw); !strings.Contains(body, `hx-trigger="every 3s`) {
+		t.Error("running rollout page must poll for progress")
+	}
 }
 
 // fakeFleetWorker stands in for *fleetupdate.Worker in HTTP tests.