fix the tanssi build & tests

OpenZeppelin · Dec 27, 2024 · a32b7a0 · a32b7a0
1 parent 9158633
commit a32b7a0
Show file tree

Hide file tree

Showing 8 changed files with 56 additions and 22 deletions.
diff --git a/evm-template/runtime/src/configs/mod.rs b/evm-template/runtime/src/configs/mod.rs
@@ -120,8 +120,6 @@ impl SystemConfig for OpenZeppelinRuntime {
     type SlotDuration = ConstU64<SLOT_DURATION>;
     type Version = Version;
 }
-#[cfg(feature = "tanssi")]
-impl TanssiConfig for OpenZeppelinRuntime {}
 #[cfg(not(feature = "tanssi"))]
 impl ConsensusConfig for OpenZeppelinRuntime {
     type CollatorSelectionUpdateOrigin = CollatorSelectionUpdateOrigin;
@@ -215,12 +213,17 @@ impl AssetsConfig for OpenZeppelinRuntime {
     type ForeignAssetModifierOrigin = EnsureRoot<AccountId>;
     type WeightToFee = WeightToFee;
 }
+#[cfg(feature = "tanssi")]
+impl TanssiConfig for OpenZeppelinRuntime {
+    type AuthorInherent = pallet_author_inherent::weights::SubstrateWeight<Runtime>;
+    type AuthoritiesNothing = pallet_cc_authorities_noting::weights::SubstrateWeight<Runtime>;
+}
 impl_openzeppelin_assets!(OpenZeppelinRuntime);
 impl_openzeppelin_system!(OpenZeppelinRuntime);
 #[cfg(not(feature = "tanssi"))]
 impl_openzeppelin_consensus!(OpenZeppelinRuntime);
 #[cfg(feature = "tanssi")]
-impl_openzeppelin_tanssi!();
+impl_openzeppelin_tanssi!(OpenZeppelinRuntime);
 impl_openzeppelin_governance!(OpenZeppelinRuntime);
 impl_openzeppelin_xcm!(OpenZeppelinRuntime);
 impl_openzeppelin_evm!(OpenZeppelinRuntime);

diff --git a/evm-template/runtime/src/types.rs b/evm-template/runtime/src/types.rs
@@ -206,6 +206,7 @@ mod test {
         use crate::{types::ProxyType, AssetManager, RuntimeCall};
 
         #[test]
+        #[cfg(not(feature = "tanssi"))]
         fn test_filter_any() {
             let call = RuntimeCall::CollatorSelection(
                 pallet_collator_selection::Call::set_desired_candidates { max: 10 },
@@ -215,6 +216,7 @@ mod test {
         }
 
         #[test]
+        #[cfg(not(feature = "tanssi"))]
         fn test_filter_nontransfer() {
             let proxy_type = ProxyType::NonTransfer;
             let valid_call = RuntimeCall::CollatorSelection(
@@ -227,6 +229,7 @@ mod test {
         }
 
         #[test]
+        #[cfg(not(feature = "tanssi"))]
         fn test_filter_cancel_proxy() {
             let proxy_type = ProxyType::CancelProxy;
             let invalid_call = RuntimeCall::CollatorSelection(
@@ -241,6 +244,7 @@ mod test {
         }
 
         #[test]
+        #[cfg(not(feature = "tanssi"))]
         fn test_filter_collator() {
             let proxy_type = ProxyType::Collator;
             let valid_call = RuntimeCall::CollatorSelection(

diff --git a/evm-template/runtime/tests/constants_test.rs b/evm-template/runtime/tests/constants_test.rs
@@ -209,6 +209,7 @@ mod runtime_tests {
     }
 
     #[test]
+    #[cfg(not(feature = "tanssi"))]
     fn session_constants() {
         assert_eq!(configs::Period::get(), 6 * HOURS);
 
@@ -217,6 +218,7 @@ mod runtime_tests {
 
     #[test]
     #[allow(clippy::assertions_on_constants)]
+    #[cfg(not(feature = "tanssi"))]
     fn aura_constants() {
         #[cfg(not(feature = "async-backing"))]
         assert!(!<Runtime as pallet_aura::Config>::AllowMultipleBlocksPerSlot::get());
@@ -227,6 +229,7 @@ mod runtime_tests {
     }
 
     #[test]
+    #[cfg(not(feature = "tanssi"))]
     fn collator_selection_constants() {
         let pallet_id_to_string = |id: PalletId| -> String {
             core::str::from_utf8(&id.0).unwrap_or_default().to_string()

diff --git a/evm-template/runtime/tests/storage.rs b/evm-template/runtime/tests/storage.rs
@@ -1,8 +1,9 @@
 // Storage indices integration checks
+#[cfg(not(feature = "tanssi"))]
+use evm_runtime_template::{Aura, AuraExt, Authorship, CollatorSelection, Session};
 use evm_runtime_template::{
-    Aura, AuraExt, Authorship, Balances, CollatorSelection, CumulusXcm, MessageQueue, Multisig,
-    ParachainInfo, ParachainSystem, PolkadotXcm, Proxy, Runtime, Session, Sudo, System, Timestamp,
-    TransactionPayment, XcmpQueue,
+    Balances, CumulusXcm, MessageQueue, Multisig, ParachainInfo, ParachainSystem, PolkadotXcm,
+    Proxy, Runtime, Sudo, System, Timestamp, TransactionPayment, XcmpQueue,
 };
 use frame_support::traits::PalletInfo;
 
@@ -21,13 +22,17 @@ fn verify_pallet_prefixes() {
     assert_pallet_prefix::<TransactionPayment>("TransactionPayment");
     assert_pallet_prefix::<Sudo>("Sudo");
     assert_pallet_prefix::<Multisig>("Multisig");
-    assert_pallet_prefix::<Authorship>("Authorship");
-    assert_pallet_prefix::<CollatorSelection>("CollatorSelection");
-    assert_pallet_prefix::<Session>("Session");
-    assert_pallet_prefix::<Aura>("Aura");
-    assert_pallet_prefix::<AuraExt>("AuraExt");
     assert_pallet_prefix::<XcmpQueue>("XcmpQueue");
     assert_pallet_prefix::<PolkadotXcm>("PolkadotXcm");
     assert_pallet_prefix::<CumulusXcm>("CumulusXcm");
     assert_pallet_prefix::<MessageQueue>("MessageQueue");
+
+    #[cfg(not(feature = "tanssi"))]
+    {
+        assert_pallet_prefix::<Authorship>("Authorship");
+        assert_pallet_prefix::<CollatorSelection>("CollatorSelection");
+        assert_pallet_prefix::<Session>("Session");
+        assert_pallet_prefix::<Aura>("Aura");
+        assert_pallet_prefix::<AuraExt>("AuraExt");
+    }
 }
diff --git a/generic-template/runtime/src/configs/mod.rs b/generic-template/runtime/src/configs/mod.rs
@@ -31,9 +31,7 @@ use frame_system::{
 pub use governance::origins::pallet_custom_origins;
 use governance::{origins::Treasurer, tracks, Spender, WhitelistedCaller};
 use openzeppelin_pallet_abstractions::{
-    impl_openzeppelin_assets, impl_openzeppelin_governance, impl_openzeppelin_system,
-    impl_openzeppelin_xcm, AssetsConfig, AssetsWeight, GovernanceConfig, GovernanceWeight,
-    SystemConfig, SystemWeight, XcmConfig, XcmWeight,
+    impl_openzeppelin_assets, impl_openzeppelin_governance, impl_openzeppelin_system, impl_openzeppelin_xcm, AssetsConfig, AssetsWeight, GovernanceConfig, GovernanceWeight, SystemConfig, SystemWeight, XcmConfig, XcmWeight
 };
 #[cfg(not(feature = "tanssi"))]
 use openzeppelin_pallet_abstractions::{
@@ -189,6 +187,11 @@ impl AssetsConfig for OpenZeppelinRuntime {
     type ForeignAssetModifierOrigin = EnsureRoot<AccountId>;
     type WeightToFee = WeightToFee;
 }
+#[cfg(feature="tanssi")]
+impl TanssiConfig for OpenZeppelinRuntime {
+    type AuthorInherent = pallet_author_inherent::weights::SubstrateWeight<Runtime>;
+    type AuthoritiesNothing = pallet_cc_authorities_noting::weights::SubstrateWeight<Runtime>;
+}
 impl_openzeppelin_system!(OpenZeppelinRuntime);
 #[cfg(not(feature = "tanssi"))]
 impl_openzeppelin_consensus!(OpenZeppelinRuntime);
@@ -197,4 +200,4 @@ impl_openzeppelin_xcm!(OpenZeppelinRuntime);
 impl_openzeppelin_assets!(OpenZeppelinRuntime);
 
 #[cfg(feature = "tanssi")]
-impl_openzeppelin_tanssi!();
+impl_openzeppelin_tanssi!(OpenZeppelinRuntime);
diff --git a/generic-template/runtime/src/types.rs b/generic-template/runtime/src/types.rs
@@ -207,6 +207,7 @@ mod test {
         use crate::{types::ProxyType, AssetManager, RuntimeCall};
 
         #[test]
+        #[cfg(not(feature = "tanssi"))]
         fn test_filter_any() {
             let call = RuntimeCall::CollatorSelection(
                 pallet_collator_selection::Call::set_desired_candidates { max: 10 },
@@ -216,6 +217,7 @@ mod test {
         }
 
         #[test]
+        #[cfg(not(feature = "tanssi"))]
         fn test_filter_nontransfer() {
             let proxy_type = ProxyType::NonTransfer;
             let valid_call = RuntimeCall::CollatorSelection(
@@ -228,6 +230,7 @@ mod test {
         }
 
         #[test]
+        #[cfg(not(feature = "tanssi"))]
         fn test_filter_cancel_proxy() {
             let proxy_type = ProxyType::CancelProxy;
             let invalid_call = RuntimeCall::CollatorSelection(
@@ -242,6 +245,7 @@ mod test {
         }
 
         #[test]
+        #[cfg(not(feature = "tanssi"))]
         fn test_filter_collator() {
             let proxy_type = ProxyType::Collator;
             let valid_call = RuntimeCall::CollatorSelection(

diff --git a/generic-template/runtime/tests/constants_test.rs b/generic-template/runtime/tests/constants_test.rs
@@ -204,6 +204,7 @@ mod runtime_tests {
     }
 
     #[test]
+    #[cfg(not(feature = "tanssi"))]
     fn session_constants() {
         assert_eq!(configs::Period::get(), 6 * HOURS);
 
@@ -212,6 +213,7 @@ mod runtime_tests {
 
     #[test]
     #[allow(clippy::assertions_on_constants)]
+    #[cfg(not(feature = "tanssi"))]
     fn aura_constants() {
         #[cfg(not(feature = "async-backing"))]
         assert!(!<Runtime as pallet_aura::Config>::AllowMultipleBlocksPerSlot::get());
@@ -222,6 +224,7 @@ mod runtime_tests {
     }
 
     #[test]
+    #[cfg(not(feature = "tanssi"))]
     fn collator_selection_constants() {
         let pallet_id_to_string = |id: PalletId| -> String {
             core::str::from_utf8(&id.0).unwrap_or_default().to_string()

diff --git a/generic-template/runtime/tests/storage.rs b/generic-template/runtime/tests/storage.rs
@@ -1,10 +1,14 @@
 // Storage indices integration checks
 use frame_support::traits::PalletInfo;
 use generic_runtime_template::{
-    Aura, AuraExt, Authorship, Balances, CollatorSelection, CumulusXcm, MessageQueue, Multisig,
-    ParachainInfo, ParachainSystem, PolkadotXcm, Proxy, Runtime, Session, Sudo, System, Timestamp,
+    Balances, CumulusXcm, MessageQueue, Multisig,
+    ParachainInfo, ParachainSystem, PolkadotXcm, Proxy, Runtime,  Sudo, System, Timestamp,
     TransactionPayment, XcmpQueue,
 };
+#[cfg(not(feature = "tanssi"))]
+use generic_runtime_template::{
+    Aura, AuraExt, Authorship, CollatorSelection, Session
+};
 
 fn assert_pallet_prefix<P: 'static>(name: &str) {
     assert_eq!(<Runtime as frame_system::Config>::PalletInfo::name::<P>(), Some(name));
@@ -21,13 +25,18 @@ fn verify_pallet_prefixes() {
     assert_pallet_prefix::<TransactionPayment>("TransactionPayment");
     assert_pallet_prefix::<Sudo>("Sudo");
     assert_pallet_prefix::<Multisig>("Multisig");
-    assert_pallet_prefix::<Authorship>("Authorship");
-    assert_pallet_prefix::<CollatorSelection>("CollatorSelection");
-    assert_pallet_prefix::<Session>("Session");
-    assert_pallet_prefix::<Aura>("Aura");
-    assert_pallet_prefix::<AuraExt>("AuraExt");
     assert_pallet_prefix::<XcmpQueue>("XcmpQueue");
     assert_pallet_prefix::<PolkadotXcm>("PolkadotXcm");
     assert_pallet_prefix::<CumulusXcm>("CumulusXcm");
     assert_pallet_prefix::<MessageQueue>("MessageQueue");
+
+    #[cfg(not(feature = "tanssi"))]
+    {
+
+        assert_pallet_prefix::<Authorship>("Authorship");
+        assert_pallet_prefix::<Aura>("Aura");
+        assert_pallet_prefix::<AuraExt>("AuraExt");
+        assert_pallet_prefix::<CollatorSelection>("CollatorSelection");
+        assert_pallet_prefix::<Session>("Session");
+    }
 }