Skip to content

Commit

Permalink
Fix assertion that warmup succeeded
Browse files Browse the repository at this point in the history
In the preamble to this benchmark, we need to successfully warm up each
TCP client. However, this assertion is testing an error created earlier;
fix it so that the benchmark doesn't proceed if we fail warmup.
  • Loading branch information
Akshay Shah committed Jun 21, 2016
1 parent e978a5a commit 648a978
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion relay_benchmark_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -188,7 +188,7 @@ func BenchmarkRelayNoLatencies(b *testing.B) {
benchmark.WithTimeout(10*time.Second),
)
defer client.Close()
require.NoError(b, err, client.Warmup(), "client.Warmup failed")
require.NoError(b, client.Warmup(), "client.Warmup failed")

b.ResetTimer()
started := time.Now()
Expand Down

0 comments on commit 648a978

Please sign in to comment.