fix: use SkipNow instead of Skip to prevent log line (#2433)

eddycharly · web-flow · commit a76a606f5cbf · 2025-08-19T13:03:39.000+02:00
Signed-off-by: Charles-Edouard Brétéché &lt;charles.edouard@nirmata.com&gt;
diff --git a/pkg/runner/runner.go b/pkg/runner/runner.go
@@ -138,7 +138,7 @@ func (r *runner) run(ctx context.Context, m mainstart, nsOptions v1alpha2.Namesp
 						}()
 						// skip check
 						if test.Test.Spec.Skip != nil && *test.Test.Spec.Skip {
-							t.Skip()
+							t.SkipNow()
 							return
 						}
 						// setup context
@@ -148,7 +148,7 @@ func (r *runner) run(ctx context.Context, m mainstart, nsOptions v1alpha2.Namesp
 						}
 						// fail fast check
 						if tc.FailFast() && tc.Failed() > 0 {
-							t.Skip()
+							t.SkipNow()
 							return
 						}
 						// setup cleaner

Original file line number	Diff line number	Diff line change
`@@ -138,7 +138,7 @@ func (r *runner) run(ctx context.Context, m mainstart, nsOptions v1alpha2.Namesp`
`138`	`138`	`}()`
`139`	`139`	`// skip check`
`140`	`140`	`if test.Test.Spec.Skip != nil && *test.Test.Spec.Skip {`
`141`		`- t.Skip()`
	`141`	`+ t.SkipNow()`
`142`	`142`	`return`
`143`	`143`	`}`
`144`	`144`	`// setup context`
`@@ -148,7 +148,7 @@ func (r *runner) run(ctx context.Context, m mainstart, nsOptions v1alpha2.Namesp`
`148`	`148`	`}`
`149`	`149`	`// fail fast check`
`150`	`150`	`if tc.FailFast() && tc.Failed() > 0 {`
`151`		`- t.Skip()`
	`151`	`+ t.SkipNow()`
`152`	`152`	`return`
`153`	`153`	`}`
`154`	`154`	`// setup cleaner`