From fc4b264e76f914849b35589ac1c0dd355519e4ed Mon Sep 17 00:00:00 2001 From: Steven Hartland Date: Fri, 9 Aug 2024 16:03:29 +0100 Subject: [PATCH] fix: skip unhealthy (#2717) Pass the right variable to skip unhealthy function in tests, so it actually functions. --- reaper_test.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/reaper_test.go b/reaper_test.go index 225d1f882e..c757fbb3ea 100644 --- a/reaper_test.go +++ b/reaper_test.go @@ -467,7 +467,7 @@ func Test_ReaperReusedIfHealthy(t *testing.T) { testProvider := newMockReaperProvider(t) t.Cleanup(testProvider.RestoreReaperState) - SkipIfProviderIsNotHealthy(&testing.T{}) + SkipIfProviderIsNotHealthy(t) ctx := context.Background() // As other integration tests run with the (shared) Reaper as well, re-use the instance to not interrupt other tests @@ -507,7 +507,7 @@ func Test_RecreateReaperIfTerminated(t *testing.T) { mockProvider := newMockReaperProvider(t) t.Cleanup(mockProvider.RestoreReaperState) - SkipIfProviderIsNotHealthy(&testing.T{}) + SkipIfProviderIsNotHealthy(t) provider, _ := ProviderDocker.GetProvider() ctx := context.Background() @@ -552,7 +552,7 @@ func TestReaper_reuseItFromOtherTestProgramUsingDocker(t *testing.T) { reaperInstance = nil reaperOnce = sync.Once{} - SkipIfProviderIsNotHealthy(&testing.T{}) + SkipIfProviderIsNotHealthy(t) ctx := context.Background() // As other integration tests run with the (shared) Reaper as well, re-use the instance to not interrupt other tests