address comments and test fix

README.md

@@ -31,8 +31,6 @@ Coinswap v0.1.0 marketplace is now live on Testnet4.
 
 This library is currently under beta development and is in an experimental stage. There are known and unknown bugs. **Mainnet use is strictly NOT recommended.** 
 
-Additionally, note that this code is currently designed to run exclusively on Linux systems.
-
 # About
 
 Coinswap is a decentralized [atomic swap](https://bitcoinops.org/en/topics/coinswap/) protocol that enables trustless swaps of Bitcoin UTXOs through a decentralized, Sybil-resistant marketplace.
@@ -94,7 +92,7 @@ taker --help
 
   `maker-cli`: The RPC controler of the server deamon. This can be used to manage the server, access internal wallet, see swap statistics, etc. App demo [here](https://github.com/citadel-tech/coinswap/blob/master/docs/app%20demos/maker-cli.md)
 
-  `taker`: The swap client app. This acts as a regular bitcoin wallet with swap capability. App dmeo [here](https://github.com/citadel-tech/coinswap/blob/master/docs/app%20demos/taker-tutorial.md)
+  `taker`: The swap client app. This acts as a regular bitcoin wallet with swap capability. App demo [here](https://github.com/citadel-tech/coinswap/blob/master/docs/app%20demos/taker-tutorial.md)
 
 All the apps will require a Bitcoin Core RPC connection running on testnet4. For running bitcoin instrcutions, see [here](https://github.com/citadel-tech/coinswap/blob/master/docs/app%20demos/bitcoind.md)
 

docs/app demos/maker-cli.md

@@ -161,7 +161,7 @@ $ ./maker-cli show-fidelity
 ```
 
 **Output:**  
-```json
+```bash
 {
     0: (
         FidelityBond {

tests/abort1.rs

@@ -118,6 +118,11 @@ fn test_stop_taker_after_setup() {
     };
     taker.do_coinswap(swap_params).unwrap();
 
+    // After Swap is done,  wait for maker threads to conclude.
+    makers
+        .iter()
+        .for_each(|maker| maker.shutdown.store(true, Relaxed));
+
     // After Swap is done,  wait for maker threads to conclude.
     maker_threads
         .into_iter()

tests/abort3_case3.rs

@@ -114,6 +114,11 @@ fn abort3_case3_close_at_hash_preimage_handover() {
     };
     taker.do_coinswap(swap_params).unwrap();
 
+    // After Swap is done,  wait for maker threads to conclude.
+    makers
+        .iter()
+        .for_each(|maker| maker.shutdown.store(true, Relaxed));
+
     maker_threads
         .into_iter()
         .for_each(|thread| thread.join().unwrap());

tests/test_framework/mod.rs

@@ -345,10 +345,12 @@ pub fn verify_swap_results(
             );
 
             assert_eq!(fidelity_balance, Amount::from_btc(0.05).unwrap());
-            // Live contract balance will remain
+
+            // Live contract balance can be non-zero, if a maker shuts down in middle of recovery.
             assert!(
                 live_contract_balance == Amount::ZERO
-                    || live_contract_balance == Amount::from_btc(0.004605).unwrap() // for cases like abort3_case3: where the fauly maker doesn't recover.
+                    || live_contract_balance == Amount::from_btc(0.00460500).unwrap() // For the first maker in hop
+                    || live_contract_balance == Amount::from_btc(0.00435642).unwrap() // For the second maker in hop
             );
 
             // Check spendable balance difference.