Seems odd to have this one in a separate file. Having related tests in one file makes it easier to edit them during addrman refactoring. It also makes browsing easier.
fuzz: Move all addrman fuzz targets to one file #22940
pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:2109-fuzzMergeAddrman changing 3 files +11 −31-
MarcoFalke commented at 2:22 PM on September 10, 2021: member
-
faa861bd0c
fuzz: Move all addrman fuzz targets to one file
Can be reviewed with --color-moved=dimmed-zebra
- MarcoFalke added the label Refactoring on Sep 10, 2021
- MarcoFalke added the label Tests on Sep 10, 2021
-
DrahtBot commented at 2:41 AM on September 11, 2021: member
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--174a7506f384e20aa4161008e828411d-->
Conflicts
Reviewers, this pull request conflicts with the following ones:
- #22950 ([p2p] Pimpl AddrMan to abstract implementation details by amitiuttarwar)
- #22910 ([RFC] Encapsulate asmap in NetGroupManager by jnewbery)
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.
- MarcoFalke closed this on Sep 15, 2021
- MarcoFalke deleted the branch on Sep 15, 2021
-
jnewbery commented at 2:49 PM on September 21, 2021: member
Why close?
-
MarcoFalke commented at 2:53 PM on September 21, 2021: member
Sorry, forgot to say that #22734 ate this commit.
- DrahtBot locked this on Oct 30, 2022
Contributors
Labels