Fix incorrect rebasing issue: use StepState.GetError() method instead of directly accessing internal variables

rihter007 · rihter007 · commit 8196bfadf90d · 2022-08-08T11:43:08.000+01:00
Signed-off-by: Ilya &lt;rihter007@inbox.ru&gt;
diff --git a/pkg/runner/test_runner.go b/pkg/runner/test_runner.go
@@ -223,7 +223,7 @@ func (tr *TestRunner) Run(
 			ctx.Errorf("Failed to get steps variables: %v", err)
 			return nil, nil, err
 		}
-		stepErr := tr.steps[tgs.CurStep].runErr
+		stepErr := tr.steps[tgs.CurStep].GetError()
 		if tgs.CurPhase == targetStepPhaseRun {
 			numInFlightTargets++
 			if stepErr != xcontext.ErrPaused {

Original file line number	Diff line number	Diff line change
`@@ -223,7 +223,7 @@ func (tr *TestRunner) Run(`
`223`	`223`	`ctx.Errorf("Failed to get steps variables: %v", err)`
`224`	`224`	`return nil, nil, err`
`225`	`225`	`}`
`226`		`- stepErr := tr.steps[tgs.CurStep].runErr`
	`226`	`+ stepErr := tr.steps[tgs.CurStep].GetError()`
`227`	`227`	`if tgs.CurPhase == targetStepPhaseRun {`
`228`	`228`	`numInFlightTargets++`
`229`	`229`	`if stepErr != xcontext.ErrPaused {`