Skip to content

Commit 7381afd

Browse files
committed
docs: testing guideline to extend existing tests instead of adding redundant ones
1 parent a99c1e8 commit 7381afd

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

AGENTS.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@
2828
- Add integration tests under `src/runtime/tests/` when changing CLI/runtime behavior.
2929
- Prefer `assert_cmd` for CLI and `predicates` for output checks. Keep fixtures in `src/runtime/tests/data/`.
3030
- Run `cargo test --workspace` locally; ensure tests don’t rely on network input.
31+
- Prefer modifying existing tests over adding new ones for new code paths. Extend current scenarios with extra assertions/fixtures to avoid redundant tests proliferating. For example, if adding null-handling in diff/patch, enhance the existing diff tests rather than introducing separate "basic diff works" tests that become redundant.
3132

3233
## Commit & Pull Request Guidelines
3334
- Commits: short, imperative summary (e.g., “Add __repr__ for LinkMLValue”); group related changes.

0 commit comments

Comments
 (0)