refactor: bind to 127.0.0.1:60000 synchronously instead of synchronising around it

Author: nabijaczleweli

src/client.rs

@@ -451,26 +451,18 @@ mod tests {
         // try.
 
         use std::net::TcpListener;
-        use std::sync::mpsc::channel;
         use std::time::{Duration, Instant};
 
         let endpoint =
             std::env::var("TEST_ELECTRUM_TIMEOUT_PORT").unwrap_or("localhost:60000".into());
-        let (sender, receiver) = channel();
 
+        let listener = TcpListener::bind("127.0.0.1:60000").unwrap();
         std::thread::spawn(move || {
-            let listener = TcpListener::bind("127.0.0.1:60000").unwrap();
-            sender.send(()).unwrap();
-
             for _stream in listener.incoming() {
                 std::thread::sleep(Duration::from_secs(60))
             }
         });
 
-        receiver
-            .recv_timeout(Duration::from_secs(5))
-            .expect("Can't start local listener");
-
         let now = Instant::now();
         let client = Client::from_config(
             &endpoint,