From 5f42d82293cd6358349c5e7ead338c7b760e57d5 Mon Sep 17 00:00:00 2001 From: LIghtJUNction Date: Tue, 24 Mar 2026 20:49:48 +0800 Subject: [PATCH] fix(test_abp_client): update test to match new non-Future implementation shutdown() now clears pending requests directly instead of calling cancel() on asyncio.Future instances. --- tests/unit/test_internal/test_abp_client.py | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/tests/unit/test_internal/test_abp_client.py b/tests/unit/test_internal/test_abp_client.py index 8211d85f3..b34b22c9d 100644 --- a/tests/unit/test_internal/test_abp_client.py +++ b/tests/unit/test_internal/test_abp_client.py @@ -98,15 +98,12 @@ class TestAstrbotAbpClient: @pytest.mark.asyncio async def test_shutdown_cancels_pending_requests(self, abp_client): - """shutdown should cancel any pending requests.""" + """shutdown should clear any pending requests.""" await abp_client.connect() - # Create a pending future - future = MagicMock(done=MagicMock(return_value=False)) - future.cancel = MagicMock() - abp_client._pending_requests["req-1"] = future + # Add a pending request entry + abp_client._pending_requests["req-1"] = {"status": "pending"} await abp_client.shutdown() - future.cancel.assert_called_once() assert len(abp_client._pending_requests) == 0