The actor_compute_impact_error_handling.feature file references the
'Given an actor CLI runner' step in its Background section, but the
step definition was missing from the step file. This caused the tests
to fail with an undefined step error.
Added the missing step definition that initializes a CliRunner context
for testing. Also removed the unused noqa comment from the CliRunner
import since the import is now used in the step definition.
Replace three bare 'except Exception: pass' blocks in _compute_actor_impact()
with proper exception handling that logs at WARNING level with exception type
and message for diagnostics. The function still returns (0, 0, 0) on failure
(graceful degradation) but failures are now visible in logs.
Also adds BDD scenarios covering the error paths (DB unavailable -> warning
logged, counts return 0) and removes the pragma: no cover annotations from
the exception handlers.
ISSUES CLOSED: #8434
HAL9000
Test User
Test User