fix: step cleanup not capturing current step (#401)

eddycharly · web-flow · commit 8a65c1c1cb78 · 2022-11-08T12:53:42.000-07:00
Signed-off-by: Charles-Edouard Brétéché &lt;charles.edouard@nirmata.com&gt;
diff --git a/pkg/test/case.go b/pkg/test/case.go
@@ -347,11 +347,11 @@ func (t *Case) Run(test *testing.T, tc *report.Testcase) {
 		tc.Assertions += len(testStep.Errors)
 
 		if !t.SkipDelete {
-			defer func() {
-				if err := testStep.Clean(ns.Name); err != nil {
+			defer func(step *Step) {
+				if err := step.Clean(ns.Name); err != nil {
 					test.Error(err)
 				}
-			}()
+			}(testStep)
 		}
 
 		if errs := testStep.Run(ns.Name); len(errs) > 0 {

Original file line number	Diff line number	Diff line change
`@@ -347,11 +347,11 @@ func (t Case) Run(test testing.T, tc *report.Testcase) {`
`347`	`347`	`tc.Assertions += len(testStep.Errors)`
`348`	`348`
`349`	`349`	`if !t.SkipDelete {`
`350`		`- defer func() {`
`351`		`- if err := testStep.Clean(ns.Name); err != nil {`
	`350`	`+ defer func(step *Step) {`
	`351`	`+ if err := step.Clean(ns.Name); err != nil {`
`352`	`352`	`test.Error(err)`
`353`	`353`	`}`
`354`		`- }()`
	`354`	`+ }(testStep)`
`355`	`355`	`}`
`356`	`356`
`357`	`357`	`if errs := testStep.Run(ns.Name); len(errs) > 0 {`