test: improve torus0 and governance pallet tests (#84)

Closes CHAIN-70

Author: devwckd

pallets/governance/tests/application.rs

@@ -1,5 +1,5 @@
 use pallet_governance::{
-    application::ApplicationStatus, AgentApplications, GlobalGovernanceConfig,
+    application::ApplicationStatus, AgentApplications, GlobalGovernanceConfig, Whitelist,
 };
 use polkadot_sdk::frame_support::assert_err;
 use test_utils::*;
@@ -118,7 +118,7 @@ fn whitelist_executes_application_correctly_remove() {
         zero_min_burn();
 
         let key = 0;
-        let adding_key = 1;
+        let removing_key = 1;
         pallet_governance::Curators::<Test>::insert(key, ());
 
         let proposal_cost = GlobalGovernanceConfig::<Test>::get().agent_application_cost;
@@ -128,19 +128,21 @@ fn whitelist_executes_application_correctly_remove() {
         // first submit an application
         let balance_before = get_balance(key);
 
+        Whitelist::<Test>::set(removing_key, Some(()));
+
         assert_ok!(pallet_governance::Pallet::<Test>::submit_application(
             get_origin(key),
-            adding_key,
+            removing_key,
             data.clone(),
-            false
+            true
         ));
 
         let balance_after = get_balance(key);
         assert_eq!(balance_after, balance_before - proposal_cost);
 
         let mut application_id: u32 = u32::MAX;
         for (_, value) in AgentApplications::<Test>::iter() {
-            assert_eq!(value.agent_key, adding_key);
+            assert_eq!(value.agent_key, removing_key);
             assert_eq!(value.data, data);
             application_id = value.id;
         }
@@ -154,8 +156,8 @@ fn whitelist_executes_application_correctly_remove() {
 
         assert_eq!(balance_after_accept, balance_before);
 
-        assert!(pallet_governance::whitelist::is_whitelisted::<Test>(
-            &adding_key
+        assert!(!pallet_governance::whitelist::is_whitelisted::<Test>(
+            &removing_key
         ));
 
         let application =

pallets/governance/tests/config.rs

@@ -0,0 +1,40 @@
+use polkadot_sdk::{
+    frame_support::assert_err,
+    sp_runtime::{traits::BadOrigin, Percent},
+};
+use test_utils::{assert_ok, get_origin, new_test_ext, RuntimeOrigin, Test};
+
+#[test]
+fn set_emission_params() {
+    new_test_ext().execute_with(|| {
+        assert_ok!(pallet_governance::Pallet::<Test>::set_emission_params(
+            RuntimeOrigin::root(),
+            Percent::from_percent(15),
+            Percent::from_percent(15),
+        ));
+
+        assert_eq!(
+            pallet_emission0::EmissionRecyclingPercentage::<Test>::get(),
+            Percent::from_percent(15)
+        );
+
+        assert_eq!(
+            pallet_governance::TreasuryEmissionFee::<Test>::get(),
+            Percent::from_percent(15)
+        );
+    });
+}
+
+#[test]
+fn set_emission_params_non_root() {
+    new_test_ext().execute_with(|| {
+        assert_err!(
+            pallet_governance::Pallet::<Test>::set_emission_params(
+                get_origin(0),
+                Percent::from_percent(15),
+                Percent::from_percent(15),
+            ),
+            BadOrigin
+        );
+    });
+}

pallets/governance/tests/delegation.rs

@@ -0,0 +1,33 @@
+use pallet_governance::NotDelegatingVotingPower;
+use polkadot_sdk::frame_support::assert_ok;
+use test_utils::{get_origin, new_test_ext, Test};
+
+#[test]
+fn delegates_voting_power_correctly() {
+    new_test_ext().execute_with(|| {
+        const MODULE: u32 = 0;
+
+        assert_ok!(pallet_governance::Pallet::<Test>::enable_vote_delegation(
+            get_origin(MODULE)
+        ));
+
+        assert!(!NotDelegatingVotingPower::<Test>::get().contains(&MODULE));
+    });
+}
+
+#[test]
+fn disable_voting_power_delegation_correctly() {
+    new_test_ext().execute_with(|| {
+        const MODULE: u32 = 0;
+
+        assert_ok!(pallet_governance::Pallet::<Test>::enable_vote_delegation(
+            get_origin(MODULE)
+        ));
+
+        assert_ok!(pallet_governance::Pallet::<Test>::disable_vote_delegation(
+            get_origin(MODULE)
+        ));
+
+        assert!(NotDelegatingVotingPower::<Test>::get().contains(&MODULE));
+    });
+}