diff --git a/AGENTS.md b/AGENTS.md index c13a75c396..3dc1d79c72 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -65,6 +65,7 @@ lives under `pyiceberg/`, organized by concern rather than by engine: - Bias towards adding tests to existing files, rather than creating new files. - Use existing test fixtures when possible. - We have a strong bias towards integration testing over mocks. Mocks should be avoided whenever possible and should only be used if similar, existing tests are using mocks. +- Prefer a `for` loop over `@pytest.mark.parametrize` for many simple cases; each parametrized case adds per-test overhead. Use `parametrize` when cases need different fixtures, and separate tests when cases are complex. ### Required CI and Merge Queue