10000 Add pytest-asyncio support for integration tests by ddkang1 · Pull Request #25 · ddkang1/smart-agent · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Add pytest-asyncio support for integration tests #25

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 2, 2025

Conversation

ddkang1
Copy link
Owner
@ddkang1 ddkang1 commented Apr 2, 2025

This PR adds proper support for pytest-asyncio to fix the integration tests.

Key changes:

  1. Register the asyncio marker in pytest.ini:

    • Added the asyncio marker to the list of registered markers
    • This fixes the 'asyncio not found in configuration option' error
  2. Add pytest-asyncio as a dependency:

    • Added to GitHub Actions workflow
    • Added to setup_dev.sh script
    • Created a new 'dev' optional dependencies group in pyproject.toml

This change aligns with the Smart Agent's unified approach to tool launching and testing by ensuring that asynchronous tests can run properly in the CI/CD pipeline.

@ddkang1 ddkang1 merged commit 465109e into main Apr 2, 2025
2 checks passed
@ddkang1 ddkang1 deleted the fix-pytest-asyncio branch April 2, 2025 09:23
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.

1 participant
0