Skip to content

feat: Listen to 202 status#214

Merged
Flix6x merged 2 commits into
mainfrom
feat/listen-to-202-status-in-getschedule
Jun 12, 2026
Merged

feat: Listen to 202 status#214
Flix6x merged 2 commits into
mainfrom
feat/listen-to-202-status-in-getschedule

Conversation

@nhoening

Copy link
Copy Markdown
Member

The client now will react to all 202 responses and poll properly.

  • The client's response handling handles 202, meaning to poll once more. Also if maximal polling steps have passed, raise
  • add test for get_schedule() to see that all job states are recognized and polled correctly.
  • simplify get_schedule() as it was implemeting its own 202 polling (which went against the architecture idea of the client)

Closes #213 (the main purpose there is to listen to 202 status from the server's get_schedule endpoint)

nhoening added 2 commits June 11, 2026 11:53
Signed-off-by: Nicolas Höning <nicolas@seita.nl>
Signed-off-by: Nicolas Höning <nicolas@seita.nl>
@nhoening nhoening self-assigned this Jun 11, 2026
@coveralls

Copy link
Copy Markdown

Coverage Report for CI Build 27339519955

Coverage increased (+0.4%) to 96.591%

Details

  • Coverage increased (+0.4%) from the base build.
  • Patch coverage: 1 uncovered change across 1 file (18 of 19 lines covered, 94.74%).
  • No coverage regressions found.

Uncovered Changes

File Changed Covered %
src/flexmeasures_client/client.py 8 7 87.5%
Total (2 files) 19 18 94.74%

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 704
Covered Lines: 680
Line Coverage: 96.59%
Coverage Strength: 5.8 hits per line

💛 - Coveralls

@nhoening nhoening requested a review from Flix6x June 11, 2026 10:17
@Flix6x Flix6x merged commit 3825ab6 into main Jun 12, 2026
12 checks passed
@Flix6x Flix6x deleted the feat/listen-to-202-status-in-getschedule branch June 12, 2026 07:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Prepare client for HTTP 202 Accepted in get-schedule endpoint

3 participants