collective / icalendar / 32126729299

98%

push

github

fix: validate non-negative integer values for repeat, sequence, priority (#1598)

* fix: validate Alarm repeat value is non-negative

Alarm.REPEAT now validates the repeat value is an integer >= 0,
raising InvalidCalendar if a negative or non-integer value is provided.
Adds optional min_value parameter to single_int_property().

Fixes #1594

* Address review: fix min_value validation, add tests

* Address review: raise TypeError for non-integer and bool values

* Address review: fix Sphinx docstring formatting, tidy tests, update news entry

* Address review: simplify Raises docstring per suggestion

* Address review: wrap Raises field, use versionchanged for behavior notes

* Address review: retain both sentences in versionadded notes

* Address review: align version notes and add Raises sections

* Address review: move Raises docs to property docstrings

---------

Co-authored-by: Sashank <hello@sashank.wiki>
Co-authored-by: Steve Piercy <web@stevepiercy.com>

862 of 882 branches covered (97.73%)

Branch coverage included in aggregate %.

81 of 81 new or added lines in 3 files covered. (100.0%)

14180 of 14450 relevant lines covered (98.13%)

6.86 hits per line