Reorganize tests into logical subdirectories#2071
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
- Group integration tests by category: collision, constraint, dynamics, io, optimization, simulation, utils - Organize unit tests: collision, constraint, dynamics, lcpsolver - Rename regression tests from Issue#### to descriptive names - Consolidate test helpers into tests/helpers/ - Remove deprecated tests - Update CMake configuration
71ae251 to
15b03b7
Compare
|
@codex review |
|
Codex Review: Didn't find any major issues. Another round soon, please! ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2071 +/- ##
==========================================
- Coverage 59.06% 58.47% -0.59%
==========================================
Files 386 386
Lines 32475 32475
Branches 3969 3964 -5
==========================================
- Hits 19182 18991 -191
- Misses 13293 13484 +191
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
Before creating a pull request
pixi run test-allto lint, build, and test your changes-DDART_TREAT_WARNINGS_AS_ERRORS=ONand resolve warningsBefore merging
CHANGELOG.md