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.

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

Move only: Move TxOrphange tests to orphange_tests.cpp #24968

Merged
merged 1 commit into from
Apr 26, 2022

Conversation

jnewbery
Copy link
Contributor

PR #21148 moved the orphan transaction handling functionality from net_processing into its own translation unit txorphanage.cpp. The unit tests for that code should be in its own file rather than mixed with the net_processing unit tests in denialofservive_tests.cpp.

@fanquake fanquake added the P2P label Apr 25, 2022
@fanquake fanquake requested a review from ajtowns April 25, 2022 07:48
Copy link

@vincenzopalazzo vincenzopalazzo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK b8f17fb

@DrahtBot
Copy link
Contributor

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

Conflicts

Reviewers, this pull request conflicts with the following ones:

  • #23352 (test: Extend stale_tip_peer_management test by MarcoFalke)
  • #22702 (Add allocator for node based containers by martinus)
  • #21527 (net_processing: lock clean up by ajtowns)

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.

@maflcko maflcko merged commit 1ae65b4 into bitcoin:master Apr 26, 2022
@jnewbery jnewbery deleted the 202204_move_orphanage_tests branch April 26, 2022 08:18
// Distributed under the MIT software license, see the accompanying
// file COPYING or http://www.opensource.org/licenses/mit-license.php.

#include <arith_uint256.h>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

iwyu (@fanquake) will fix this, but for completeness I wanted to mention that you partially reverted 173c796

sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Apr 26, 2022
@bitcoin bitcoin locked and limited conversation to collaborators Apr 26, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants