do not start/stop benchmark timer for no-ops

weaveworks · Dec 7, 2017 · d3e23e1 · d3e23e1
1 parent f305c35
commit d3e23e1
Showing 1 changed file with 1 addition and 4 deletions.
diff --git a/app/benchmark_internal_test.go b/app/benchmark_internal_test.go
@@ -42,8 +42,6 @@ func readReportFiles(path string) ([]report.Report, error) {
 func BenchmarkReportUnmarshal(b *testing.B) {
 	b.ResetTimer()
 	for i := 0; i < b.N; i++ {
-		b.StopTimer()
-		b.StartTimer()
 		if _, err := readReportFiles(*benchReportPath); err != nil {
 			b.Fatal(err)
 		}
@@ -58,9 +56,8 @@ func BenchmarkReportMerge(b *testing.B) {
 	merger := NewSmartMerger()
 
 	b.ResetTimer()
+
 	for i := 0; i < b.N; i++ {
-		b.StopTimer()
-		b.StartTimer()
 		merger.Merge(reports)
 	}
 }