fix ternary test

heliaxdev · Jul 10, 2024 · a777772 · a777772
1 parent 2d7fdaa
commit a777772
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/tools/integration-test/src/tests/tendermint/sequential.rs b/tools/integration-test/src/tests/tendermint/sequential.rs
@@ -110,7 +110,7 @@ impl BinaryChannelTest for SequentialCommitTest {
                 ChainType::Namada => (
                     Duration::from_millis((BLOCK_TIME_MILLIS * TOTAL_TRANSACTIONS as u64) - 1000),
                     Duration::from_millis(
-                        (BLOCK_TIME_MILLIS * TOTAL_TRANSACTIONS as u64) * 2 + 1000,
+                        (BLOCK_TIME_MILLIS * TOTAL_TRANSACTIONS as u64 * 2) + 1000,
                     ),
                 ),
                 _ => {
@@ -164,7 +164,10 @@ impl BinaryChannelTest for SequentialCommitTest {
             );
 
             let max_duration = match chains.node_b.chain_driver().value().chain_type {
-                ChainType::Namada => Duration::from_millis(BLOCK_TIME_MILLIS * 2 * 2),
+                ChainType::Namada => {
+                    // Shorter than the sequential batches
+                    Duration::from_millis(BLOCK_TIME_MILLIS * TOTAL_TRANSACTIONS as u64 * 2)
+                }
                 _ => Duration::from_millis(BLOCK_TIME_MILLIS * 2),
             };
             assert!(duration < max_duration);

diff --git a/tools/test-framework/src/framework/nary/node.rs b/tools/test-framework/src/framework/nary/node.rs
@@ -60,7 +60,9 @@ where
         let mut node_processes = Vec::new();
 
         for i in 0..SIZE {
-            let node = if builder.command_paths.contains(&"namada".to_string()) {
+            let is_namada = builder.command_paths == vec!["namada".to_string()]
+                || builder.command_paths.get(i) == Some(&"namada".to_string());
+            let node = if is_namada {
                 bootstrap_namada_node(
                     builder,
                     &format!("{}", i + 1),