FlexMeasures / flexmeasures / 29183674716

81%

main: 81%

Build Type

Pull #2303

github

Committed by Flix6x

Commit Message

fix: three more long-standing failures on main

- [GET] /api/ops/getLatestTaskRun turned any database error into an opaque
  500: the retry in get_task_run() ran on the still-aborted transaction,
  so it was rejected with InFailedSqlTransaction and the original error was
  never surfaced. Roll back before retrying.
- test_api_task_run_get_recent_entry did not declare the fixture seeding its
  data, so it queried a database without the latest_task_run table at all
  (which is what aborted the transaction above).
- test_build_asset_jobs_data and test_get_job_status_requires_read_access
  asserted on job counts and job identity without flushing the Redis job
  cache, so leftover jobs from earlier tests (which @job_cache also reuses
  for identical requests) made them fail depending on test order. Both now
  use the existing clean_redis fixture.

Fixes #2302

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Rbix8k1JfeUWNXEmHEZVpX

Pull Request Pull Request #2303: Fix three long-standing failures on main (regressor splits on pandas 2.2+, swallowed CLI validation messages, task page 500)

Coverage Stats

25 of 29 new or added lines in 4 files covered. (86.21%)

273 existing lines in 10 files now uncovered.

14315 of 17610 relevant lines covered (81.29%)

0.81 hits per line