Fix panic in test case (overwriting a locked sync.Mutex with an unlocked one).
authorTom Clegg <tom@curoverse.com>
Mon, 24 Apr 2017 18:57:41 +0000 (14:57 -0400)
committerTom Clegg <tom@curoverse.com>
Mon, 24 Apr 2017 18:57:41 +0000 (14:57 -0400)
No issue #

sdk/go/dispatch/throttle_test.go

index d91513ed4013b7fe7acd9bbe04eec332136836d5..a6286d9924a3e2240b16bc8a88f6ef79da34e9d6 100644 (file)
@@ -18,21 +18,20 @@ type ThrottleTestSuite struct{}
 
 func (*ThrottleTestSuite) TestThrottle(c *check.C) {
        uuid := "zzzzz-zzzzz-zzzzzzzzzzzzzzz"
+       t0 := throttle{}
+       c.Check(t0.Check(uuid), check.Equals, true)
+       c.Check(t0.Check(uuid), check.Equals, true)
 
-       t := throttle{}
-       c.Check(t.Check(uuid), check.Equals, true)
-       c.Check(t.Check(uuid), check.Equals, true)
-
-       t = throttle{hold: time.Nanosecond}
-       c.Check(t.Check(uuid), check.Equals, true)
+       tNs := throttle{hold: time.Nanosecond}
+       c.Check(tNs.Check(uuid), check.Equals, true)
        time.Sleep(time.Microsecond)
-       c.Check(t.Check(uuid), check.Equals, true)
-
-       t = throttle{hold: time.Minute}
-       c.Check(t.Check(uuid), check.Equals, true)
-       c.Check(t.Check(uuid), check.Equals, false)
-       c.Check(t.Check(uuid), check.Equals, false)
-       t.seen[uuid].last = time.Now().Add(-time.Hour)
-       c.Check(t.Check(uuid), check.Equals, true)
-       c.Check(t.Check(uuid), check.Equals, false)
+       c.Check(tNs.Check(uuid), check.Equals, true)
+
+       tMin := throttle{hold: time.Minute}
+       c.Check(tMin.Check(uuid), check.Equals, true)
+       c.Check(tMin.Check(uuid), check.Equals, false)
+       c.Check(tMin.Check(uuid), check.Equals, false)
+       tMin.seen[uuid].last = time.Now().Add(-time.Hour)
+       c.Check(tMin.Check(uuid), check.Equals, true)
+       c.Check(tMin.Check(uuid), check.Equals, false)
 }