fix comments

pallets/axelar-router/src/lib.rs

@@ -156,7 +156,7 @@ pub mod pallet {
 	#[pallet::error]
 	pub enum Error<T> {
 		/// Emit when the router configuration is not found.
-		RouterNotFound,
+		RouterConfigurationNotFound,
 
 		/// Emit when the evm account code is not registered
 		ContractCodeMismatch,
@@ -222,7 +222,8 @@ pub mod pallet {
 				.try_into()
 				.map_err(|_| Error::<T>::SourceChainTooLong)?;
 
-			let config = Configuration::<T>::get(chain_name).ok_or(Error::<T>::RouterNotFound)?;
+			let config = Configuration::<T>::get(chain_name)
+				.ok_or(Error::<T>::RouterConfigurationNotFound)?;
 
 			ensure!(
 				caller == config.liquidity_pools_contract_address,
@@ -316,9 +317,10 @@ pub mod pallet {
 		type Origin = DomainAddress;
 
 		fn send(axelar_id: AxelarId, origin: Self::Origin, message: Vec<u8>) -> DispatchResult {
-			let chain_name =
-				ChainNameById::<T>::get(axelar_id).ok_or(Error::<T>::RouterNotFound)?;
-			let config = Configuration::<T>::get(&chain_name).ok_or(Error::<T>::RouterNotFound)?;
+			let chain_name = ChainNameById::<T>::get(axelar_id)
+				.ok_or(Error::<T>::RouterConfigurationNotFound)?;
+			let config = Configuration::<T>::get(&chain_name)
+				.ok_or(Error::<T>::RouterConfigurationNotFound)?;
 
 			match config.domain {
 				DomainConfig::Evm(evm_config) => {

pallets/axelar-router/src/tests.rs

@@ -113,7 +113,7 @@ mod send {
 		new_test_ext().execute_with(|| {
 			assert_err!(
 				Router::send(AxelarId::Evm(CHAIN_ID), SENDER, MESSAGE.to_vec()),
-				Error::<Runtime>::RouterNotFound,
+				Error::<Runtime>::RouterConfigurationNotFound,
 			);
 		});
 	}
@@ -167,7 +167,7 @@ mod receive {
 					&SOURCE_ADDRESS.0,
 					MESSAGE
 				),
-				Error::<Runtime>::RouterNotFound
+				Error::<Runtime>::RouterConfigurationNotFound
 			);
 		});
 	}

pallets/liquidity-pools-gateway/src/lib.rs

@@ -179,7 +179,7 @@ pub mod pallet {
 		UnknownInstance,
 
 		/// Router not found.
-		RouterNotFound,
+		RouterConfigurationNotFound,
 
 		/// Emitted when you call `start_batch_messages()` but that was already
 		/// called. You should finalize the message with `end_batch_messages()`

runtime/common/src/routing.rs

@@ -12,9 +12,11 @@ use sp_core::{H160, H256};
 use sp_runtime::traits::{BlakeTwo256, Hash};
 use sp_std::{marker::PhantomData, vec, vec::Vec};
 
-/// Identification of the router where the message is sent and received
-/// RouterId is more specific than Domain, because RouterId also identify by
-/// where the message is sent/received
+/// Identification of the router where the messages are sent and received.
+///
+/// NOTE: `RouterId` is more specific than `Domain`. `Domain` identifies the
+/// source and destination of the message, but `RouterId` also identifies how
+/// to reach them.
 #[derive(Debug, Encode, Decode, Clone, PartialEq, Eq, TypeInfo, MaxEncodedLen)]
 pub enum RouterId {
 	/// The message must be sent/received by EVM using Axelar

runtime/integration-tests/src/cases/liquidity_pools_gateway_queue.rs

@@ -67,7 +67,7 @@ fn outbound<T: Runtime + FudgeSupport>() {
 		pallet_liquidity_pools_gateway_queue::Event::<T>::MessageExecutionFailure {
 			nonce,
 			message,
-			error: pallet_axelar_router::Error::<T>::RouterNotFound.into(),
+			error: pallet_axelar_router::Error::<T>::RouterConfigurationNotFound.into(),
 		}
 	});