From 1b65617d7f30b386e880f908949a4f56b524d566 Mon Sep 17 00:00:00 2001
From: Alin Sinpalean <58422065+alin-at-dfinity@users.noreply.github.com>
Date: Thu, 5 Dec 2024 11:21:45 +0100
Subject: [PATCH] fix: Fix XNet end-to-end tests (#2985)

Encode the arguments to the start method as the newly defined StartArgs
struct instead of a tuple.
---
 rs/tests/message_routing/common/common.rs | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/rs/tests/message_routing/common/common.rs b/rs/tests/message_routing/common/common.rs
index 078d1cbe221..dc72b6836f5 100644
--- a/rs/tests/message_routing/common/common.rs
+++ b/rs/tests/message_routing/common/common.rs
@@ -7,6 +7,7 @@ use ic_system_test_driver::driver::test_env::TestEnv;
 use ic_system_test_driver::driver::test_env_api::get_dependency_path;
 use slog::info;
 use std::{convert::TryFrom, env};
+use xnet_test::StartArgs;
 
 /// Concurrently calls `start` on all canisters in `canisters` with the
 /// given parameters.
@@ -29,10 +30,14 @@ pub async fn start_all_canisters(
         .enumerate()
         .flat_map(|(x, v)| v.iter().enumerate().map(move |(y, v)| (x, y, v)))
     {
-        let input = (&topology, canister_to_subnet_rate, payload_size_bytes);
+        let input = StartArgs {
+            network_topology: topology.clone(),
+            canister_to_subnet_rate,
+            payload_size_bytes,
+        };
         futures.push(async move {
             let _: String = canister
-                .update_("start", candid, input)
+                .update_("start", candid, (input,))
                 .await
                 .unwrap_or_else(|e| {
                     panic!(