scoringengine / scoringengine / 22558297470

90%

push

github

Fix CI test failures: coveralls, test hangs, and signal handler bug (#1148)

* Fix nightly CI failures caused by coveralls.io outage

- Add continue-on-error to coveralls upload step so third-party service
  outages don't fail the build
- Fix cache-dependency-paths → cache-dependency-path (renamed in
  actions/setup-python@v5)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* Fix CI unit test hangs and runner shutdown failures

In CI, engine.conf is gitignored so only engine.conf.inc is loaded,
which sets cache_type=redis. This causes Setting.get_setting() to
attempt Redis connections to localhost:6379 (nothing listening), adding
latency that combined with SQLite contention between parallel pytest
workers causes intermittent multi-minute test hangs and GitHub Actions
runner shutdowns.

- Set SCORINGENGINE_CACHE_TYPE=null in CI to disable Redis caching
- Add pytest-timeout (60s per test) to catch hanging tests early
- Add timeout-minutes: 10 to the step for clearer failure messages
- Fix engine signal handler bug: partial() passed obj= but the
  function parameter is named engine, causing TypeError on SIGTERM

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

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

2966 of 3919 relevant lines covered (75.68%)

0.76 hits per line