FlexMeasures / flexmeasures-client / 35260004170

96%

Build Type

push

github

Committed by web-flow

Commit Message

Feat/update client with 429 and job progress (#225)

* fix(client): retry rate-limited requests using 429 / Retry-After; add sparse background-job progress notifications

Signed-off-by: Nicolas Höning <nicolas@seita.nl>

* chore: opt asset update call sites into JSON parsing

Signed-off-by: Nicolas Höning <nicolas@seita.nl>

* feat(hems): report rate-limit and background-job waits

Signed-off-by: Nicolas Höning <nicolas@seita.nl>

* fix(hems): stop aggregation after a site report fails

Signed-off-by: Nicolas Höning <nicolas@seita.nl>

* fix(client): preserve positional polling options

The notification callbacks were inserted ahead of polling settings that are already part of the v0.9.6 constructor signature. Positional callers would silently configure callbacks instead of their intended polling intervals, so keep the new notification settings keyword-only and cover the released positional layout with a regression test.

Signed-off-by: Mohamed Belhsan Hmida <mohamedbelhsanhmida@gmail.com>

* fix(client): qualify rate-limit recovery notices

Job status requests intentionally pass HTTP 422 responses through for wait_for_job to interpret. After a 429 retry, calling every such response successful could contradict the failed job result, so report the actual HTTP status for passed-through errors and retain success wording only for 2xx responses.

Signed-off-by: Mohamed Belhsan Hmida <mohamedbelhsanhmida@gmail.com>

---------

Signed-off-by: Nicolas Höning <nicolas@seita.nl>
Signed-off-by: Mohamed Belhsan Hmida <mohamedbelhsanhmida@gmail.com>
Co-authored-by: Mohamed Belhsan Hmida <mohamedbelhsanhmida@gmail.com>

Coverage Stats

78 of 84 new or added lines in 2 files covered. (92.86%)

5 existing lines in 1 file now uncovered.

886 of 923 relevant lines covered (95.99%)

9.6 hits per line