Merge #15696: [qa] test_runner: Move feature_pruning to base tests

fafb55e2c2 [qa] test_runner: Move feature_pruning to base tests (MarcoFalke)
8728a66782 [tests] fix block time in feature_pruning.py (John Newbery)

Pull request description:

ACKs for commit fafb55:

Tree-SHA512: 88abef94379fbad6629da11dccb080d5f0644490d6f2cc2756a33fac34bcf72e84245cef596dfae5a40f7a99b3f4da0dd85d306d4c1b452d310d3f36eef75a8b
This commit is contained in:
MarcoFalke 2019-04-30 10:09:51 -04:00 committed by Munkybooty
parent 843a42d750
commit e607643bb7
2 changed files with 8 additions and 7 deletions

View File

@ -28,16 +28,17 @@ def mine_large_blocks(node, n):
# followed by 950k of OP_NOP. This would be non-standard in a non-coinbase # followed by 950k of OP_NOP. This would be non-standard in a non-coinbase
# transaction but is consensus valid. # transaction but is consensus valid.
# Set the nTime if this is the first time this function has been called.
# A static variable ensures that time is monotonicly increasing and is therefore
# different for each block created => blockhash is unique.
if "nTimes" not in mine_large_blocks.__dict__:
mine_large_blocks.nTime = 0
# Get the block parameters for the first block # Get the block parameters for the first block
big_script = CScript([OP_RETURN] + [OP_NOP] * 950000) big_script = CScript([OP_RETURN] + [OP_NOP] * 950000)
best_block = node.getblock(node.getbestblockhash()) best_block = node.getblock(node.getbestblockhash())
height = int(best_block["height"]) + 1 height = int(best_block["height"]) + 1
try: mine_large_blocks.nTime = max(mine_large_blocks.nTime, int(best_block["time"])) + 1
# Static variable ensures that time is monotonicly increasing and is therefore
# different for each block created => blockhash is unique.
mine_large_blocks.nTime = min(mine_large_blocks.nTime, int(best_block["time"])) + 1
except AttributeError:
mine_large_blocks.nTime = int(best_block["time"]) + 1
previousblockhash = int(best_block["hash"], 16) previousblockhash = int(best_block["hash"], 16)
for _ in range(n): for _ in range(n):

View File

@ -72,6 +72,7 @@ BASE_SCRIPTS = [
'feature_dip3_deterministicmns.py', # NOTE: needs dash_hash to pass 'feature_dip3_deterministicmns.py', # NOTE: needs dash_hash to pass
'feature_block_reward_reallocation.py', 'feature_block_reward_reallocation.py',
'feature_llmq_data_recovery.py', 'feature_llmq_data_recovery.py',
'feature_pruning.py', # NOTE: Prune mode is incompatible with -txindex, should work with governance validation disabled though.
'feature_fee_estimation.py', 'feature_fee_estimation.py',
'wallet_hd.py', 'wallet_hd.py',
'wallet_backup.py', 'wallet_backup.py',
@ -230,7 +231,6 @@ BASE_SCRIPTS = [
EXTENDED_SCRIPTS = [ EXTENDED_SCRIPTS = [
# These tests are not run by default. # These tests are not run by default.
# Longest test should go first, to favor running tests in parallel # Longest test should go first, to favor running tests in parallel
'feature_pruning.py', # NOTE: Prune mode is incompatible with -txindex, should work with governance validation disabled though.
'feature_dbcrash.py', 'feature_dbcrash.py',
] ]