Page MenuHomePhabricator

[tests] Remove maxblocksinflight testcase
ClosedPublic

Authored by deadalnix on Jan 20 2018, 18:33.

Details

Summary

maxblocksinflight tested that a node would not send get_data messages
for more than 16 new blocks at the same time. bitcoin core no longer
responds to block invs with get_data, since it does headers-first
sync'ing. This test was therefore testing nothing and can be removed.

the sendheaders test script tests that bitcoin will not send get_headers
for more than 16 blocks simultaneously.

Backport core's PR10023

Test Plan
make check
./test/functional/test_runner.py

Diff Detail

Repository
rABC Bitcoin ABC
Branch
deletemaxblockinflight
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 1676
Build 1676: arc lint + arc unit