Skip to content

Fix 3 async tests.#368

Merged
Thump604 merged 1 commit intowaybarrios:mainfrom
perry2of5:switch-async-test-to-pytest-mark-anyio
Apr 18, 2026
Merged

Fix 3 async tests.#368
Thump604 merged 1 commit intowaybarrios:mainfrom
perry2of5:switch-async-test-to-pytest-mark-anyio

Conversation

@perry2of5
Copy link
Copy Markdown
Contributor

Switch from @pytest.mark.asyncio to @pytest.mark.anyio. The Apple Silicon CI runner only has anyio, not pytest-asyncio. Switching to @pytest.mark.anyio ensures tests can be run.

This partially fixes #364.

Thanks @Thump604 for the guidance.

Switch from @pytest.mark.asyncio to @pytest.mark.anyio. The Apple
Silicon CI runner only has anyio, not pytest-asyncio. Switching
to @pytest.mark.anyio ensures tests can be run.

This partially fixes waybarrios#364.
Copy link
Copy Markdown
Collaborator

@Thump604 Thump604 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Correct fix — same pattern we applied in #288 and #302. The Apple Silicon CI runner has anyio but not pytest-asyncio, so @pytest.mark.anyio is the right marker. All 9 checks green.

@Thump604 Thump604 merged commit 67632e9 into waybarrios:main Apr 18, 2026
9 checks passed
@perry2of5 perry2of5 deleted the switch-async-test-to-pytest-mark-anyio branch April 18, 2026 06:57
arozanov pushed a commit to arozanov/vllm-mlx that referenced this pull request Apr 30, 2026
Switch from @pytest.mark.asyncio to @pytest.mark.anyio. The Apple
Silicon CI runner only has anyio, not pytest-asyncio. Switching
to @pytest.mark.anyio ensures tests can be run.

This partially fixes waybarrios#364.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Build failures on main

2 participants