Skip to content

Commit

Permalink
Merge pull request #984 from libp2p/fix/TestRTEvictionOnFailedQuery
Browse files Browse the repository at this point in the history
fix(query): reverting changes on TestRTEvictionOnFailedQuery
  • Loading branch information
guillaumemichel authored Aug 22, 2024
2 parents 9b16e14 + d628ba6 commit a900221
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions query_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -39,10 +39,10 @@ func TestRTEvictionOnFailedQuery(t *testing.T) {
return nil
}))

// clear the addresses of the peers so that the next queries fail
d1.host.Peerstore().ClearAddrs(d2.self)
d2.host.Peerstore().ClearAddrs(d1.self)
// peers will still be in the RT because RT is decoupled with the host and peerstore
// close both hosts so query fails
require.NoError(t, d1.host.Close())
require.NoError(t, d2.host.Close())
// peers will still be in the RT because we have decoupled membership from connectivity
require.NoError(t, tu.WaitFor(ctx, func() error {
if !checkRoutingTable(d1, d2) {
return fmt.Errorf("should have routes")
Expand Down

0 comments on commit a900221

Please sign in to comment.