Fix memo_overwrite configuration

crates/relayer-cli/src/commands/tx/channel.rs

@@ -23,7 +23,7 @@ use crate::prelude::*;
 ///
 /// The macro takes the following arguments:
 /// - `$dbg_string`: a string literal that will be used to identify the subcommand
-///  in debug logs
+///   in debug logs
 /// - `$func`: the method that will be called to build and send the `Channel` message
 /// - `$self`: the type that `Runnable` is being implemented for
 /// - `$chan`: a closure that specifies how to build the `Channel` object

crates/relayer-cli/src/conclude.rs

@@ -17,9 +17,9 @@
 //! ```
 //!
 //! - Exit from a query/tx with an error of type `anomaly`:
-//! In the case where the error is a complex type such as anomaly (including backtraces), it is
-//! better to simplify the output and only write out the chain of error sources, which we can
-//! achieve with `format!("{}", e)`. The complete solution is as follows:
+//!   In the case where the error is a complex type such as anomaly (including backtraces), it is
+//!   better to simplify the output and only write out the chain of error sources, which we can
+//!   achieve with `format!("{}", e)`. The complete solution is as follows:
 //!
 //! ```ignore
 //! let e: Error = Kind::Query.into();

crates/relayer-cli/tests/acceptance.rs

@@ -24,7 +24,6 @@ use once_cell::sync::Lazy;
 pub static RUNNER: Lazy<CmdRunner> = Lazy::new(CmdRunner::default);
 
 /// Use `Config::default()` value if no config or args
-#[cfg(not(tarpaulin))]
 #[test]
 fn start_no_args() {
     let mut runner = RUNNER.clone();
@@ -40,7 +39,6 @@ fn start_no_args() {
     cmd.wait().unwrap().expect_success();
 }
 
-#[cfg(not(tarpaulin))]
 #[test]
 fn example_configuration_is_valid() {
     let mut runner = RUNNER.clone();

crates/relayer/src/config.rs

@@ -3,6 +3,8 @@
 pub mod compat_mode;
 pub mod dynamic_gas;
 pub mod error;
+/// TODO: Fix mutable_key_type warning
+#[allow(clippy::mutable_key_type)]
 pub mod filter;
 pub mod gas_multiplier;
 pub mod proof_specs;

crates/relayer/src/config/filter.rs

@@ -31,6 +31,8 @@ impl Default for PacketFilter {
     }
 }
 
+/// TODO: Fix mutable_key_type warning
+#[allow(clippy::mutable_key_type)]
 impl PacketFilter {
     pub fn new(
         channel_policy: ChannelPolicy,

crates/relayer/src/link/relay_path.rs

@@ -744,7 +744,7 @@ impl<ChainA: ChainHandle, ChainB: ChainHandle> RelayPath<ChainA, ChainB> {
     /// Return value:
     ///   - `Some(..)`: a new operational data from which to retry sending,
     ///   - `None`: all the events in the initial operational data were exhausted (i.e., turned
-    ///   into timeouts), so there is nothing to retry.
+    ///     into timeouts), so there is nothing to retry.
     ///
     /// Side effects: may schedule a new operational data targeting the source chain, comprising
     /// new timeout messages.

tools/integration-test/src/tests/fee/filter_fees.rs

@@ -18,6 +18,8 @@ fn test_filter_by_channel_incentivized_fees_relayer() -> Result<(), Error> {
 
 struct FilterIncentivizedFeesRelayerTest;
 
+/// TODO: Fix mutable_key_type warning
+#[allow(clippy::mutable_key_type)]
 impl TestOverrides for FilterIncentivizedFeesRelayerTest {
     fn modify_relayer_config(&self, config: &mut Config) {
         config.mode.packets.auto_register_counterparty_payee = true;
@@ -166,6 +168,8 @@ impl BinaryChannelTest for FilterIncentivizedFeesRelayerTest {
 
 struct FilterByChannelIncentivizedFeesRelayerTest;
 
+/// TODO: Fix mutable_key_type warning
+#[allow(clippy::mutable_key_type)]
 impl TestOverrides for FilterByChannelIncentivizedFeesRelayerTest {
     fn modify_relayer_config(&self, config: &mut Config) {
         config.mode.packets.auto_register_counterparty_payee = true;

tools/integration-test/src/tests/memo.rs

@@ -23,7 +23,7 @@ fn test_memo() -> Result<(), Error> {
 #[test]
 fn test_memo_overwrite() -> Result<(), Error> {
     let memo = Memo::new(random_string()).unwrap();
-    let test = MemoTest { memo };
+    let test = MemoOverwriteTest { memo };
     run_binary_channel_test(&test)
 }