FlexMeasures / flexmeasures-client / 28551102362

96%

push

github

Make HEMS report generation dockerizable + fix false-negative on async uploads (#216)

* feat(HEMS): allow overriding the CLI command used for report generation

The HEMS tutorial's report-generation step shells out to a bare
`flexmeasures` on PATH, which only works if that CLI is configured
against the same database as the server being scripted against.
Add FLEXMEASURES_CLI_CMD and FLEXMEASURES_CLI_CONFIG_DIR env vars so
callers can point report generation at a `flexmeasures` running
elsewhere, e.g. inside a Docker Compose service
(`docker compose exec -T server flexmeasures`), which unblocks
running this step as part of automated release QA.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

* fix: treat any 2xx status as success for sensor data file upload

_post_sensor_data_file() hardcoded `status != 200` as the failure
condition, but the sensors/<id>/data/upload endpoint can legitimately
return 202 Accepted when the server processes the upload as an async
job. This made every file upload look like a failure whenever the
server queues the work, which is what silently broke the HEMS
tutorial's data-loading step (PART 2) before it ever reached
forecasting/scheduling/reporting.

Match check_for_status()'s existing convention elsewhere in this
client: accept any 2xx, only raise on a genuine non-2xx status.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>

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

688 of 714 relevant lines covered (96.36%)

2.89 hits per line