test(openai): Replace mocks with httpx types for streaming Responses#5882
Conversation
Semver Impact of This PR🟢 Patch (bug fixes) 📋 Changelog PreviewThis is how your changes will appear in the changelog. New Features ✨Langchain
Bug Fixes 🐛Ci
Openai
Other
Internal Changes 🔧Langchain
Openai
Other
Other
🤖 This preview updates automatically when you update the PR. |
Codecov Results 📊✅ 85 passed | ⏭️ 12 skipped | Total: 97 | Pass Rate: 87.63% | Execution Time: 26.28s 📊 Comparison with Base Branch
✨ No test changes detected All tests are passing successfully. ✅ Patch coverage is 100.00%. Project has 15181 uncovered lines. Coverage diff@@ Coverage Diff @@
## main #PR +/-##
==========================================
+ Coverage 26.48% 26.48% —%
==========================================
Files 189 189 —
Lines 20648 20648 —
Branches 6760 6760 —
==========================================
+ Hits 5467 5467 —
- Misses 15181 15181 —
- Partials 479 479 —Generated by Codecov Action |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 2 potential issues.
Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
ericapisani
left a comment
There was a problem hiding this comment.
One cleanup thing, but otherwise LGTM

Description
Replace mocks with httpx types to avoid test failures when library internals change.
Issues
Reminders
tox -e linters.feat:,fix:,ref:,meta:)