test(langchain): Add text completion test #5740
Merged
@sentry/warden / warden
completed
Mar 24, 2026 in 2m 43s
1 issue
Medium
Deleted test for langchain-core-only installations has no replacement - `tests/integrations/langchain/test_langchain.py:1093-1098`
The removed test_langchain_integration_with_langchain_core_only test validated a specific edge case: the integration working when langchain.agents.AgentExecutor is unavailable but langchain-core is installed. The new test_langchain_text_completion test uses the full langchain library flow but doesn't cover this compatibility scenario. Users who only install langchain-core without the full langchain package may not have regression test coverage.
4 skills analyzed
| Skill | Findings | Duration | Cost |
|---|---|---|---|
| code-review | 1 | 2m 24s | $0.83 |
| find-bugs | 0 | 2m 37s | $1.46 |
| skill-scanner | 0 | 1m 7s | $0.34 |
| security-review | 0 | 1m 3s | $0.62 |
Duration: 7m 11s · Tokens: 1.7M in / 14.7k out · Cost: $3.26 (+extraction: $0.00, +dedup: $0.01)
Loading