Skip to content
This repository was archived by the owner on Aug 2, 2021. It is now read-only.

swarm/network: Add tests for complex connectivity evaluations (2)#1108

Closed
nolash wants to merge 7 commits intomasterfrom
fill-me-up-again
Closed

swarm/network: Add tests for complex connectivity evaluations (2)#1108
nolash wants to merge 7 commits intomasterfrom
fill-me-up-again

Conversation

@nolash
Copy link
Copy Markdown
Contributor

@nolash nolash commented Jan 14, 2019

Obsoletes #1069

This PR adds tests for all kademlia connectivity conditions relevant to the peer discovery mechanisms. This is a preparatory step to verify the correctness of the SuggestPeer component.

It also adds tests for less naïve health evaluations, to inform simulation implementations that would be interested in this information.

Added tests are:

  • Bins shallower than depth are connected to minimum of peers if we know any peers in them
  • Unit test for saturation calculation

Merge dependencies (in order of merge to master):

@nolash
Copy link
Copy Markdown
Contributor Author

nolash commented Jan 16, 2019

Work abandoned after ethereum/go-ethereum#18404

@nolash nolash closed this Jan 16, 2019
@frncmx frncmx deleted the fill-me-up-again branch January 22, 2019 09:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants