test: authority rotation integration tests

state-chain/amm/src/common.rs

@@ -335,13 +335,13 @@ pub(super) fn sqrt_price_at_tick(tick: Tick) -> SqrtPriceQ64F96 {
 	/* Proof that r is never zero (therefore avoiding the divide by zero case here):
 		We can think of an application of the `handle_tick_bit` macro as increasing the index I of r's MSB/`r.ilog2()` (mul by constant), and then decreasing it by 128 (the right shift).
 
-		Note the increase in I caused by the constant mul will be atleast constant.ilog2().
+		Note the increase in I caused by the constant mul will be at least constant.ilog2().
 
 		Also note each application of `handle_tick_bit` decreases (if the if branch is entered) or else maintains r's value as all the constants are less than 2^128.
 
 		Therefore the largest decrease would be caused if all the macros application's if branches where entered.
 
-		So we assuming all if branches are entered, after all the applications `I` would be atleast I_initial + bigsum(constant.ilog2()) - 19*128.
+		So we assuming all if branches are entered, after all the applications `I` would be at least I_initial + bigsum(constant.ilog2()) - 19*128.
 
 		The test `r_non_zero` checks with value is >= 0, therefore imply the smallest value r could have is more than 0.
 	*/

state-chain/amm/src/limit_orders.rs

@@ -75,7 +75,7 @@ impl FloatBetweenZeroAndOne {
 
 		let (mul_div_normalised_mantissa, div_normalise_shift) = {
 			// As the denominator <= U256::MAX, this div will not right-shift the mantissa more than
-			// 256 bits, so we maintain atleast 256 accurate bits in the result.
+			// 256 bits, so we maintain at least 256 accurate bits in the result.
 			let (d, div_remainder) =
 				U512::div_mod(mul_normalised_mantissa, U512::from(denominator));
 			let d = if div_remainder.is_zero() { d } else { d + U512::one() };
@@ -96,11 +96,11 @@ impl FloatBetweenZeroAndOne {
 			Self { normalised_mantissa: mul_div_normalised_mantissa, negative_exponent }
 		} else {
 			// This bounding will cause swaps to get bad prices, but this case will effectively
-			// never happen, as atleast (U256::MAX / 256) (~10^74) swaps would have to happen to get
-			// into this situation. TODO: A possible solution is disabling minting for pools "close"
-			// to this minimum. With a small change to the swapping logic it would be possible to
-			// guarantee that the pool would be emptied before percent_remaining could reach this
-			// min bound.
+			// never happen, as at least (U256::MAX / 256) (~10^74) swaps would have to happen to
+			// get into this situation. TODO: A possible solution is disabling minting for pools
+			// "close" to this minimum. With a small change to the swapping logic it would be
+			// possible to guarantee that the pool would be emptied before percent_remaining could
+			// reach this min bound.
 			Self { normalised_mantissa: U256::one() << 255, negative_exponent: U256::MAX }
 		}
 	}

state-chain/cf-integration-tests/Cargo.toml

@@ -13,6 +13,7 @@ targets = ['x86_64-unknown-linux-gnu']
 
 [dependencies]
 state-chain-runtime = { path = '../runtime' }
+log = { version = '0.4.16', default-features = false }
 
 [dev-dependencies]
 libsecp256k1 = { version = "0.7", features = ['static-context'] }
@@ -78,3 +79,4 @@ sp-std = { git = "https://github.com/chainflip-io/substrate.git", tag = "chainfl
 sp-transaction-pool = { git = "https://github.com/chainflip-io/substrate.git", tag = "chainflip-monthly-2023-08+1" }
 sp-version = { git = "https://github.com/chainflip-io/substrate.git", tag = "chainflip-monthly-2023-08+1" }
 sp-consensus-grandpa = { git = "https://github.com/chainflip-io/substrate.git", tag = "chainflip-monthly-2023-08+1" }
+sp-timestamp = { git = "https://github.com/chainflip-io/substrate.git", tag = "chainflip-monthly-2023-08+1" }

state-chain/cf-integration-tests/src/account.rs

@@ -4,10 +4,11 @@ use crate::network;
 use cf_chains::eth::Address as EthereumAddress;
 use cf_primitives::GENESIS_EPOCH;
 use cf_traits::EpochInfo;
+use frame_support::traits::OnNewAccount;
 use pallet_cf_funding::{MinimumFunding, RedemptionAmount};
 use pallet_cf_reputation::Reputations;
 use pallet_cf_validator::{AccountPeerMapping, MappedPeers, VanityNames};
-use state_chain_runtime::{Reputation, Runtime, Validator};
+use state_chain_runtime::{AccountRoles, Reputation, Runtime, Validator};
 
 #[test]
 fn account_deletion_removes_relevant_storage_items() {
@@ -28,6 +29,7 @@ fn account_deletion_removes_relevant_storage_items() {
 		);
 		testnet.move_forward_blocks(1);
 
+		AccountRoles::on_new_account(&backup_node);
 		network::Cli::register_as_validator(&backup_node);
 
 		network::setup_peer_mapping(&backup_node);
@@ -44,6 +46,15 @@ fn account_deletion_removes_relevant_storage_items() {
 		let vanity_names = VanityNames::<Runtime>::get();
 		assert_eq!(*vanity_names.get(&backup_node).unwrap(), elon_vanity_name.as_bytes().to_vec());
 
+		// Fund the account
+		testnet.state_chain_gateway_contract.fund_account(
+			backup_node.clone(),
+			crate::genesis::GENESIS_BALANCE,
+			GENESIS_EPOCH,
+		);
+		testnet.move_forward_blocks(2);
+
+		// Redeem all
 		network::Cli::redeem(
 			&backup_node,
 			RedemptionAmount::Max,