-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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: remove mn_rr fork usage from functional test of EHF #6270
Conversation
test/functional/feature_mnehf.py
Outdated
self.log.info("activate MN_RR also by enabling spork 24") | ||
assert_equal(get_bip9_details(node, 'mn_rr')['status'], 'defined') | ||
self.log.info("Test spork 24 (EHF)") | ||
assert_equal(get_bip9_details(node, 'testdummy')['status'], 'defined') |
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.
assert_equal(get_bip9_details(node, 'testdummy')['status'], 'defined') | |
self.check_fork('defined') |
should also drop
self.log.info(f"bip9: {get_bip9_details(node, 'mn_rr')}")
assert_equal(get_bip9_details(node, 'mn_rr')['status'], 'active')
at the end of the test
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 056d869
nit: PR title is misleading
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 056d869
Issue being fixed or feature implemented
Using mn_rr in feature_mnehf.py is a blocker for burying mn_rr fork.
What was done?
Removed useless conditions, uses testdummy fork instead mn_rr in ehf functional test.
How Has This Been Tested?
Run it
test/functional/feature_mnehf.py
Breaking Changes
N/A
Checklist: