-
Notifications
You must be signed in to change notification settings - Fork 257
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
Add OpenAI Integration Test (Sync/Async + Streaming) #598
Conversation
- Add integration test demonstrating four OpenAI call patterns: - Sync (non-streaming) - Sync (streaming) - Async (non-streaming) - Async (streaming) - Add python-dotenv to dev and test dependencies Co-Authored-By: Alex Reibman <[email protected]>
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
⚙️ Control Options:
|
WalkthroughThis update introduces a new integration test for OpenAI, covering four call patterns: synchronous non-streaming, synchronous streaming, asynchronous non-streaming, and asynchronous streaming. The tests ensure accurate tracking by AgentOps analytics. Additionally, the Changes
🔗 Related PRs
InstructionsEmoji Descriptions:
Interact with the Bot:
Execute a command using the format:
Available Commands:
Tips for Using @bot Effectively:
Need More Help?📚 Visit our documentation for detailed guides on using Entelligence.AI. |
Co-Authored-By: Alex Reibman <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved but IDK 😆
🔍 Review Summary
Purpose
Enhance testing coverage with new integration tests and improve development environments management.
Changes
python-dotenv
for efficient environment variable management in development and testing.Impact
These updates increase system reliability by expanding test coverage and ensuring consistent environment variable management.
Original Description
OpenAI Integration Test
This PR adds an integration test that calls OpenAI in four ways:
The test verifies that AgentOps correctly tracks all LLM calls by checking the analytics at the end of the session.
Changes
tests/openai_handlers/test_openai_integration.py
python-dotenv
to dev and test dependencies for environment variable managementTesting
Link to Devin run: https://app.devin.ai/sessions/27239b4aa9fa4638bec5b425f5cdee9c