Alan-Jowett / CoPilot-For-Consensus / 20383542568

78%

push

github

Fix reporting service calling non-existent capture_exception() method (#473)

* Initial plan

* Fix capture_exception() calls - replace with report() method

- Added sys import (though not strictly needed for current implementation)
- Replaced all three capture_exception() calls with proper report() calls
- Fixed line 278: Use publish_error from exception handler with context
- Fixed line 349: Capture exception as 'e' and pass to report() with context
- Fixed line 391: Capture exception as 'e' and pass to report() with context
- Added comprehensive tests for error reporter usage in exception scenarios
- All existing tests pass, verifying no regressions

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

* Remove unused sys import

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

* Improve test: use Mock side_effect instead of subclassing

Address code review comment - use more conventional pytest/Mock pattern

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

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: Alan-Jowett <20480683+Alan-Jowett@users.noreply.github.com>

5 of 5 new or added lines in 1 file covered. (100.0%)

5136 of 6406 relevant lines covered (80.17%)

0.9 hits per line