Move report creation outside of timing loop

This commit is contained in:
Bryan Boreham
2018-04-12 10:02:50 +00:00
parent c938a5ffa4
commit 593952cd6e
+6 -2
View File
@@ -67,13 +67,17 @@ func benchmarkMerger(b *testing.B, merger app.Merger) {
for i := 0; i < numHosts*5; i++ {
reports = append(reports, makeReport())
}
replacements := []report.Report{}
for i := 0; i < numHosts/3; i++ {
replacements = append(replacements, makeReport())
}
b.ResetTimer()
for i := 0; i < b.N; i++ {
// replace 1/3 of hosts work of reports & merge them all
for i := 0; i < numHosts/3; i++ {
reports[rand.Intn(len(reports))] = makeReport()
for i := 0; i < len(replacements); i++ {
reports[rand.Intn(len(reports))] = replacements[i]
}
merger.Merge(reports)