Fix async test methods missing pytest.mark.asyncio decorator #21

Closed
opened 2026-02-20 06:53:54 +00:00 by cal · 1 comment
Owner

tests/test_automated_data_fetcher.py:162-220 — Multiple async def test methods without @pytest.mark.asyncio. Pytest will collect and "pass" them without running the coroutine, giving false confidence in coverage.

Priority: medium | Labels: bug, tech-debt

`tests/test_automated_data_fetcher.py:162-220` — Multiple `async def` test methods without `@pytest.mark.asyncio`. Pytest will collect and "pass" them without running the coroutine, giving false confidence in coverage. **Priority**: medium | **Labels**: bug, tech-debt
Claude added the
ai-working
label 2026-03-20 19:31:25 +00:00
Collaborator

Fixed in PR #30: #30

Added @pytest.mark.asyncio to all 14 async test methods in tests/test_automated_data_fetcher.py. The decorator was missing from every async def test_* method, meaning pytest was silently passing them without executing the coroutine bodies.

Fixed in PR #30: https://git.manticorum.com/cal/paper-dynasty-card-creation/pulls/30 Added `@pytest.mark.asyncio` to all 14 async test methods in `tests/test_automated_data_fetcher.py`. The decorator was missing from every `async def test_*` method, meaning pytest was silently passing them without executing the coroutine bodies.
Claude added
ai-reviewed
ai-pr-opened
and removed
ai-working
labels 2026-03-20 19:34:19 +00:00
cal closed this issue 2026-03-23 13:25:12 +00:00
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: cal/paper-dynasty-card-creation#21
No description provided.