diff options
| author | Ava Chow <github@achow101.com> | 2024-05-09 16:20:43 -0400 |
|---|---|---|
| committer | Ava Chow <github@achow101.com> | 2024-05-09 16:20:43 -0400 |
| commit | 012e540acee4c06dbade5451e76c606f987e5957 (patch) | |
| tree | db6197c08163f0d20bf173ff5798402b45a56c22 /test/functional/p2p_initial_headers_sync.py | |
| parent | ceb1e078f8c0ae58ff72748b039184a205efe337 (diff) | |
| parent | d53d84834747c37f4060a9ef379e0a6b50155246 (diff) | |
Merge bitcoin/bitcoin#29122: test: adds outbound eviction functional tests, updates comment in ConsiderEviction
d53d84834747c37f4060a9ef379e0a6b50155246 test: adds outbound eviction tests for non outbound-full-relay peers (Sergi Delgado Segura)
a8d9a0edc7cef2c31a557ef53eb45520976b0d65 test: adds outbound eviction functional tests, updates comment in ConsiderEviction (Sergi Delgado Segura)
Pull request description:
## Motivation
While checking the outbound eviction code I realized a case was not considered within the comments, which in turn made me realize we had no functional tests for the outbound eviction case (when I went to check/add the test case).
This PR updates the aforementioned comment and adds functional tests to cover the outbound eviction logic, in addition to the existing unit tests found at `src/test/denialofservice_tests.cpp`.
ACKs for top commit:
davidgumberg:
reACK https://github.com/bitcoin/bitcoin/commit/d53d84834747c37f4060a9ef379e0a6b50155246
tdb3:
Re ACK for d53d84834747c37f4060a9ef379e0a6b50155246
achow101:
ACK d53d84834747c37f4060a9ef379e0a6b50155246
cbergqvist:
ACK d53d84834747c37f4060a9ef379e0a6b50155246
Tree-SHA512: 633b84bb1229fe21e2f650c1beada33ca7f190b64eafd64df2266516d21175e5d652e019ff7114f00cb8bd19f5817dc19e65adf75767a88e24dc0842ce40c63e
Diffstat (limited to 'test/functional/p2p_initial_headers_sync.py')
0 files changed, 0 insertions, 0 deletions