lamoda / gonkey

Gonkey - a testing automation tool

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Fatal when set status skipped or broken

lechefer opened this issue · comments

Проблема:
Когда тест помечается как skipped или broken (в том числе из-за focus), то вылетает fatal error.
Выполнение прекращается и тесты далее не запускаются.

Как вариант при скипе не пробрасывать ошибку далее

index b14495c..0697650 100644                                   
--- 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)
                        }
                }

Pull request #225

commented

#204 - решали вроде уже проблему?

#204 - решали вроде уже проблему?

Да, видел, но практика показывает что проблема была решена не полностью

Самый простой тест кейс:
2 теста:
1 - status skipped
2 - без статуса

Если так запустить то будет ошибка test skipped и 2-й тест даже не запуститься