test(langchain): Separate test to remove conditional#6076
test(langchain): Separate test to remove conditional#6076alexander-alderman-webb wants to merge 2 commits intowebb/langchain/tool-pipeline-namefrom
Conversation
Semver Impact of This PR🟢 Patch (bug fixes) 📋 Changelog PreviewThis is how your changes will appear in the changelog. New Features ✨Langchain
Other
Bug Fixes 🐛
Internal Changes 🔧
🤖 This preview updates automatically when you update the PR. |
Codecov Results 📊✅ 142 passed | Total: 142 | Pass Rate: 100% | Execution Time: 23.84s 📊 Comparison with Base Branch
All tests are passing successfully. ✅ Patch coverage is 100.00%. Project has 14086 uncovered lines. Coverage diff@@ Coverage Diff @@
## main #PR +/-##
==========================================
+ Coverage 27.64% 33.74% +6.1%
==========================================
Files 190 191 +1
Lines 21358 21259 -99
Branches 7066 7008 -58
==========================================
+ Hits 5903 7173 +1270
- Misses 15455 14086 -1369
- Partials 561 700 +139Generated by Codecov Action |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 1e622db. Configure here.

Description
Dedicate
test_langchain_openai_tools_agent_no_prompts()to all configuration combinations that omit sensitive data, and keeptest_langchain_openai_tools_agent()focused on the sensitive data path.Issues
Reminders
tox -e linters.feat:,fix:,ref:,meta:)