Skip to content
This repository has been archived by the owner on Feb 1, 2023. It is now read-only.

Commit

Permalink
fix(providequerymanager): improve test stability
Browse files Browse the repository at this point in the history
Removed a minor condition check that could fail in some cases just due to timing, but not a code
issue
  • Loading branch information
hannahhoward committed Jan 30, 2019
1 parent 6af352a commit c5678f1
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions providerquerymanager/providerquerymanager_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -211,9 +211,7 @@ func TestCancelManagerExitsGracefully(t *testing.T) {
secondPeersReceived = append(secondPeersReceived, p)
}

if len(firstPeersReceived) <= 0 ||
len(firstPeersReceived) >= len(peers) ||
len(secondPeersReceived) <= 0 ||
if len(firstPeersReceived) >= len(peers) ||
len(secondPeersReceived) >= len(peers) {
t.Fatal("Did not cancel requests in progress correctly")
}
Expand Down

0 comments on commit c5678f1

Please sign in to comment.