fixup a few missing penumbra branches in tests

astriaorg · Feb 26, 2024 · b867d4c · b867d4c
1 parent e669165
commit b867d4c
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 3 deletions.
diff --git a/crates/relayer/src/config.rs b/crates/relayer/src/config.rs
@@ -657,11 +657,10 @@ impl ChainConfig {
         }
     }
 
-    // TODO(extract): needs to be changed to return &str so penumbra can return ""
-    pub fn key_name(&self) -> &String {
+    pub fn key_name(&self) -> &str {
         match self {
             Self::CosmosSdk(config) => &config.key_name,
-            Self::Penumbra(config) => &config.stub_key_name,
+            Self::Penumbra(_) => "",
         }
     }
 

diff --git a/tools/integration-test/src/tests/fee/filter_fees.rs b/tools/integration-test/src/tests/fee/filter_fees.rs
@@ -31,6 +31,7 @@ impl TestOverrides for FilterIncentivizedFeesRelayerTest {
                 ChainConfig::CosmosSdk(chain_config) => {
                     chain_config.packet_filter = packet_filter.clone();
                 }
+                ChainConfig::Penumbra(_) => todo!(),
             }
         }
     }
@@ -183,6 +184,7 @@ impl TestOverrides for FilterByChannelIncentivizedFeesRelayerTest {
                 ChainConfig::CosmosSdk(chain_config) => {
                     chain_config.packet_filter = packet_filter.clone();
                 }
+                ChainConfig::Penumbra(_) => todo!(),
             }
         }
     }

diff --git a/tools/integration-test/src/tests/ica.rs b/tools/integration-test/src/tests/ica.rs
@@ -73,6 +73,7 @@ impl TestOverrides for IcaFilterTestAllow {
                 ChainConfig::CosmosSdk(chain_config) => {
                     chain_config.packet_filter = self.packet_filter.clone();
                 }
+                ChainConfig::Penumbra(_) => todo!(),
             }
         }
     }
@@ -203,6 +204,7 @@ impl TestOverrides for IcaFilterTestDeny {
                             FilterPattern::Wildcard("*".parse().unwrap()),
                         )]));
                 }
+                ChainConfig::Penumbra(_) => todo!(),
             }
         }
     }