Skip to content

Commit

Permalink
fix: the test of peering.PeeringService
Browse files Browse the repository at this point in the history
This commit fixes the issue that the test of
peering.PeeringService must check the connection of h3
but did not. This line seemed to be unintended.
  • Loading branch information
TakashiMatsuda committed May 15, 2021
1 parent afa9899 commit b3a6de8
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion peering/peering_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ func TestPeeringService(t *testing.T) {
t.Logf("waiting for h1's connection to h3 to work")
require.NoError(t, h1.Connect(ctx, peer.AddrInfo{ID: h3.ID(), Addrs: h3.Addrs()}))
require.Eventually(t, func() bool {
return h1.Network().Connectedness(h2.ID()) == network.Connected
return h1.Network().Connectedness(h3.ID()) == network.Connected
}, 30*time.Second, 100*time.Millisecond)

require.Len(t, h1.Network().Peers(), 3)
Expand Down

0 comments on commit b3a6de8

Please sign in to comment.