Add test cases for code coverage (#719)

* Add test cases
* More test cases
* One more simple test case
* Remove removed staking pallet
* Update CI script
* Auto format

.github/workflows/ci.yml

@@ -89,7 +89,7 @@ jobs:
       - name: Install cargo-llvm-cov
         uses: taiki-e/install-action@cargo-llvm-cov
       - name: Test and gather coverage
-        run: cargo llvm-cov --lcov --output-path lcov.info --all-features --workspace --exclude runtimes-eden --exclude nodle-parachain  
+        run: cargo llvm-cov --lcov --output-path lcov.info --all-features --workspace --exclude nodle-parachain
       - name: Upload to Codecov
         uses: codecov/[email protected]
         with:

pallets/allocations/src/tests.rs

@@ -704,3 +704,30 @@ fn no_issuance() {
 fn test_releases_default() {
 	assert_eq!(Releases::default(), Releases::V0);
 }
+
+#[test]
+fn check_weight_info() {
+	use crate::weights::SubstrateWeight;
+	use frame_support::pallet_prelude::Weight;
+
+	assert_ne!(<()>::set_curve_starting_block(), Weight::from_ref_time(0));
+	assert_ne!(
+		SubstrateWeight::<Test>::set_curve_starting_block(),
+		Weight::from_ref_time(0)
+	);
+
+	assert_ne!(<()>::checked_update_session_quota(), Weight::from_ref_time(0));
+	assert_ne!(
+		SubstrateWeight::<Test>::checked_update_session_quota(),
+		Weight::from_ref_time(0)
+	);
+
+	assert_ne!(<()>::allocate(100), Weight::from_ref_time(0));
+	assert_ne!(SubstrateWeight::<Test>::allocate(100), Weight::from_ref_time(0));
+
+	assert_ne!(<()>::calc_quota(), Weight::from_ref_time(0));
+	assert_ne!(SubstrateWeight::<Test>::calc_quota(), Weight::from_ref_time(0));
+
+	assert_ne!(<()>::renew_quota(), Weight::from_ref_time(0));
+	assert_ne!(SubstrateWeight::<Test>::renew_quota(), Weight::from_ref_time(0));
+}

pallets/grants/src/lib.rs

@@ -307,23 +307,6 @@ pub mod pallet {
 	}
 }
 
-#[cfg(feature = "std")]
-impl<T: Config> GenesisConfig<T> {
-	/// Direct implementation of `GenesisBuild::build_storage`.
-	///
-	/// Kept in order not to break dependency.
-	pub fn build_storage(&self) -> Result<sp_runtime::Storage, String> {
-		<Self as GenesisBuild<T>>::build_storage(self)
-	}
-
-	/// Direct implementation of `GenesisBuild::assimilate_storage`.
-	///
-	/// Kept in order not to break dependency.
-	pub fn assimilate_storage(&self, storage: &mut sp_runtime::Storage) -> Result<(), String> {
-		<Self as GenesisBuild<T>>::assimilate_storage(self, storage)
-	}
-}
-
 pub const VESTING_LOCK_ID: LockIdentifier = *b"nvesting";
 
 impl<T: Config> Pallet<T> {

pallets/reserve/src/lib.rs

@@ -158,23 +158,6 @@ pub mod pallet {
 	}
 }
 
-#[cfg(feature = "std")]
-impl<T: Config<I>, I: 'static> GenesisConfig<T, I> {
-	/// Direct implementation of `GenesisBuild::build_storage`.
-	///
-	/// Kept in order not to break dependency.
-	pub fn build_storage(&self) -> Result<sp_runtime::Storage, String> {
-		<Self as GenesisBuild<T, I>>::build_storage(self)
-	}
-
-	/// Direct implementation of `GenesisBuild::assimilate_storage`.
-	///
-	/// Kept in order not to break dependency.
-	pub fn assimilate_storage(&self, storage: &mut sp_runtime::Storage) -> Result<(), String> {
-		<Self as GenesisBuild<T, I>>::assimilate_storage(self, storage)
-	}
-}
-
 impl<T: Config<I>, I: 'static> WithAccountId<T::AccountId> for Pallet<T, I> {
 	fn account_id() -> T::AccountId {
 		T::PalletId::get().into_account_truncating()

pallets/reserve/src/tests.rs

@@ -121,6 +121,17 @@ fn spend_error_if_bad_origin() {
 	})
 }
 
+#[test]
+fn check_weight_info() {
+	use crate::weights::SubstrateWeight;
+	use frame_support::pallet_prelude::Weight;
+
+	assert_ne!(<()>::tip(), Weight::from_ref_time(0));
+	assert_ne!(<()>::spend(), Weight::from_ref_time(0));
+	assert_ne!(SubstrateWeight::<Test>::tip(), Weight::from_ref_time(0));
+	assert_ne!(SubstrateWeight::<Test>::spend(), Weight::from_ref_time(0));
+}
+
 #[test]
 fn spend_funds_to_target() {
 	new_test_ext().execute_with(|| {