test: declare test runner dependencies - #388
Conversation
|
Warning Review limit reachedNext included review available in 59 minutes. View limit detailsLimit details: You’ve used the included review currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #388 +/- ##
==========================================
- Coverage 84.43% 84.42% -0.01%
==========================================
Files 104 104
Lines 6110 6107 -3
==========================================
- Hits 5159 5156 -3
Misses 951 951 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Add coverage, mock, and pytest to the test extra and make CI rely on that advertised installation path. Closes deepmodeling#362 Coding-Agent: Codex Codex-Version: codex-cli 0.149.1 Model: gpt-5.6-sol Reasoning-Effort: xhigh
0d5ebc4 to
fb8c9e3
Compare
Summary
coverage,mock, andpytestto thetestextrajsonpickleis a runtime import and is declared separately by PR #385 for issue #359.Tests
pyproject.tomlwithtomlliband verified the direct test dependencies.github/workflows/test.ymlwith PyYAMLgit diff --checkCloses #362
Coding agent: Codex
Codex version: codex-cli 0.149.0
Model: gpt-5.6-sol
Reasoning effort: xhigh