Skip to content

Commit b817a12

Browse files
committed
CLAUDE.md: Don't use mocks
1 parent 3f9d13a commit b817a12

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

CLAUDE.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -72,8 +72,8 @@ When implementing new loaders:
7272
5. Follow existing patterns from PostgreSQL and Redis loaders
7373

7474
### Testing Strategy
75-
- **Unit tests**: Mock external dependencies, test business logic
76-
- **Integration tests**: Use testcontainers for real database testing
75+
- **Unit tests**: Test pure logic and data structures WITHOUT mocking. Unit tests should be simple, fast, and test isolated components (dataclasses, utility functions, partitioning logic, etc.). Do NOT add tests that require mocking to `tests/unit/`.
76+
- **Integration tests**: Use testcontainers for real database testing. Tests that require external dependencies (databases, Flight SQL server, etc.) belong in `tests/integration/`.
7777
- **Performance tests**: Benchmark data loading operations
7878
- Tests can be filtered using pytest markers (e.g., `-m unit` for unit tests only)
7979

0 commit comments

Comments
 (0)