From 985bde83d6e41fe8e157372780b2b7fe47bb80ef Mon Sep 17 00:00:00 2001 From: Michael Schurter Date: Thu, 6 Apr 2017 11:10:36 -0700 Subject: [PATCH] Improve test timings 1234ms was far longer than needed and not sleeping between iterations was just mean. --- client/client_test.go | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/client/client_test.go b/client/client_test.go index d08a5e9f0..0c4dee962 100644 --- a/client/client_test.go +++ b/client/client_test.go @@ -414,12 +414,13 @@ func TestClient_MixedTLS(t *testing.T) { QueryOptions: structs.QueryOptions{Region: "global"}, } var out structs.SingleNodeResponse - deadline := time.Now().Add(1234 * time.Millisecond) + deadline := time.Now().Add(100 * time.Millisecond) for time.Now().Before(deadline) { err := c1.RPC("Node.GetNode", &req, &out) if err == nil { t.Fatalf("client RPC succeeded when it should have failed:\n%+v", out) } + time.Sleep(3 * time.Millisecond) } } @@ -466,12 +467,13 @@ func TestClient_BadTLS(t *testing.T) { QueryOptions: structs.QueryOptions{Region: "global"}, } var out structs.SingleNodeResponse - deadline := time.Now().Add(1234 * time.Millisecond) + deadline := time.Now().Add(100 * time.Millisecond) for time.Now().Before(deadline) { err := c1.RPC("Node.GetNode", &req, &out) if err == nil { t.Fatalf("client RPC succeeded when it should have failed:\n%+v", out) } + time.Sleep(3 * time.Millisecond) } }