From 024ca6356bcd09274594e965c1cce45185272eac Mon Sep 17 00:00:00 2001 From: Chad Crum Date: Thu, 18 Jun 2026 13:14:20 -0400 Subject: [PATCH] fix(tests): add Label("cluster") to rehydration tests with K8s assertions Tests using inline `if kubectlAvailable` guards to assert K8s resource state were missing Label("cluster"), causing failures in CI when running against a mock backend on a reachable cluster. The guard is always true in that environment since kubectlAvailable checks cluster reachability and provider namespaces are hardcoded non-empty. Co-Authored-By: Claude Opus 4.6 Signed-off-by: Chad Crum --- tests/e2e/rehydration_failover_test.go | 4 ++-- tests/e2e/rehydration_policy_test.go | 6 +++--- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/tests/e2e/rehydration_failover_test.go b/tests/e2e/rehydration_failover_test.go index 21cf1f8..0214b4d 100644 --- a/tests/e2e/rehydration_failover_test.go +++ b/tests/e2e/rehydration_failover_test.go @@ -50,7 +50,7 @@ var _ = Describe("Rehydration Failover", Label("rehydration", "failover", "disru } }) - It("rehydrate after provider stop moves workload to healthy provider", func() { + It("rehydrate after provider stop moves workload to healthy provider", Label("cluster"), func() { preInst := getInstance(instanceUID) origResourceID := preInst.ResourceID @@ -76,7 +76,7 @@ var _ = Describe("Rehydration Failover", Label("rehydration", "failover", "disru } }) - It("rehydrate back after provider restore (bidirectional failover)", func() { + It("rehydrate back after provider restore (bidirectional failover)", Label("cluster"), func() { startProvider(providerA) waitForProviderHealth(providerA.Name, "ready", healthTimeout) diff --git a/tests/e2e/rehydration_policy_test.go b/tests/e2e/rehydration_policy_test.go index c9710a3..660b497 100644 --- a/tests/e2e/rehydration_policy_test.go +++ b/tests/e2e/rehydration_policy_test.go @@ -15,7 +15,7 @@ var _ = Describe("Rehydration Policy", Label("rehydration", "policy"), func() { }) Context("sovereignty", Label("disruptive"), func() { - It("placement policy constrains instance to provider group", func() { + It("placement policy constrains instance to provider group", Label("cluster"), func() { requireThreeProviders() requirePodman() @@ -44,7 +44,7 @@ var _ = Describe("Rehydration Policy", Label("rehydration", "policy"), func() { } }) - It("failover stays within provider group", func() { + It("failover stays within provider group", Label("cluster"), func() { requireThreeProviders() requirePodman() @@ -145,7 +145,7 @@ var _ = Describe("Rehydration Policy", Label("rehydration", "policy"), func() { "user_values should be identical before and after rehydration") }) - It("policy change causes rehydration to select different provider", func() { + It("policy change causes rehydration to select different provider", Label("cluster"), func() { requireMultiProvider() providerA := threeTierProviders[0]