From e16ca9b701c9ac21f0f2810b3e2ffd9cc0ec3a86 Mon Sep 17 00:00:00 2001 From: unkin-agent Date: Sun, 6 Sep 2026 00:55:25 +1000 Subject: [PATCH] Drop the recovery test the per-state table now covers --- health_test.go | 15 --------------- 1 file changed, 15 deletions(-) diff --git a/health_test.go b/health_test.go index 5c74a40..0fe6b0d 100644 --- a/health_test.go +++ b/health_test.go @@ -772,21 +772,6 @@ func TestProber_SuccessResetsTheRunInEveryDirection(t *testing.T) { } } -func TestProber_RecoversFromProbeUnsupported(t *testing.T) { - p := newTestProber(t, healthConfig("http://a.invalid", "http://b.invalid")) - markUnsupported(t, p, "a") - - for i := 0; i < p.successes; i++ { - p.record("a", nil) - } - if !p.healthy("a") { - t.Fatal("backend left the pool while recovering") - } - if got := p.snapshot()["a"].stateName(); got != stateHealthy { - t.Fatalf("state = %q, want %q", got, stateHealthy) - } -} - // The misconfigured-path case: nothing has ever answered the probe on this // backend, so there is no health signal to gate on and it keeps serving queries // however its probes fail — including real failures, since a probe that cannot