Add discovery process - Closes #900 - Githubissues

LiskArchive / lisk-elements

⚙️ Library for sending Lisk transactions from the client or server

https://lisk.io/

GNU General Public License v3.0

250 stars 64 forks source link

Add discovery process - Closes #900 #1095

Closed jondubois closed 5 years ago

jondubois commented 5 years ago

What was the problem?

Peer discovery was only running once. It needs to run on an interval to guarantee full discovery of the network.

How did I fix it?

Made discovery run on a configurable interval.
Minor code cleanup.
Added new integration test case to check that all peers eventually get discovered.

How to test it?

npm run test:integration

Review checklist

The PR resolves #900
All new code is covered with unit tests
All new code was formatted with Prettier
Linting passes
Tests pass
Commit messages follow the commit guidelines
Documentation has been added/updated