unit-tests issues fixing

Author: ssrlive

src/args.rs

@@ -43,7 +43,7 @@ pub struct Args {
 
     /// run in client mode; value is the server's address
     #[arg(short, long, value_name = "host", conflicts_with = "server")]
-    pub client: Option<std::net::IpAddr>,
+    pub client: Option<String>,
 
     /// run in reverse-mode (server sends, client receives)
     #[arg(short = 'R', long, conflicts_with = "server")]
@@ -109,7 +109,7 @@ pub struct Args {
     /// primarily to avoid including TCP ramp-up in averages;
     /// using this option may result in disagreement between bytes sent and received,
     /// since data can be in-flight across time-boundaries
-    #[arg(short, long, default_value = "0", value_name = "seconds", conflicts_with = "server")]
+    #[arg(short = 'o', long, default_value = "0", value_name = "seconds", conflicts_with = "server")]
     pub omit: usize,
 
     /// use no-delay mode for TCP tests, disabling Nagle's Algorithm

src/client.rs

@@ -129,7 +129,7 @@ pub fn execute(args: &args::Args) -> BoxResult<()> {
     let mut download_config = prepare_download_configuration(args, test_id.as_bytes())?;
 
     //connect to the server
-    let mut stream = connect_to_server(&args.client.unwrap().to_string(), &args.port)?;
+    let mut stream = connect_to_server(args.client.as_ref().unwrap(), &args.port)?;
     let server_addr = stream.peer_addr()?;
 
     //scaffolding to track and relay the streams and stream-results associated with this test