docs: compact code comments per review feedback

Author: haiyuechen-nearone

crates/e2e-tests/src/foreign_chain_mock.rs

@@ -9,10 +9,7 @@ use httpmock::prelude::*;
 use httpmock::{HttpMockRequest, HttpMockResponse};
 
 /// Client credentials a mock server requires, mirroring the node's
-/// [`AuthConfig`](mpc_node_config::AuthConfig) kinds. Requests without the
-/// expected credentials are answered with 401 like real providers do, so
-/// verification can only succeed if the node applied the configured
-/// authentication.
+/// [`AuthConfig`](mpc_node_config::AuthConfig) kinds.
 #[derive(Clone, Debug)]
 pub enum MockAuthExpectation {
     None,
@@ -34,8 +31,6 @@ impl MockAuthExpectation {
 
 /// Rejects requests missing the expected credentials the way real providers
 /// do.
-/// Must be registered AFTER the credentialed mock — httpmock picks the
-/// first-registered matching mock, so authenticated requests keep hitting it.
 fn register_unauthorized_catch_all(server: &MockServer, auth: &MockAuthExpectation) {
     if matches!(auth, MockAuthExpectation::None) {
         return;

crates/e2e-tests/tests/foreign_chain_tx_validation.rs

@@ -541,15 +541,10 @@ async fn verify_polygon(env: &ForeignTxTestEnv) -> anyhow::Result<()> {
     verify_foreign_tx_response(&outcome)
 }
 
-/// Sets up a single 2-node cluster with mock RPC servers for all chains,
-/// then submits verify_foreign_transaction requests for Bitcoin, Abstract,
-/// BNB, Base, Starknet, Arbitrum, HyperEVM, and Polygon and verifies the MPC
-/// nodes return valid signed responses. Also verifies rejection for unsupported
-/// chains and non-existent domains.
-///
-/// Bitcoin, Base and BNB providers require authentication (one per
-/// credential-carrying [`AuthConfig`] kind), so the test also proves the node
-/// applies configured RPC credentials end to end.
+/// Verifies all supported chains sign, and unsupported chains and non-existent
+/// domains are rejected. Bitcoin, Base and BNB require authentication (one per
+/// credential-carrying [`AuthConfig`] kind), proving the node applies configured
+/// RPC credentials end to end.
 #[tokio::test]
 #[expect(non_snake_case)]
 async fn verify_foreign_transaction__should_sign_all_supported_chains() {