Merge branch 'master' into version_call

Author: Stephenlawrence00

.rustfmt.toml

@@ -8,16 +8,18 @@ imports_granularity = "Crate"
 reorder_imports = true
 # Consistency
 newline_style = "Unix"
+normalize_comments = true
+normalize_doc_attributes = true
 # Misc
 chain_width = 80
 spaces_around_ranges = false
 binop_separator = "Back"
 reorder_impl_items = false
-match_arm_leading_pipes = "Preserve"
+match_arm_leading_pipes = "Never"
 match_arm_blocks = false
 match_block_trailing_comma = true
 trailing_comma = "Vertical"
-trailing_semicolon = false
+trailing_semicolon = true
 use_field_init_shorthand = true
 # Format comments
 comment_width = 100

Cargo.lock

@@ -424,6 +424,7 @@ members = [
 	"substrate/frame/revive/proc-macro",
 	"substrate/frame/revive/rpc",
 	"substrate/frame/revive/uapi",
+	"substrate/frame/revive/ui-tests",
 	"substrate/frame/root-offences",
 	"substrate/frame/root-testing",
 	"substrate/frame/safe-mode",
@@ -1040,6 +1041,7 @@ pallet-revive-eth-rpc = { path = "substrate/frame/revive/rpc", default-features
 pallet-revive-fixtures = { path = "substrate/frame/revive/fixtures", default-features = false }
 pallet-revive-proc-macro = { path = "substrate/frame/revive/proc-macro", default-features = false }
 pallet-revive-uapi = { path = "substrate/frame/revive/uapi", default-features = false }
+pallet-revive-ui-tests = { path = "substrate/frame/revive/ui-tests", default-features = false }
 pallet-root-offences = { default-features = false, path = "substrate/frame/root-offences" }
 pallet-root-testing = { path = "substrate/frame/root-testing", default-features = false }
 pallet-safe-mode = { default-features = false, path = "substrate/frame/safe-mode" }

Cargo.toml

@@ -105,7 +105,7 @@ where
 			has_failed || !SubmitFinalityProofHelper::<T, I>::was_successful(bundled_block_number);
 
 		if !has_failed {
-			return
+			return;
 		}
 
 		// let's slash registered relayer
@@ -176,7 +176,7 @@ where
 			!SubmitParachainHeadsHelper::<T, ParachainsInstance>::was_successful(&update);
 
 		if !has_failed {
-			return
+			return;
 		}
 
 		// let's slash registered relayer
@@ -465,7 +465,7 @@ mod tests {
 		type ToPostDispatch = u64;
 		fn validate(_who: &u64, call: &MockCall) -> (u64, TransactionValidity) {
 			if call.data <= 1 {
-				return (1, InvalidTransaction::Custom(1).into())
+				return (1, InvalidTransaction::Custom(1).into());
 			}
 
 			(1, Ok(ValidTransaction { priority: 1, ..Default::default() }))
@@ -493,7 +493,7 @@ mod tests {
 		type ToPostDispatch = u64;
 		fn validate(_who: &u64, call: &MockCall) -> (u64, TransactionValidity) {
 			if call.data <= 2 {
-				return (2, InvalidTransaction::Custom(2).into())
+				return (2, InvalidTransaction::Custom(2).into());
 			}
 
 			(2, Ok(ValidTransaction { priority: 2, ..Default::default() }))

bridges/bin/runtime-common/src/extensions.rs

@@ -48,7 +48,7 @@ fn prepare_inbound_message<LaneId>(
 
 	// if we don't need a correct message, then we may just return some random blob
 	if !params.is_successful_dispatch_expected {
-		return vec![0u8; expected_size]
+		return vec![0u8; expected_size];
 	}
 
 	// else let's prepare successful message.

bridges/bin/runtime-common/src/messages_benchmarking.rs

@@ -75,7 +75,7 @@ fn verify_signatures<T: Config<I>, I: 'static>(
 			if authority.verify(sig, &msg) {
 				missing_signatures = missing_signatures.saturating_sub(1);
 				if missing_signatures == 0 {
-					break
+					break;
 				}
 			} else {
 				tracing::debug!(

bridges/modules/beefy/src/utils.rs

@@ -154,7 +154,7 @@ impl<T: Config<I>, I: 'static> SubmitFinalityProofHelper<T, I> {
 					"Cannot finalize obsolete header"
 				);
 
-				return Err(Error::<T, I>::OldHeader)
+				return Err(Error::<T, I>::OldHeader);
 			},
 		};
 
@@ -168,7 +168,7 @@ impl<T: Config<I>, I: 'static> SubmitFinalityProofHelper<T, I> {
 					"Cannot finalize header signed by unknown authority set"
 				);
 
-				return Err(Error::<T, I>::InvalidAuthoritySetId)
+				return Err(Error::<T, I>::InvalidAuthoritySetId);
 			}
 		}
 
@@ -200,7 +200,7 @@ pub trait CallSubType<T: Config<I, RuntimeCall = Self>, I: 'static>:
 				justification,
 				None,
 				false,
-			))
+			));
 		} else if let Some(crate::Call::<T, I>::submit_finality_proof_ex {
 			finality_target,
 			justification,
@@ -213,7 +213,7 @@ pub trait CallSubType<T: Config<I, RuntimeCall = Self>, I: 'static>:
 				justification,
 				Some(*current_set_id),
 				*is_free_execution_expected,
-			))
+			));
 		}
 
 		None
