test: Use ConnmanTestMsg from test lib in denialofservice_tests #22203

pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:2106-testConnman changing 1 files +14 −31

MarcoFalke commented at 7:32 PM on June 9, 2021: member

This allows to remove code.

Also, required for https://github.com/bitcoin/bitcoin/pull/18470
test: Use ConnmanTestMsg from test lib in denialofservice_tests fa72fce7c9
practicalswift commented at 8:03 PM on June 9, 2021: contributor

Concept ACK: +14 -31 LOC is nice
DrahtBot added the label Tests on Jun 9, 2021
DrahtBot commented at 11:54 PM on June 9, 2021: member


The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.



Conflicts

Reviewers, this pull request conflicts with the following ones:
- #20966 (banman: save the banlist in a JSON format on disk by vasild)
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.
fanquake approved
fanquake commented at 12:53 PM on June 10, 2021: member

ACK fa72fce7c948185752a01002000ea511809146ed
mjdietzx commented at 2:41 PM on June 10, 2021: contributor

crACK fa72fce7c948185752a01002000ea511809146ed 👍👍
fanquake merged this on Jun 11, 2021
fanquake closed this on Jun 11, 2021
MarcoFalke deleted the branch on Jun 11, 2021
sidhujag referenced this in commit 5d4e00cdc3 on Jun 11, 2021
gwillen referenced this in commit e39e6e320d on Jun 1, 2022
DrahtBot locked this on Aug 16, 2022

Contributors

Labels