Skip to content

Commit c1acd91

Browse files
committed
fix: update OneBot tests to mock _post_json instead of post_data
1 parent e864f38 commit c1acd91

1 file changed

Lines changed: 6 additions & 6 deletions

File tree

backend/src/test/test_notification.py

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -317,7 +317,7 @@ async def test_send_private_message(self, provider):
317317
official_title="Test Anime", season=1, episode=5, poster_path="https://example.com/poster.jpg"
318318
)
319319

320-
with patch.object(provider, "post_data", new_callable=AsyncMock) as mock_post:
320+
with patch.object(provider, "_post_json", new_callable=AsyncMock) as mock_post:
321321
mock_response = MagicMock(status_code=200)
322322
mock_response.json.return_value = {"status": "ok", "retcode": 0}
323323
mock_post.return_value = mock_response
@@ -335,7 +335,7 @@ async def test_send_group_message(self, provider):
335335
official_title="Test Anime", season=1, episode=5
336336
)
337337

338-
with patch.object(provider, "post_data", new_callable=AsyncMock) as mock_post:
338+
with patch.object(provider, "_post_json", new_callable=AsyncMock) as mock_post:
339339
mock_response = MagicMock(status_code=200)
340340
mock_response.json.return_value = {"status": "ok", "retcode": 0}
341341
mock_post.return_value = mock_response
@@ -350,7 +350,7 @@ async def test_send_api_error(self, provider):
350350
"""Handles OneBot API error response."""
351351
notify = Notification(official_title="Test Anime", season=1, episode=5)
352352

353-
with patch.object(provider, "post_data", new_callable=AsyncMock) as mock_post:
353+
with patch.object(provider, "_post_json", new_callable=AsyncMock) as mock_post:
354354
mock_response = MagicMock(status_code=200)
355355
mock_response.json.return_value = {"status": "failed", "retcode": 100, "msg": "bad request"}
356356
mock_post.return_value = mock_response
@@ -362,15 +362,15 @@ async def test_send_http_error(self, provider):
362362
"""Handles HTTP error."""
363363
notify = Notification(official_title="Test Anime", season=1, episode=5)
364364

365-
with patch.object(provider, "post_data", new_callable=AsyncMock) as mock_post:
365+
with patch.object(provider, "_post_json", new_callable=AsyncMock) as mock_post:
366366
mock_post.return_value = MagicMock(status_code=401)
367367
result = await provider.send(notify)
368368

369369
assert result is False
370370

371371
async def test_test_success(self, provider):
372372
"""Test method sends test message successfully."""
373-
with patch.object(provider, "post_data", new_callable=AsyncMock) as mock_post:
373+
with patch.object(provider, "_post_json", new_callable=AsyncMock) as mock_post:
374374
mock_response = MagicMock(status_code=200)
375375
mock_response.json.return_value = {"status": "ok", "retcode": 0}
376376
mock_post.return_value = mock_response
@@ -381,7 +381,7 @@ async def test_test_success(self, provider):
381381

382382
async def test_test_failure(self, provider):
383383
"""Test method handles API error."""
384-
with patch.object(provider, "post_data", new_callable=AsyncMock) as mock_post:
384+
with patch.object(provider, "_post_json", new_callable=AsyncMock) as mock_post:
385385
mock_response = MagicMock(status_code=200)
386386
mock_response.json.return_value = {"status": "failed", "retcode": 100, "wording": "invalid target"}
387387
mock_post.return_value = mock_response

0 commit comments

Comments
 (0)