Fix bug where negative durations are reported when tests fail

steinfletcher · Oct 27, 2019 · 48c3507 · 48c3507
1 parent f583ff5
commit 48c3507
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/.gitignore b/.gitignore
@@ -3,3 +3,4 @@
 coverage.out
 resources/_gen/images/
 .DS_Store
+.sequence
diff --git a/apitest.go b/apitest.go
@@ -460,6 +460,7 @@ func (r *Response) End() Result {
 	apiTest := r.apiTest
 	defer func() {
 		if apiTest.debugEnabled {
+			apiTest.finished = time.Now()
 			fmt.Println(fmt.Sprintf("Duration: %s\n", apiTest.finished.Sub(apiTest.started)))
 		}
 	}()
@@ -468,14 +469,12 @@ func (r *Response) End() Result {
 		apiTest.t.Fatal("either define a http.Handler or enable networking")
 	}
 
+	apiTest.started = time.Now()
 	if apiTest.reporter != nil {
 		res := apiTest.report()
 		return Result{Response: res}
 	}
-
-	apiTest.started = time.Now()
 	res := r.runTest()
-	apiTest.finished = time.Now()
 
 	return Result{Response: res}
 }