flake

Author: Yostra

src/wallet/trade_manager.py

@@ -193,10 +193,10 @@ async def maybe_create_wallets_for_offer(self, file_path: Path) -> bool:
 
         return True
 
-    async def respond_to_offer(self, file_path: Path) -> bool:
+    async def respond_to_offer(self, file_path: Path) -> Tuple[bool, Optional[str]]:
         has_wallets = await self.maybe_create_wallets_for_offer(file_path)
         if not has_wallets:
-            return False
+            return False, "Unknown Error"
         trade_offer_hex = file_path.read_text()
         trade_offer = SpendBundle.from_bytes(bytes.fromhex(trade_offer_hex))
 

tests/cc_wallet/test_cc_wallet.py

@@ -337,7 +337,7 @@ async def test_cc_trade(self, two_wallet_nodes):
         assert offer["chia"] == -10
         assert offer[colour] == 30
 
-        success = await trade_manager_2.respond_to_offer(file_path)
+        success, reason = await trade_manager_2.respond_to_offer(file_path)
 
         assert success is True
 
@@ -520,7 +520,7 @@ async def test_cc_trade_with_multiple_colours(self, two_wallet_nodes):
         assert offer[red] == 30
         assert offer[blue] == -50
 
-        success = await trade_manager_2.respond_to_offer(file_path)
+        success, reason = await trade_manager_2.respond_to_offer(file_path)
 
         assert success is True
         for i in range(0, num_blocks):
@@ -617,7 +617,7 @@ async def test_create_offer_with_zero_val(self, two_wallet_nodes):
         assert offer["chia"] == 10
         assert offer[colour] == -30
 
-        success = await trade_manager_1.respond_to_offer(file_path)
+        success, reason = await trade_manager_1.respond_to_offer(file_path)
 
         assert success is True