From 3308e960ceefd7aab56186b854fc98403448cc1d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Alexandre=20Bald=C3=A9?= Date: Mon, 11 Nov 2024 15:03:36 +0000 Subject: [PATCH] Remvoe `require_weight_at_most` from test `Transact`s --- .../people/people-rococo/src/tests/governance.rs | 8 -------- .../people/people-westend/src/tests/governance.rs | 11 ----------- 2 files changed, 19 deletions(-) diff --git a/cumulus/parachains/integration-tests/emulated/tests/people/people-rococo/src/tests/governance.rs b/cumulus/parachains/integration-tests/emulated/tests/people/people-rococo/src/tests/governance.rs index 6bc443903fdb4..95e33618930d4 100644 --- a/cumulus/parachains/integration-tests/emulated/tests/people/people-rococo/src/tests/governance.rs +++ b/cumulus/parachains/integration-tests/emulated/tests/people/people-rococo/src/tests/governance.rs @@ -54,7 +54,6 @@ fn relay_commands_add_registrar() { UnpaidExecution { weight_limit: Unlimited, check_origin: None }, Transact { origin_kind, - require_weight_at_most: Weight::from_parts(5_000_000_000, 500_000), call: add_registrar_call.encode().into(), } ]))), @@ -117,7 +116,6 @@ fn relay_commands_add_registrar_wrong_origin() { UnpaidExecution { weight_limit: Unlimited, check_origin: None }, Transact { origin_kind, - require_weight_at_most: Weight::from_parts(5_000_000_000, 500_000), call: add_registrar_call.encode().into(), } ]))), @@ -204,7 +202,6 @@ fn relay_commands_kill_identity() { // Making the weight's ref time any lower will prevent the XCM from triggering // execution of the intended extrinsic on the People chain - beware of spurious // test failure due to this. - require_weight_at_most: Weight::from_parts(11_000_000_000, 500_000), call: kill_identity_call.encode().into(), } ]))), @@ -268,7 +265,6 @@ fn relay_commands_kill_identity_wrong_origin() { UnpaidExecution { weight_limit: Unlimited, check_origin: None }, Transact { origin_kind, - require_weight_at_most: Weight::from_parts(11_000_000_000, 500_000), call: kill_identity_call.encode().into(), } ]))), @@ -334,7 +330,6 @@ fn relay_commands_add_remove_username_authority() { UnpaidExecution { weight_limit: Unlimited, check_origin: None }, Transact { origin_kind, - require_weight_at_most: Weight::from_parts(500_000_000, 500_000), call: add_username_authority.encode().into(), } ]))), @@ -424,7 +419,6 @@ fn relay_commands_add_remove_username_authority() { UnpaidExecution { weight_limit: Unlimited, check_origin: None }, Transact { origin_kind, - require_weight_at_most: Weight::from_parts(500_000_000, 500_000), call: remove_username_authority.encode().into(), } ]))), @@ -491,7 +485,6 @@ fn relay_commands_add_remove_username_authority_wrong_origin() { UnpaidExecution { weight_limit: Unlimited, check_origin: None }, Transact { origin_kind, - require_weight_at_most: Weight::from_parts(500_000_000, 500_000), call: add_username_authority.encode().into(), } ]))), @@ -543,7 +536,6 @@ fn relay_commands_add_remove_username_authority_wrong_origin() { UnpaidExecution { weight_limit: Unlimited, check_origin: None }, Transact { origin_kind: OriginKind::SovereignAccount, - require_weight_at_most: Weight::from_parts(500_000_000, 500_000), call: remove_username_authority.encode().into(), } ]))), diff --git a/cumulus/parachains/integration-tests/emulated/tests/people/people-westend/src/tests/governance.rs b/cumulus/parachains/integration-tests/emulated/tests/people/people-westend/src/tests/governance.rs index d21b7eeaba36f..3650d43ec1d73 100644 --- a/cumulus/parachains/integration-tests/emulated/tests/people/people-westend/src/tests/governance.rs +++ b/cumulus/parachains/integration-tests/emulated/tests/people/people-westend/src/tests/governance.rs @@ -55,7 +55,6 @@ fn relay_commands_add_registrar() { UnpaidExecution { weight_limit: Unlimited, check_origin: None }, Transact { origin_kind, - require_weight_at_most: Weight::from_parts(5_000_000_000, 500_000), call: add_registrar_call.encode().into(), } ]))), @@ -118,7 +117,6 @@ fn relay_commands_add_registrar_wrong_origin() { UnpaidExecution { weight_limit: Unlimited, check_origin: None }, Transact { origin_kind, - require_weight_at_most: Weight::from_parts(5_000_000_000, 500_000), call: add_registrar_call.encode().into(), } ]))), @@ -209,10 +207,6 @@ fn relay_commands_kill_identity() { UnpaidExecution { weight_limit: Unlimited, check_origin: None }, Transact { origin_kind, - // Making the weight's ref time any lower will prevent the XCM from triggering - // execution of the intended extrinsic on the People chain - beware of spurious - // test failure due to this. - require_weight_at_most: Weight::from_parts(11_000_000_000, 500_000), call: kill_identity_call.encode().into(), } ]))), @@ -274,7 +268,6 @@ fn relay_commands_kill_identity_wrong_origin() { UnpaidExecution { weight_limit: Unlimited, check_origin: None }, Transact { origin_kind, - require_weight_at_most: Weight::from_parts(11_000_000_000, 500_000), call: kill_identity_call.encode().into(), } ]))), @@ -325,7 +318,6 @@ fn relay_commands_add_remove_username_authority() { UnpaidExecution { weight_limit: Unlimited, check_origin: None }, Transact { origin_kind, - require_weight_at_most: Weight::from_parts(500_000_000, 500_000), call: add_username_authority.encode().into(), } ]))), @@ -415,7 +407,6 @@ fn relay_commands_add_remove_username_authority() { UnpaidExecution { weight_limit: Unlimited, check_origin: None }, Transact { origin_kind, - require_weight_at_most: Weight::from_parts(500_000_000, 500_000), call: remove_username_authority.encode().into(), } ]))), @@ -482,7 +473,6 @@ fn relay_commands_add_remove_username_authority_wrong_origin() { UnpaidExecution { weight_limit: Unlimited, check_origin: None }, Transact { origin_kind, - require_weight_at_most: Weight::from_parts(500_000_000, 500_000), call: add_username_authority.encode().into(), } ]))), @@ -534,7 +524,6 @@ fn relay_commands_add_remove_username_authority_wrong_origin() { UnpaidExecution { weight_limit: Unlimited, check_origin: None }, Transact { origin_kind: OriginKind::SovereignAccount, - require_weight_at_most: Weight::from_parts(500_000_000, 500_000), call: remove_username_authority.encode().into(), } ]))),