Merge pull request #435 from HorizenOfficial/sc_creation_version

HorizenOfficial · Mar 14, 2022 · 3d540f9 · 3d540f9
2 parents bb69ac6 + 3bb2c20
commit 3d540f9
Show file tree

Hide file tree

Showing 99 changed files with 3,041 additions and 743 deletions.
diff --git a/qa/pull-tester/rpc-tests.sh b/qa/pull-tester/rpc-tests.sh
@@ -99,6 +99,7 @@ testScripts=(
   'sc_bwt_request.py'
   'sc_cert_quality_wallet.py'
   'ws_messages.py'
+  'ws_getsidechainversions.py'
   'sc_cert_ceasing_split.py'
   'sc_async_proof_verifier.py'
   'sc_quality_blockchain.py'
@@ -131,6 +132,7 @@ testScripts=(
   'txindex.py'
   'getblockexpanded.py'
   'sc_rpc_cmds_json_output.py'
+  'sc_getscgenesisinfo.py'
 );
 testScriptsExt=(
   'getblocktemplate_longpoll.py'

diff --git a/qa/rpc-tests/getblockexpanded.py b/qa/rpc-tests/getblockexpanded.py
@@ -56,8 +56,14 @@ def run_test(self):
         vk = mcTest.generate_params("sc1")
         constant = generate_random_field_element_hex()
 
-        cmdInput = {'withdrawalEpochLength': EPOCH_LENGTH, 'toaddress': "dada", 'amount': creation_amount, 'wCertVk': vk,
-                    'constant': constant}
+        cmdInput = {
+            "version": 0,
+            'withdrawalEpochLength': EPOCH_LENGTH,
+            'toaddress': "dada",
+            'amount': creation_amount,
+            'wCertVk': vk,
+            'constant': constant
+        }
 
         ret = self.nodes[0].sc_create(cmdInput)
         creating_tx = ret['txid']

diff --git a/qa/rpc-tests/getblockmerkleroots.py b/qa/rpc-tests/getblockmerkleroots.py
@@ -97,8 +97,15 @@ def run_test(self):
         vk = mcTest.generate_params("sc1")
         constant = generate_random_field_element_hex()
 
-        cmdInput = {'withdrawalEpochLength': 123, 'toaddress': "dada", 'amount': 3.0, 'wCertVk': vk,
-                    'customData': "bb" * 1024, 'constant': constant}
+        cmdInput = {
+            'version': 0,
+            'withdrawalEpochLength': 123,
+            'toaddress': "dada",
+            'amount': 3.0,
+            'wCertVk': vk,
+            'customData': "bb" * 1024,
+            'constant': constant
+        }
         ret = self.nodes[0].sc_create(cmdInput)
         scid = ret['scid']
         self.sync_all()

diff --git a/qa/rpc-tests/getblocktemplate_proposals.py b/qa/rpc-tests/getblocktemplate_proposals.py
@@ -161,8 +161,15 @@ def run_test(self):
         vk = mcTest.generate_params("sc1")
         constant = generate_random_field_element_hex()
 
-        cmdInput = {'withdrawalEpochLength': SC_EPOCH_LENGTH, 'toaddress': "dada", 'amount': SC_CREATION_AMOUNT, 'wCertVk': vk,
-                    'customData': "bb" * 1024, 'constant': constant}
+        cmdInput = {
+            'version': 0,
+            'withdrawalEpochLength': SC_EPOCH_LENGTH,
+            'toaddress': "dada",
+            'amount': SC_CREATION_AMOUNT,
+            'wCertVk': vk,
+            'customData': "bb" * 1024,
+            'constant': constant
+        }
         ret = self.nodes[1].sc_create(cmdInput)
         creating_tx = ret['txid']
         scid = ret['scid']

diff --git a/qa/rpc-tests/listtransactions.py b/qa/rpc-tests/listtransactions.py
@@ -361,7 +361,15 @@ def run_test(self):
         vk = mcTest.generate_params("sc1")
         constant = generate_random_field_element_hex()
 
-        cmdInput = {'fromaddress': fromaddr, 'toaddress': sidechain_address, 'amount': sc_creation_amount, 'fee': fee, 'wCertVk': vk, "constant": constant}
+        cmdInput = {
+            'version': 0,
+            'fromaddress': fromaddr,
+            'toaddress': sidechain_address,
+            'amount': sc_creation_amount,
+            'fee': fee,
+            'wCertVk': vk,
+            'constant': constant
+        }
         try:
             res = self.nodes[1].sc_create(cmdInput)
             tx = res['txid']

diff --git a/qa/rpc-tests/merkle_blocks.py b/qa/rpc-tests/merkle_blocks.py
@@ -120,6 +120,7 @@ def run_test(self):
 
         sc_cr = []
         sc_cr.append({
+            "version": 0,
             "epoch_length": sc_epoch_len,
             "amount": sc_cr_amount,
             "address": sc_address,

diff --git a/qa/rpc-tests/rawtransactions.py b/qa/rpc-tests/rawtransactions.py
@@ -221,6 +221,7 @@ def run_test(self):
 
         sc_cr = []
         sc_cr.append({
+            "version": 0,
             "epoch_length": sc_epoch_len,
             "amount": sc_cr_amount,
             "address": sc_address,
@@ -230,6 +231,7 @@ def run_test(self):
         })
 
         sc_cr.append({
+            "version": 0,
             "epoch_length": sc_epoch2_len,
             "amount": sc_cr_amount2,
             "address": sc_address,
@@ -238,8 +240,19 @@ def run_test(self):
             "constant": constant
         })
 
+        # missing sc version
+        sc_cr_without_version = [ {
+            "epoch_length": sc_epoch_len,
+            "amount": sc_cr_amount,
+            "address": sc_address,
+            "wCertVk": vk,
+            "wCeasedVk": cswVk,
+            "constant": constant
+        }]
+
         # too big an epoch (max is 4032)
         sc_cr_bad = [ {
+            "version": 0,
             "epoch_length": 4033,
             "amount": sc_cr_amount2,
             "address": sc_address,
@@ -248,6 +261,15 @@ def run_test(self):
             "constant": constant
         } ]
 
+        #Try create a SC without providing the version
+        print("Try creating a SC without version")
+
+        try:
+            rawtx=self.nodes[0].createrawtransaction([], {}, [], sc_cr_without_version)
+        except JSONRPCException as e:
+            errorString = e.error['message']
+            assert_equal("version" in errorString, True)
+
         #Try create a SC with too big an epoch len
         print("Try creating a SC with an epoch too big")
 
@@ -408,6 +430,7 @@ def run_test(self):
         cswVk2 = self.cswMcTest.generate_params("csw2")
         constant2 = generate_random_field_element_hex()
         sc_cr2 = [{
+            "version": 0,
             "epoch_length": sc_epoch_len,
             "amount": Decimal("4.0"),
             "address": "ccc",

diff --git a/qa/rpc-tests/run_sc_tests.sh b/qa/rpc-tests/run_sc_tests.sh
@@ -4,7 +4,7 @@ set -o pipefail
 
 rm /tmp/result.log
 
-for i in sbh_rpc_cmds.py rawtransactions.py getblocktemplate_proposals.py listtransactions.py ws_messages.py sc_*py
+for i in sbh_rpc_cmds.py rawtransactions.py getblocktemplate_proposals.py listtransactions.py ws_messages.py ws_getsidechainversions.py sc_*py
 do
 	echo "$i ---------------------------" | tee -a /tmp/result.log
 	time stdbuf --output=L python2 ./$i 2>&1 | tee -a /tmp/result.log

diff --git a/qa/rpc-tests/sbh_rpc_cmds.py b/qa/rpc-tests/sbh_rpc_cmds.py
@@ -110,6 +110,7 @@ def run_test(self):
         minconf = 1
         fee = Decimal("0.000025")
         cmdInput = {
+            "version": 0,
             "withdrawalEpochLength": EPOCH_LENGTH,
             "fromaddress": taddr_1,
             "toaddress": sc_toaddress,

diff --git a/qa/rpc-tests/sc_async_proof_verifier.py b/qa/rpc-tests/sc_async_proof_verifier.py
@@ -115,6 +115,7 @@ def run_test(self):
 
         sc_cr = []
         sc_cr.append({
+            "version": 0,
             "epoch_length": sc_epoch_len,
             "amount": sc_cr_amount,
             "address": sc_address,

diff --git a/qa/rpc-tests/sc_big_block.py b/qa/rpc-tests/sc_big_block.py
@@ -8,7 +8,8 @@
 from test_framework.test_framework import MINIMAL_SC_HEIGHT
 from test_framework.util import assert_true, assert_equal, initialize_chain_clean, \
     start_nodes, stop_nodes, wait_bitcoinds, sync_blocks, sync_mempools, connect_nodes_bi, mark_logs, \
-    dump_sc_info, dump_sc_info_record, get_epoch_data, get_spendable, swap_bytes, advance_epoch
+    dump_sc_info, dump_sc_info_record, get_epoch_data, get_spendable, swap_bytes, advance_epoch, \
+    get_field_element_with_padding
 from test_framework.mc_test.mc_test import *
 import os
 import pprint
@@ -160,9 +161,17 @@ def advance_sidechains_epoch(num_of_scs):
         cmtCfg.append([[254*4, 151]])
 
         cmdInput = {
-            'withdrawalEpochLength': EPOCH_LENGTH, 'amount': amount, 'fee': fee,
-            'constant':constant , 'wCertVk': certVk, 'wCeasedVk': cswVk, 'toaddress':"cdcd",
-            'vFieldElementCertificateFieldConfig':feCfg[0], 'vBitVectorCertificateFieldConfig':cmtCfg[0] }
+            'version': 0,
+            'withdrawalEpochLength': EPOCH_LENGTH,
+            'amount': amount,
+            'fee': fee,
+            'constant':constant ,
+            'wCertVk': certVk,
+            'wCeasedVk': cswVk,
+            'toaddress':"cdcd",
+            'vFieldElementCertificateFieldConfig':feCfg[0],
+            'vBitVectorCertificateFieldConfig':cmtCfg[0]
+        }
 
         scids = []
         scc_txs = []
@@ -177,7 +186,7 @@ def advance_sidechains_epoch(num_of_scs):
         vCfe = ["ab000100"]
         vCmt = [BIT_VECTOR_BUF]
 
-        fe1 = "ab000100" + "00000000000000000000000000000000000000000000000000000000"
+        fe1 = get_field_element_with_padding("ab000100", 0)
         fe2 = BIT_VECTOR_FE
 
         proofCfeArray = [fe1, fe2]

diff --git a/qa/rpc-tests/sc_block_partitions.py b/qa/rpc-tests/sc_block_partitions.py
@@ -98,8 +98,13 @@ def create_sc(cmdInput, node):
 
         #-------------------------------------------------------
         cmdInput = {
-            'withdrawalEpochLength': EPOCH_LENGTH, 'amount': amount, 'fee': 0.0001,
-            'constant':constant , 'wCertVk': certVk, 'toaddress':"cdcd",
+            'version': 0,
+            'withdrawalEpochLength': EPOCH_LENGTH,
+            'amount': amount,
+            'fee': 0.0001,
+            'constant':constant ,
+            'wCertVk': certVk,
+            'toaddress':"cdcd",
         }
 
         tx, scid = create_sc(cmdInput, self.nodes[0])

diff --git a/qa/rpc-tests/sc_bwt_request.py b/qa/rpc-tests/sc_bwt_request.py
@@ -99,6 +99,7 @@ def run_test(self):
 
         fee_cr1 = Decimal("0.0002")
         cmdInput = {
+            "version": 0,
             "withdrawalEpochLength":EPOCH_LENGTH,
             "toaddress":"dada",
             "amount":creation_amount1,
@@ -306,6 +307,7 @@ def run_test(self):
         prev_epoch_hash_2 = self.nodes[0].getbestblockhash()
         epoch_len_2 = 10
         cmdInput = {
+            "version": 0,
             "withdrawalEpochLength":epoch_len_2,
             "toaddress":"dada",
             "amount":creation_amount2,
@@ -350,7 +352,7 @@ def run_test(self):
         mark_logs("Node0 creates a tx with a few bwt request and mixed outputs using raw version of cmd", self.nodes, DEBUG_MODE)
         outputs = { self.nodes[0].getnewaddress() :4.998 }
         sc_cr_amount = 1.0
-        sc_cr = [ {"epoch_length":10, "amount":sc_cr_amount, "address":"effe", "wCertVk":vk3, "constant":c3} ]
+        sc_cr = [ {"version": 0, "epoch_length":10, "amount":sc_cr_amount, "address":"effe", "wCertVk":vk3, "constant":c3} ]
         ft_amount_1 = 1.0
         ft_amount_2 = 2.0
         mc_return_address = self.nodes[0].getnewaddress()

diff --git a/qa/rpc-tests/sc_cert_addressindex.py b/qa/rpc-tests/sc_cert_addressindex.py
@@ -89,11 +89,14 @@ def run_test(self):
         vk = mcTest.generate_params("sc1")
         constant = generate_random_field_element_hex()
 
-        cmdInput = {'withdrawalEpochLength': EPOCH_LENGTH,
-                    'toaddress': "dada",
-                    'amount': creation_amount,
-                    'wCertVk': vk,
-                    'constant': constant}
+        cmdInput = {
+            'version': 0,
+            'withdrawalEpochLength': EPOCH_LENGTH,
+            'toaddress': "dada",
+            'amount': creation_amount,
+            'wCertVk': vk,
+            'constant': constant
+        }
 
         ret = self.nodes[0].sc_create(cmdInput)
         creating_tx = ret['txid']

diff --git a/qa/rpc-tests/sc_cert_addrmempool.py b/qa/rpc-tests/sc_cert_addrmempool.py
@@ -112,11 +112,14 @@ def run_test(self):
         constant = generate_random_field_element_hex()
 
         # Create a SC
-        cmdInput = {'withdrawalEpochLength': EPOCH_LENGTH,
-                    'toaddress': "dada",
-                    'amount': creation_amount,
-                    'wCertVk': vk,
-                    'constant': constant}
+        cmdInput = {
+            'version': 0,
+            'withdrawalEpochLength': EPOCH_LENGTH,
+            'toaddress': "dada",
+            'amount': creation_amount,
+            'wCertVk': vk,
+            'constant': constant
+        }
 
         ret = self.nodes[0].sc_create(cmdInput)
         scid = ret['scid']

diff --git a/qa/rpc-tests/sc_cert_base.py b/qa/rpc-tests/sc_cert_base.py
@@ -83,6 +83,7 @@ def run_test(self):
         vk = mcTest.generate_params("sc1")
         constant = generate_random_field_element_hex()
         cmdInput = {
+            "version": 0,
             "withdrawalEpochLength":EPOCH_LENGTH,
             "toaddress":"dada",
             "amount":creation_amount,
@@ -693,6 +694,7 @@ def run_test(self):
         mark_logs("Node0 creates new sidechain", self.nodes, DEBUG_MODE)
         vk2 = mcTest.generate_params("sc2", "cert_no_const")
         cmdInput = {
+            "version": 0,
             "withdrawalEpochLength":EPOCH_LENGTH,
             "toaddress":"dada",
             "amount":creation_amount,

diff --git a/qa/rpc-tests/sc_cert_bt_immature_balances.py b/qa/rpc-tests/sc_cert_bt_immature_balances.py
@@ -113,11 +113,14 @@ def run_test(self):
         mcTest = CertTestUtils(self.options.tmpdir, self.options.srcdir)
         vk = mcTest.generate_params("sc1")
         constant = generate_random_field_element_hex()
-        cmdInput = {'withdrawalEpochLength': EPOCH_LENGTH,
-                    'toaddress': "dada",
-                    'amount': creation_amount,
-                    'wCertVk': vk,
-                    'constant': constant}
+        cmdInput = {
+            'version': 0,
+            'withdrawalEpochLength': EPOCH_LENGTH,
+            'toaddress': "dada",
+            'amount': creation_amount,
+            'wCertVk': vk,
+            'constant': constant
+        }
 
         # Create a SC with a budget of 10 coins
         ret = self.nodes[0].sc_create(cmdInput)

diff --git a/qa/rpc-tests/sc_cert_bwt_amount_rounding.py b/qa/rpc-tests/sc_cert_bwt_amount_rounding.py
@@ -84,8 +84,13 @@ def create_sc(cmdInput, node):
         fee = 0.000025
 
         cmdInput = {
-            'withdrawalEpochLength': EPOCH_LENGTH, 'amount': cr_amount, 'fee': fee,
-            'constant':constant , 'wCertVk': certVk, 'toaddress':"cdcd"
+            'version': 0,
+            'withdrawalEpochLength': EPOCH_LENGTH,
+            'amount': cr_amount,
+            'fee': fee,
+            'constant':constant,
+            'wCertVk': certVk,
+            'toaddress':"cdcd"
         }
 
         tx, scid = create_sc(cmdInput, self.nodes[0])

diff --git a/qa/rpc-tests/sc_cert_ceasing.py b/qa/rpc-tests/sc_cert_ceasing.py
@@ -93,12 +93,13 @@ def run_test(self):
             tag = "sc"+str(i+1)
             vk = mcTest.generate_params(tag)
             cmdInput = {
+                "version": 0,
                 "withdrawalEpochLength": EPOCH_LENGTH,
                 "toaddress": "dada",
                 "amount": creation_amount[i],
                 "wCertVk": vk,
                 "constant": constant,
-                'customData': "abcdef"
+                "customData": "abcdef"
             }
 
             ret = self.nodes[0].sc_create(cmdInput)

diff --git a/qa/rpc-tests/sc_cert_ceasing_sg.py b/qa/rpc-tests/sc_cert_ceasing_sg.py
@@ -89,6 +89,7 @@ def run_test(self):
         #----------------------------------------------------------------------
         vk = mcTest.generate_params("sc1")
         cmdInput = {
+            "version": 0,
             "withdrawalEpochLength": EPOCH_LENGTH,
             "toaddress": "dada",
             "amount": creation_amount,

diff --git a/qa/rpc-tests/sc_cert_ceasing_split.py b/qa/rpc-tests/sc_cert_ceasing_split.py
@@ -101,6 +101,7 @@ def run_test(self):
         constant = generate_random_field_element_hex()
 
         cmdInput = {
+            'version': 0,
             'withdrawalEpochLength': sc_epoch_len,
             'amount': sc_cr_amount,
             'toaddress': sc_address,

diff --git a/qa/rpc-tests/sc_cert_change.py b/qa/rpc-tests/sc_cert_change.py
@@ -75,6 +75,7 @@ def run_test(self):
         vk = mcTest.generate_params("sc1")
         constant = generate_random_field_element_hex()
         cmdInput = {
+            "version": 0,
             "withdrawalEpochLength": EPOCH_LENGTH,
             "toaddress": "dada",
             "amount": creation_amount,