FlexMeasures / flexmeasures / 34319769880

85%

main: 88%

push

github

Reject a resolution that spans no positive time (#2493)

`GET /sensors/<id>/data?resolution=PT0S` crashed with an uncaught
`ZeroDivisionError` from pandas, and was reported as a bare 500,
because `DurationField` only rejects durations that are not a multiple
of 1 minute, which a zero duration passes.

Add a `ResolutionField`, which additionally requires the duration to
span a positive amount of time, and use it for every `resolution`
query parameter the API takes.
A negative resolution was accepted too, and quietly described an empty
set of events; it is now rejected the same way.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01RgnoDDysWF5CMi3k827rHM
Signed-off-by: F.N. Claessen <felix@seita.nl>

15 of 15 new or added lines in 5 files covered. (100.0%)

18490 of 21748 relevant lines covered (85.02%)

0.85 hits per line