From 648a9789bd446c6f5f2afcd0ead69c7b099c90c1 Mon Sep 17 00:00:00 2001 From: Akshay Shah Date: Tue, 21 Jun 2016 12:11:16 -0700 Subject: [PATCH] Fix assertion that warmup succeeded 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. --- relay_benchmark_test.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/relay_benchmark_test.go b/relay_benchmark_test.go index 74651c5b..f254e80e 100644 --- a/relay_benchmark_test.go +++ b/relay_benchmark_test.go @@ -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()