fix: store failure on last retry only (#181)

Signed-off-by: Yvonnick Esnault <yvonnick.esnault@corp.ovh.com>
ovh · Feb 12, 2019 · 0b0228a · 0b0228a
1 parent f363173
commit 0b0228a
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/process_teststep.go b/process_teststep.go
@@ -26,7 +26,10 @@ func (v *Venom) RunTestStep(tcc TestCaseContext, e *ExecutorWrap, ts *TestSuite,
 		result, err = runTestStepExecutor(tcc, e, ts, step, l)
 
 		if err != nil {
-			tc.Failures = append(tc.Failures, Failure{Value: RemoveNotPrintableChar(err.Error())})
+			// we save the failure only if it's the last attempt
+			if retry == e.retry {
+				tc.Failures = append(tc.Failures, Failure{Value: RemoveNotPrintableChar(err.Error())})
+			}
 			continue
 		}
 

diff --git a/tests/MyTestSuiteHTTP.yml b/tests/MyTestSuiteHTTP.yml
@@ -11,7 +11,7 @@ testcases:
     - result.body ShouldContainSubstring /dedicated/server
     - result.body ShouldContainSubstring /ipLoadbalancing
     - result.statuscode ShouldEqual 200
-    - result.bodyjson.apis.apis0.path ShouldEqual /allDom
+    - result.bodyjson.apis.apis0.path ShouldEqual /services
 
 - name: POST http testcase
   steps: