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
Add test for -walletrejectlongchains #9395
Conversation
Tests? |
As the |
@dcousens ok test added |
Awesome, thanks @morcos |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
utACK
self.nodes[0] = start_node(0, self.options.tmpdir, ["-walletrejectlongchains", "-limitancestorcount="+str(2*chainlimit)]) | ||
|
||
# wait for loadmempool | ||
timeout = 10 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
unused value?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oops thanks.. adding the update of timeout
utACK |
utACK 2f882f8 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for fixing this.
utACK 2f882f899904ab51412da463fd1854fd24426d21
ACK-before-it-was-conflicted needs rebase. |
This was fixed in #9283, but still is useful to add the test, so kept that commit . (and changed PR title) |
re-ACK ffeb195 |
ffeb195 add test for -walletrejectlongchains (Alex Morcos)
ffeb195 add test for -walletrejectlongchains (Alex Morcos)
ffeb195 add test for -walletrejectlongchains (Alex Morcos)
ffeb195 add test for -walletrejectlongchains (Alex Morcos)
txNew
was moved from to createwtxNew
so we don't really mean to be accessing it afterwards.The current code is not actually calculating mempool ancestors so
-walletrejectlongchains
is not having any effect.