Merge pull request #225 from lechefer/fix_fatal_when_set_status_skipped_or_broken

vitkarpenko · web-flow · commit e729bffc6ab5 · 2023-10-17T13:33:40.000+03:00
fix fatal when set status skipped or broken
diff --git a/runner/runner_testing.go b/runner/runner_testing.go
@@ -156,11 +156,10 @@ func (h testingHandler) HandleTest(test models.TestInterface, executeTest testEx
 	h.t.Run(test.GetName(), func(t *testing.T) {
 		result, err := executeTest(test)
 		if err != nil {
-			returnErr = err
-
 			if errors.Is(err, errTestSkipped) || errors.Is(err, errTestBroken) {
 				t.Skip()
 			} else {
+				returnErr = err
 				t.Fatal(err)
 			}
 		}

Original file line number	Diff line number	Diff line change
`@@ -156,11 +156,10 @@ func (h testingHandler) HandleTest(test models.TestInterface, executeTest testEx`
`156`	`156`	`h.t.Run(test.GetName(), func(t *testing.T) {`
`157`	`157`	`result, err := executeTest(test)`
`158`	`158`	`if err != nil {`
`159`		`- returnErr = err`
`160`		`-`
`161`	`159`	`if errors.Is(err, errTestSkipped) \|\| errors.Is(err, errTestBroken) {`
`162`	`160`	`t.Skip()`
`163`	`161`	`} else {`
	`162`	`+ returnErr = err`
`164`	`163`	`t.Fatal(err)`
`165`	`164`	`}`
`166`	`165`	`}`