status-im/consul
2
0
Fork 0
mirror of https://github.com/status-im/consul.git synced 2025-01-11 06:16:08 +00:00
Code Issues Projects Releases Wiki Activity

agent/local: don't use time.After in test since notify is instant

Browse Source
This commit is contained in:
Mitchell Hashimoto 2018-07-18 16:16:28 -05:00
parent 6be4fa9118
commit 9f128e40d6
No known key found for this signature in database
GPG Key ID: 744E147AA52F5B0A
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
agent/local/state_test.go
View File

@ -1627,7 +1627,7 @@ func TestAgent_AliasCheck(t *testing.T) {
l.UpdateCheck(types.CheckID("c1"), api.HealthCritical, "") l.UpdateCheck(types.CheckID("c1"), api.HealthCritical, "")
select { select {
case <-notifyCh: case <-notifyCh:
case <-time.After(100 * time.Millisecond): default:
t.Fatal("notify not received") t.Fatal("notify not received")
} }
@ -1653,7 +1653,7 @@ func TestAgent_AliasCheck(t *testing.T) {
l.UpdateCheck(types.CheckID("c1"), api.HealthPassing, "") l.UpdateCheck(types.CheckID("c1"), api.HealthPassing, "")
select { select {
case <-notifyCh: case <-notifyCh:
case <-time.After(100 * time.Millisecond): default:
t.Fatal("notify not received") t.Fatal("notify not received")
} }
} }