Merge pull request #3153 from IntersectMBO/refactor-common-build-methods

mkoura · web-flow · commit 74df3060fa32 · 2025-10-15T12:38:12.000+02:00
This PR refactors basics build_and_submit tx.
diff --git a/cardano_node_tests/tests/test_pools.py b/cardano_node_tests/tests/test_pools.py
@@ -2627,7 +2627,7 @@ def pools(
             cluster_obj=cluster,
             name_template=f"{temp_template}_delegate_reward_addrs",
             src_address=pool_owners[0].payment.address,
-            use_build_cmd=True,
+            build_method=clusterlib_utils.BuildMethods.BUILD,
             tx_files=tx_files,
             deposit=deposit_address_amt,
         )
diff --git a/cardano_node_tests/tests/tests_conway/test_guardrails.py b/cardano_node_tests/tests/tests_conway/test_guardrails.py
@@ -278,7 +278,7 @@ def propose_param_changes(
         cluster_obj=cluster,
         name_template=f"{temp_template}_action",
         src_address=cluster_with_constitution.payment_addr.address,
-        use_build_cmd=False,
+        build_method=clusterlib_utils.BuildMethods.BUILD_RAW,
         tx_files=tx_files,
         complex_proposals=[proposal_script],
         deposit=deposit_amt,
diff --git a/cardano_node_tests/tests/tests_conway/test_hardfork.py b/cardano_node_tests/tests/tests_conway/test_hardfork.py
@@ -117,7 +117,7 @@ def test_hardfork(
             cluster_obj=cluster,
             name_template=f"{temp_template}_action",
             src_address=pool_user_lg.payment.address,
-            use_build_cmd=True,
+            build_method=clusterlib_utils.BuildMethods.BUILD,
             tx_files=tx_files_action,
         )
 
diff --git a/cardano_node_tests/tests/tests_conway/test_no_confidence.py b/cardano_node_tests/tests/tests_conway/test_no_confidence.py
@@ -142,7 +142,7 @@ def test_no_confidence_action(
             cluster_obj=cluster,
             name_template=f"{temp_template}_action",
             src_address=pool_user_lg.payment.address,
-            use_build_cmd=True,
+            build_method=clusterlib_utils.BuildMethods.BUILD,
             tx_files=tx_files_action,
         )
 
diff --git a/cardano_node_tests/tests/tests_conway/test_treasury_donation.py b/cardano_node_tests/tests/tests_conway/test_treasury_donation.py
@@ -158,7 +158,7 @@ def test_dbsync_transfer_treasury_donation(
             current_treasury_value=treasury_val,
             treasury_donation=amount,
             change_address=payment_addr_singleton.address,
-            use_build_cmd=True,
+            build_method=clusterlib_utils.BuildMethods.BUILD,
             tx_files=tx_files,
         )
 
diff --git a/cardano_node_tests/tests/tests_plutus_v3/test_mint_build.py b/cardano_node_tests/tests/tests_plutus_v3/test_mint_build.py
@@ -121,7 +121,7 @@ def run_scenario(
                 cluster_obj=cluster_obj,
                 name_template=f"{temp_template}_step2",
                 src_address=payment_addr.address,
-                use_build_cmd=True,
+                build_method=clusterlib_utils.BuildMethods.BUILD,
                 tx_files=tx_files_step2,
                 txins=mint_utxos,
                 txouts=txouts_step2,

Original file line number	Diff line number	Diff line change
`@@ -2627,7 +2627,7 @@ def pools(`
`2627`	`2627`	`cluster_obj=cluster,`
`2628`	`2628`	`name_template=f"{temp_template}_delegate_reward_addrs",`
`2629`	`2629`	`src_address=pool_owners[0].payment.address,`
`2630`		`- use_build_cmd=True,`
	`2630`	`+ build_method=clusterlib_utils.BuildMethods.BUILD,`
`2631`	`2631`	`tx_files=tx_files,`
`2632`	`2632`	`deposit=deposit_address_amt,`
`2633`	`2633`	`)`
Original file line number	Diff line number	Diff line change
`@@ -117,7 +117,7 @@ def test_hardfork(`
`117`	`117`	`cluster_obj=cluster,`
`118`	`118`	`name_template=f"{temp_template}_action",`
`119`	`119`	`src_address=pool_user_lg.payment.address,`
`120`		`- use_build_cmd=True,`
	`120`	`+ build_method=clusterlib_utils.BuildMethods.BUILD,`
`121`	`121`	`tx_files=tx_files_action,`
`122`	`122`	`)`
`123`	`123`
Original file line number	Diff line number	Diff line change
`@@ -142,7 +142,7 @@ def test_no_confidence_action(`
`142`	`142`	`cluster_obj=cluster,`
`143`	`143`	`name_template=f"{temp_template}_action",`
`144`	`144`	`src_address=pool_user_lg.payment.address,`
`145`		`- use_build_cmd=True,`
	`145`	`+ build_method=clusterlib_utils.BuildMethods.BUILD,`
`146`	`146`	`tx_files=tx_files_action,`
`147`	`147`	`)`
`148`	`148`
Original file line number	Diff line number	Diff line change
`@@ -158,7 +158,7 @@ def test_dbsync_transfer_treasury_donation(`
`158`	`158`	`current_treasury_value=treasury_val,`
`159`	`159`	`treasury_donation=amount,`
`160`	`160`	`change_address=payment_addr_singleton.address,`
`161`		`- use_build_cmd=True,`
	`161`	`+ build_method=clusterlib_utils.BuildMethods.BUILD,`
`162`	`162`	`tx_files=tx_files,`
`163`	`163`	`)`
`164`	`164`