Alan-Jowett / CoPilot-For-Consensus / 20211388655

79%

push

github

Add comprehensive startup dependency validation to chunking and reporting services (#169)

* Initial plan

* Add startup dependency validation to chunking and reporting services

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Add dependency validation documentation

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Fix code review issues: formatting and test logic improvements

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Add schema and permission validation at startup

- Validate document store read/write permissions with test document
- Validate all required event schemas can be loaded at startup
- Add tests for schema validation failures
- Add tests for permission validation failures
- Update documentation with new validation requirements

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Fix test mocking to patch imports in service modules

- Fix all test patches to target service_name.main.function instead of copilot_*.function
- Remove unused MagicMock imports from test files
- Update documentation example to show correct patching pattern
- Add note about patching where functions are used vs where defined

Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

* Fix chunking startup validation tests - update module patch paths

- Change patch paths from 'app.main.*' to 'main.*' for correct module reference
- Update imports from 'from app import main' to 'import main' since main.py is at root
- All 6 startup validation tests now passing with correct mocking
- All 26 chunking service tests passing

* Fix reporting startup validation tests - update module patch paths

- Change patch paths from 'reporting.main.*' to 'main.*' for correct module reference
- Update imports from 'from reporting import main' to 'import main' since main.py is at root
- All 6 startup validation tests now passing with correct mock... (continued)

4 of 9 new or added lines in 1 file covered. (44.44%)

186 of 213 relevant lines covered (87.32%)

1.78 hits per line