fix: allow cancel of started as well as in-progress transfers

cancel_transfer only looked for transfers whose status was 'in_progress',
so a transfer that had been initialized but had not yet received data could
not be cancelled. Include 'started' transfers in the search.

Author: luni

dccbot/manager.py

@@ -66,7 +66,7 @@ async def cancel_transfer(self, server: str, nick: str, filename: str) -> bool:
 
         # Find the transfer in bot.current_transfers
         for dcc, transfer in bot.current_transfers.items():
-            if transfer.get("filename") == filename and transfer.get("status") == "in_progress" and transfer.get("nick", "").lower() == nick:
+            if transfer.get("filename") == filename and transfer.get("status") in ("started", "in_progress") and transfer.get("nick", "").lower() == nick:
                 # Disconnect the DCC connection
                 try:
                     dcc.disconnect("Cancelled by user")

tests/unit/test_manager.py

@@ -355,6 +355,24 @@ async def test_cancel_transfer_matches_mixed_case_nick(manager):
     assert transfer["status"] == "cancelled"
 
 
+@pytest.mark.asyncio
+async def test_cancel_transfer_started_status(manager):
+    """Test cancel_transfer works for transfers that have not received data yet."""
+    mock_bot = MagicMock()
+    mock_dcc = MagicMock()
+    transfer = {
+        "filename": "test.txt",
+        "status": "started",
+        "nick": "sender",
+    }
+    mock_bot.current_transfers = {mock_dcc: transfer}
+    manager.bots = {"irc.example.com": mock_bot}
+
+    result = await manager.cancel_transfer("irc.example.com", "sender", "test.txt")
+    assert result is True
+    assert transfer["status"] == "cancelled"
+
+
 @pytest.mark.asyncio
 async def test_cancel_transfer_not_found(manager):
     """Test transfer cancellation when transfer not found."""