Alan-Jowett / CoPilot-For-Consensus / 20348032288 / 13
78%
main: 78%

DEFAULT BRANCH: main
Ran
Files 6
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 90.135%. Remained the same
20348032288.13

push

github

web-flow 
Replace return-code error signaling with exceptions to prevent silent failures (#395)

* Initial plan

* Add custom exception classes and refactor ingestion/parsing services

- Created custom exception hierarchies for ingestion and parsing services
- Refactored ingestion service to raise exceptions instead of returning False
- Updated ingestion main.py to handle exception-based results
- Refactored parsing service to raise exceptions for missing required fields
- Changed ingest_all_enabled_sources to return Dict[str, Optional[Exception]]

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

* Update parsing service and tests for exception-based error handling

- Updated parsing service to handle new exception-based parse_mbox interface
- Updated parsing tests to expect exceptions instead of error tuples
- Updated ingestion tests to validate exception-raising behavior
- Added new tests for exception scenarios (SourceConfigurationError, FetchError)

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

* Complete exception-based error handling refactor with documentation

- Created EXCEPTION_HANDLING_AUDIT.md documenting all changes
- Added comprehensive test demonstrating exception benefits
- All Python syntax validated successfully
- Migration guide included for developers

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

* Fix failing CI tests for exception-based error handling

- Updated ingestion integration tests to expect Dict[str, Optional[Exception]] instead of Dict[str, bool]
- Fixed test_archive_ingestion_failed_event_schema_validation to catch FetchError exception
- Removed tuple unpacking from parsing tests (parse_mbox no longer returns tuple)
- Updated test_message_without_message_id to expect MessageParsingError when all messages fail
- Fixed parsing service to not re-raise exceptions in process_archive (allows graceful event handling)

Co-authored-by: Alan-Jowett ... (continued)

201 of 223 relevant lines covered (90.13%)

0.9 hits per line

Source Files on job copilot_schema_validation - 20348032288.13