Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Tests] Add tests for CAddrMan #902

Merged
merged 3 commits into from Jun 13, 2019
Merged

[Tests] Add tests for CAddrMan #902

merged 3 commits into from Jun 13, 2019

Commits on May 27, 2019

  1. Creates unittests for addrman, makes addrman testable. 

    Adds several unittests for addrman to verify it works as expected.
Makes small modifications to addrman to allow deterministic and targeted tests.
    @EthanHeilman @Warrows
    EthanHeilman authored and Warrows committed May 27, 2019
    Copy the full SHA
    d4da015 View commit details
    Browse the repository at this point in the history

  2. Increase test coverage for addrman and addrinfo 

    Adds several unittests for CAddrMan and CAddrInfo.
Increases the accuracy of addrman tests.
Removes non-determinism in tests by overriding the random number generator.
Extracts testing code from addrman class to test class.
    @EthanHeilman @Warrows
    EthanHeilman authored and Warrows committed May 27, 2019
    Copy the full SHA
    3355c86 View commit details
    Browse the repository at this point in the history

  3. [Code Style] Remove std namespace in test file

    @Warrows
    Warrows committed May 27, 2019
    Copy the full SHA
    c84e0e8 View commit details
    Browse the repository at this point in the history