Skip to content

Commit

Permalink
test: Remove BIP34, BIP65, BIP66, DERSIG and CLTV deployments
Browse files Browse the repository at this point in the history
  • Loading branch information
lateminer committed Jan 24, 2024
1 parent 3a86a92 commit 313f661
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 28 deletions.
1 change: 0 additions & 1 deletion test/functional/feature_block.py
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,6 @@ def set_test_params(self):
self.setup_clean_chain = True
self.extra_args = [[
'-acceptnonstdtxn=1', # This is a consensus block test, we don't care about tx policy
'-testactivationheight=bip34@2',
]]

def run_test(self):
Expand Down
11 changes: 3 additions & 8 deletions test/functional/feature_cltv.py
Original file line number Diff line number Diff line change
Expand Up @@ -84,21 +84,16 @@ class BIP65Test(BitcoinTestFramework):
def set_test_params(self):
self.num_nodes = 1
self.extra_args = [[
f'-testactivationheight=cltv@{CLTV_HEIGHT}',
'-whitelist=noban@127.0.0.1',
f'-whitelist=noban@127.0.0.1',
'-par=1', # Use only one script thread to get the exact reject reason for testing
'-acceptnonstdtxn=1', # cltv_invalidate is nonstandard
]]
self.setup_clean_chain = True
self.rpc_timeout = 480

def test_cltv_info(self, *, is_active):
assert_equal(self.nodes[0].getdeploymentinfo()['deployments']['bip65'], {
"active": is_active,
"height": CLTV_HEIGHT,
"type": "buried",
},
)
# Blackcoin: assume that CLTV is always active
pass

def run_test(self):
peer = self.nodes[0].add_p2p_connection(P2PInterface())
Expand Down
12 changes: 3 additions & 9 deletions test/functional/feature_dersig.py
Original file line number Diff line number Diff line change
Expand Up @@ -48,8 +48,7 @@ class BIP66Test(BitcoinTestFramework):
def set_test_params(self):
self.num_nodes = 1
self.extra_args = [[
f'-testactivationheight=dersig@{DERSIG_HEIGHT}',
'-whitelist=noban@127.0.0.1',
f'-whitelist=noban@127.0.0.1',
'-par=1', # Use only one script thread to get the exact log msg for testing
]]
self.setup_clean_chain = True
Expand All @@ -60,13 +59,8 @@ def create_tx(self, input_txid):
return self.miniwallet.create_self_transfer(utxo_to_spend=utxo_to_spend)['tx']

def test_dersig_info(self, *, is_active):
assert_equal(self.nodes[0].getdeploymentinfo()['deployments']['bip66'],
{
"active": is_active,
"height": DERSIG_HEIGHT,
"type": "buried",
},
)
# Blackcoin: assume that DERSIG is always active
pass

def run_test(self):
peer = self.nodes[0].add_p2p_connection(P2PInterface())
Expand Down
10 changes: 0 additions & 10 deletions test/functional/rpc_blockchain.py
Original file line number Diff line number Diff line change
Expand Up @@ -167,10 +167,6 @@ def _test_getblockchaininfo(self):
extra_args=['-testactivationheight=name@2'],
expected_msg='Error: Invalid name (name@2) for -testactivationheight=name@height.',
)
self.nodes[0].assert_start_raises_init_error(
extra_args=['-testactivationheight=bip34@-2'],
expected_msg='Error: Invalid height value (bip34@-2) for -testactivationheight=name@height.',
)
self.nodes[0].assert_start_raises_init_error(
extra_args=['-testactivationheight='],
expected_msg='Error: Invalid format () for -testactivationheight=name@height.',
Expand Down Expand Up @@ -198,9 +194,6 @@ def check_signalling_deploymentinfo_result(self, gdi_result, height, blockhash,
"hash": blockhash,
"height": height,
"deployments": {
'bip34': {'type': 'buried', 'active': True, 'height': 2},
'bip66': {'type': 'buried', 'active': True, 'height': 3},
'bip65': {'type': 'buried', 'active': True, 'height': 4},
'csv': {'type': 'buried', 'active': True, 'height': 5},
'segwit': {'type': 'buried', 'active': True, 'height': 6},
'testdummy': {
Expand Down Expand Up @@ -247,9 +240,6 @@ def _test_getdeploymentinfo(self):
self.log.info("Test getdeploymentinfo")
self.stop_node(0)
self.start_node(0, extra_args=[
'-testactivationheight=bip34@2',
'-testactivationheight=dersig@3',
'-testactivationheight=cltv@4',
'-testactivationheight=csv@5',
'-testactivationheight=segwit@6',
])
Expand Down

0 comments on commit 313f661

Please sign in to comment.