@@ -243,7 +243,7 @@ pub trait CallSubType<T: Config<I, RuntimeCall = Self>, I: 'static>:
 		};
 
 		if Pallet::<T, I>::ensure_not_halted().is_err() {
-			return Err(InvalidTransaction::Call.into())
+			return Err(InvalidTransaction::Call.into());
 		}
 
 		let result = SubmitFinalityProofHelper::<T, I>::check_obsolete_from_extension(&call_info);

bridges/modules/grandpa/src/call_ext.rs

@@ -191,7 +191,6 @@ impl<T: Config<I>, I: 'static> ReceiveMessagesProofSetup<T, I> {
 mod benchmarks {
 	use super::*;
 
-	//
 	// Benchmarks that are used directly by the runtime calls weight formulae.
 	//
 
@@ -501,7 +500,6 @@ mod benchmarks {
 		assert!(T::is_relayer_rewarded(&relayer2_id));
 	}
 
-	//
 	// Benchmarks that the runtime developers may use for proper pallet configuration.
 	//
 

bridges/modules/messages/src/benchmarking.rs

@@ -54,7 +54,7 @@ impl<T: Config<I>, I: 'static> CallHelper<T, I> {
 					// so if relayer slots are released, then message slots are also
 					// released
 					return post_occupation.free_message_slots >
-						info.unrewarded_relayers.free_message_slots
+						info.unrewarded_relayers.free_message_slots;
 				}
 
 				inbound_lane_data.last_delivered_nonce() == *info.base.bundled_range.end()
@@ -131,7 +131,7 @@ impl<
 					best_stored_nonce: inbound_lane_data.last_delivered_nonce(),
 				},
 				unrewarded_relayers: unrewarded_relayers_occupation::<T, I>(&inbound_lane_data),
-			})
+			});
 		}
 
 		None
@@ -157,19 +157,19 @@ impl<
 				bundled_range: outbound_lane_data.latest_received_nonce + 1..=
 					relayers_state.last_delivered_nonce,
 				best_stored_nonce: outbound_lane_data.latest_received_nonce,
-			}))
+			}));
 		}
 
 		None
 	}
 
 	fn call_info(&self) -> Option<MessagesCallInfo<T::LaneId>> {
 		if let Some(info) = self.receive_messages_proof_info() {
-			return Some(MessagesCallInfo::ReceiveMessagesProof(info))
+			return Some(MessagesCallInfo::ReceiveMessagesProof(info));
 		}
 
 		if let Some(info) = self.receive_messages_delivery_proof_info() {
-			return Some(MessagesCallInfo::ReceiveMessagesDeliveryProof(info))
+			return Some(MessagesCallInfo::ReceiveMessagesDeliveryProof(info));
 		}
 
 		None
@@ -195,7 +195,7 @@ impl<
 					"Rejecting messages transaction on halted pallet"
 				);
 
-				return sp_runtime::transaction_validity::InvalidTransaction::Call.into()
+				return sp_runtime::transaction_validity::InvalidTransaction::Call.into();
 			},
 			Some(MessagesCallInfo::ReceiveMessagesProof(proof_info))
 				if proof_info
@@ -207,7 +207,7 @@ impl<
 					"Rejecting obsolete messages delivery transaction"
 				);
 
-				return sp_runtime::transaction_validity::InvalidTransaction::Stale.into()
+				return sp_runtime::transaction_validity::InvalidTransaction::Stale.into();
 			},
 			Some(MessagesCallInfo::ReceiveMessagesDeliveryProof(proof_info))
 				if proof_info.is_obsolete() =>
@@ -218,7 +218,7 @@ impl<
 					"Rejecting obsolete messages confirmation transaction"
 				);
 
-				return sp_runtime::transaction_validity::InvalidTransaction::Stale.into()
+				return sp_runtime::transaction_validity::InvalidTransaction::Stale.into();
 			},
 			_ => {},
 		}

bridges/modules/messages/src/call_ext.rs

@@ -151,10 +151,10 @@ impl<S: InboundLaneStorage> InboundLane<S> {
 
 		if outbound_lane_data.latest_received_nonce > last_delivered_nonce {
 			// this is something that should never happen if proofs are correct
-			return None
+			return None;
 		}
 		if outbound_lane_data.latest_received_nonce <= data.last_confirmed_nonce {
-			return None
+			return None;
 		}
 
 		let new_confirmed_nonce = outbound_lane_data.latest_received_nonce;
@@ -191,18 +191,18 @@ impl<S: InboundLaneStorage> InboundLane<S> {
 	) -> ReceptionResult<Dispatch::DispatchLevelResult> {
 		let mut data = self.storage.data();
 		if Some(nonce) != data.last_delivered_nonce().checked_add(1) {
-			return ReceptionResult::InvalidNonce
+			return ReceptionResult::InvalidNonce;
 		}
 
 		// if there are more unrewarded relayer entries than we may accept, reject this message
 		if data.relayers.len() as MessageNonce >= self.storage.max_unrewarded_relayer_entries() {
-			return ReceptionResult::TooManyUnrewardedRelayers
+			return ReceptionResult::TooManyUnrewardedRelayers;
 		}
 
 		// if there are more unconfirmed messages than we may accept, reject this message
 		let unconfirmed_messages_count = nonce.saturating_sub(data.last_confirmed_nonce);
 		if unconfirmed_messages_count > self.storage.max_unconfirmed_messages() {
-			return ReceptionResult::TooManyUnconfirmedMessages
+			return ReceptionResult::TooManyUnconfirmedMessages;
 		}
 
 		// then, dispatch message