fix/audit

* chore: updated devnet config

* chore: updated devnet config

* chore: devnet config update

* fix: fixed accept_backing_offer logic

* chore: fixed warnings

* fix: fixed redelegation check and now using free_balance instead of total balance during locking

* fix: fixed pools validation function

* fix: added pools validation

* fix: fixed collator reward logic when inflation is set to 0 and create_pool now respects the MaxPools config

Author: godenzim

Cargo.lock

@@ -46,12 +46,6 @@ impl ExtBuilder {
 	}
 }
 
-impl Default for ExtBuilder {
-	fn default() -> Self {
-		Self {}
-	}
-}
-
 pub const INITIAL_BALANCE: u128 = UNIT * 10;
 pub const EXISTENTIAL_DEPOSIT: Balance = MILLIUNIT;
 pub const UNIT: Balance = 1_000_000;

pallets/acurast/src/mock.rs

@@ -1,5 +1,3 @@
-#![cfg(feature = "runtime-benchmarks")]
-
 use frame_benchmarking::v2::*;
 use frame_system::RawOrigin;
 

pallets/candidate-preselection/src/benchmarking.rs

@@ -6,7 +6,6 @@ use frame_support::{
 };
 use frame_system::EnsureRoot;
 use sp_core::H256;
-use sp_io;
 use sp_runtime::{traits::IdentityLookup, AccountId32, BuildStorage};
 
 use crate::*;
@@ -35,12 +34,6 @@ impl ExtBuilder {
 	}
 }
 
-impl Default for ExtBuilder {
-	fn default() -> Self {
-		Self {}
-	}
-}
-
 frame_support::construct_runtime!(
 	pub enum Test {
 		System: frame_system::{Pallet, Call, Config<T>, Storage, Event<T>} = 0,

pallets/candidate-preselection/src/mock.rs

@@ -79,6 +79,29 @@ impl<Epoch: Ord + Debug, Value: Copy + Default + Debug> ProvisionalBuffer<Epoch,
 
 		self.current
 	}
+
+	pub fn next(&self, epoch: Epoch) -> Option<Value> {
+		if let Some((e, v)) = &self.next {
+			if e > &epoch {
+				return Some(*v);
+			}
+		}
+
+		None
+	}
+
+	pub fn next_epoch(&self, epoch: Epoch) -> Option<Epoch>
+	where
+		Epoch: Copy,
+	{
+		if let Some((e, _)) = &self.next {
+			if e > &epoch {
+				return Some(*e);
+			}
+		}
+
+		None
+	}
 }
 
 #[cfg(test)]