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

Fix for test active_transactions.fork_filter_cleanup #3775

Conversation

dsiganos
Copy link
Contributor

@dsiganos dsiganos commented Apr 2, 2022

The publish filter and the ledger cache count are not synchronised and
therefore the last assert needs to be an assert timely.

resolves #3762

The publish filter and the ledger cache count are not synchronised and
therefore the last assert needs to be an assert timely.

resolves nanocurrency#3762
@dsiganos dsiganos requested review from clemahieu, theohax and thsfs April 2, 2022 11:48
@dsiganos dsiganos merged commit 6397a44 into nanocurrency:develop Apr 2, 2022
@dsiganos dsiganos deleted the fix_active_transactions_fork_filter_cleanup branch April 2, 2022 12:44
@zhyatt zhyatt added the unit test Related to a new, changed or fixed unit test label Apr 2, 2022
@zhyatt zhyatt added this to the V24.0 milestone Apr 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
unit test Related to a new, changed or fixed unit test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

core_test active_transactions.fork_filter_cleanup fails occasionally
3 participants