-
-
Notifications
You must be signed in to change notification settings - Fork 0
refactor: migrate test files to colocated structure in src/
#33
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #33 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 47 47
Lines 552 552
=========================================
Hits 552 552 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR refactors the test structure by migrating test files from a separate test/ directory to a colocated structure within src/. Tests are now placed alongside their corresponding source files following the pattern test_*.py.
Key changes:
- Moved all test files to
src/directory, colocated with source code - Updated pytest configuration to discover tests in
src/directory withtest_*.pypattern - Updated coverage configuration to omit test files and conftest.py
- Updated documentation in CLAUDE.md to reflect new colocated testing structure
Reviewed changes
Copilot reviewed 2 out of 46 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| test/init.py | Removed old test directory structure |
| test/common/init.py | Removed old test directory structure |
| test/common/di/init.py | Removed old test directory structure |
| test/system/init.py | Removed old test directory structure |
| test/system/di/init.py | Removed old test directory structure |
| test/system/di/module/init.py | Removed old test directory structure |
| test/system/di/module/infrastructure/init.py | Removed old test directory structure |
| test/system/di/module/infrastructure/repository/init.py | Removed old test directory structure |
| test/system/di/module/infrastructure/repository/sqlalchemy/init.py | Removed old test directory structure |
| test/system/domain/init.py | Removed old test directory structure |
| test/system/domain/interface/init.py | Removed old test directory structure |
| test/system/domain/interface/repository/init.py | Removed old test directory structure |
| test/system/domain/interface/repository/common/init.py | Removed old test directory structure |
| test/system/infrastructure/init.py | Removed old test directory structure |
| test/system/infrastructure/ext/init.py | Removed old test directory structure |
| test/system/infrastructure/ext/logfire/init.py | Removed old test directory structure |
| test/system/infrastructure/ext/sentry/init.py | Removed old test directory structure |
| test/system/infrastructure/repository/init.py | Removed old test directory structure |
| test/system/infrastructure/repository/sqlalchemy/init.py | Removed old test directory structure |
| test/system/infrastructure/repository/sqlalchemy/crud/init.py | Removed old test directory structure |
| test/system/infrastructure/repository/sqlalchemy/translator/init.py | Removed old test directory structure |
| test/system/infrastructure/repository/sqlalchemy/type/init.py | Removed old test directory structure |
| test/system/ui/init.py | Removed old test directory structure |
| test/system/ui/discord/init.py | Removed old test directory structure |
| test/system/usecase/base/init.py | Removed old test directory structure |
| test/system/util/init.py | Removed old test directory structure |
| src/system/usecase/base/test_interface.py | Test for usecase interface abstraction |
| src/system/usecase/base/test_dto.py | Tests for usecase DTO classes |
| src/system/ui/discord/test_config.py | Test for Discord configuration interface |
| src/system/infrastructure/repository/sqlalchemy/type/test_ulid.py | Tests for ULID column type converter |
| src/system/infrastructure/repository/sqlalchemy/translator/test_user.py | Tests for User domain/DB translator |
| src/system/infrastructure/repository/sqlalchemy/translator/test_nickname.py | Tests for NicknameChangelog domain/DB translator |
| src/system/infrastructure/repository/sqlalchemy/translator/test_messenger.py | Tests for Messenger domain/DB translator |
| src/system/infrastructure/repository/sqlalchemy/translator/test_base.py | Test for base translator abstraction |
| src/system/infrastructure/repository/sqlalchemy/test_config.py | Test for SQLAlchemy configuration interface |
| src/system/infrastructure/repository/sqlalchemy/crud/test_user.py | Tests for User repository CRUD operations |
| src/system/infrastructure/repository/sqlalchemy/crud/test_nickname.py | Tests for NicknameChangelog repository operations |
| src/system/infrastructure/repository/sqlalchemy/crud/test_messenger.py | Tests for Messenger repository operations |
| src/system/domain/test_config.py | Tests for domain configuration and environment enum |
| src/system/domain/interface/repository/common/test_response.py | Tests for repository response data structures |
| src/system/di/test_container.py | Test for dependency injection container |
| src/system/di/module/infrastructure/repository/sqlalchemy/test_module.py | Tests for SQLAlchemy repository DI module |
| src/conftest.py | Shared pytest fixtures for ULID and dialect |
| src/common/di/test_builder.py | Tests for DI builder utilities |
| pyproject.toml | Updated pytest and coverage config for colocated tests |
| CLAUDE.md | Updated documentation to reflect colocated test structure |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.