review feedback

Snowfork · Nov 8, 2023 · 75ccfa7 · 75ccfa7
1 parent 2118950
commit 75ccfa7
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/parachain/pallets/ethereum-beacon-client/src/impls.rs b/parachain/pallets/ethereum-beacon-client/src/impls.rs
@@ -58,7 +58,7 @@ impl<T: Config> Verifier for Pallet<T> {
 				"💫 Event log not found in receipt for transaction at index {} in block {}",
 				message.proof.tx_index, message.proof.block_hash,
 			);
-			return Err(NotFound)
+			return Err(LogNotFound)
 		}
 
 		log::info!(

diff --git a/parachain/pallets/ethereum-beacon-client/src/tests.rs b/parachain/pallets/ethereum-beacon-client/src/tests.rs
@@ -977,7 +977,7 @@ fn verify_message_receipt_does_not_contain_log() {
 
 	new_tester().execute_with(|| {
 		<ExecutionHeaderBuffer<Test>>::insert(block_hash, header);
-		assert_err!(EthereumBeaconClient::verify(&message), VerificationError::NotFound);
+		assert_err!(EthereumBeaconClient::verify(&message), VerificationError::LogNotFound);
 	});
 }
 

diff --git a/parachain/primitives/core/src/inbound.rs b/parachain/primitives/core/src/inbound.rs
@@ -20,7 +20,7 @@ pub enum VerificationError {
 	/// Execution header is missing
 	HeaderNotFound,
 	/// Log was not found in the verified transaction receipt
-	NotFound,
+	LogNotFound,
 	/// Data payload does not decode into a valid Log
 	InvalidLog,
 	/// Unable to verify the transaction receipt with the provided proof