miner: fix waitForMiningState precision
This helper function would return an affirmation on the first positive match on a desired bool. This was imprecise; it return false positives by not waiting initially for an 'updated' value. This fix causes TestMiner_2 to fail, which is expected. Signed-off-by: meows <b5c6@protonmail.com>
This commit is contained in:
parent
3b783dd1ed
commit
6d365c2851
|
@ -89,10 +89,10 @@ func TestMiner(t *testing.T) {
|
|||
// Stop the downloader and wait for the update loop to run
|
||||
mux.Post(downloader.DoneEvent{})
|
||||
waitForMiningState(t, miner, true)
|
||||
// Start the downloader, the mining state will not change because the update loop has exited
|
||||
|
||||
mux.Post(downloader.StartEvent{})
|
||||
waitForMiningState(t, miner, true)
|
||||
// Stop the downloader, the mining state will not change
|
||||
waitForMiningState(t, miner, false)
|
||||
|
||||
mux.Post(downloader.FailedEvent{})
|
||||
waitForMiningState(t, miner, true)
|
||||
}
|
||||
|
@ -158,10 +158,10 @@ func waitForMiningState(t *testing.T, m *Miner, mining bool) {
|
|||
|
||||
var state bool
|
||||
for i := 0; i < 100; i++ {
|
||||
time.Sleep(10 * time.Millisecond)
|
||||
if state = m.Mining(); state == mining {
|
||||
return
|
||||
}
|
||||
time.Sleep(10 * time.Millisecond)
|
||||
}
|
||||
t.Fatalf("Mining() == %t, want %t", state, mining)
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue