luxolus
/
open-consul
2
0
Fork 0
Code Issues 1 Pull Requests Packages Releases Wiki Activity

Fix race condition in `TestCheckTTL` 

Apparently the execution of `TestCheckTTL` could sometime take > 10ms
causing it to fail. This commit will eliminate the race condition for
most modern hardware.
This commit is contained in:
William Tisäter 2014-05-06 20:54:27 +02:00
parent a83ad5b659
commit 95f2597fe9
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
command/agent/check_test.go
View File

@ -98,13 +98,13 @@ func TestCheckTTL(t *testing.T) {
check := &CheckTTL{
Notify: mock,
CheckID: "foo",
TTL: 20 * time.Millisecond,
TTL: 100 * time.Millisecond,
Logger: log.New(os.Stderr, "", log.LstdFlags),
}
check.Start()
defer check.Stop()
time.Sleep(10 * time.Millisecond)
time.Sleep(50 * time.Millisecond)
check.SetStatus(structs.HealthPassing, "")
if mock.updates["foo"] != 1 {
@ -116,13 +116,13 @@ func TestCheckTTL(t *testing.T) {
}
// Ensure we don't fail early
time.Sleep(15 * time.Millisecond)
time.Sleep(75 * time.Millisecond)
if mock.updates["foo"] != 1 {
t.Fatalf("should have 1 updates %v", mock.updates)
}
// Wait for the TTL to expire
time.Sleep(15 * time.Millisecond)
time.Sleep(75 * time.Millisecond)
if mock.updates["foo"] != 2 {
t.Fatalf("should have 2 updates %v", mock.updates)