From 21d83b78c611ea63b375683d13d978686bff388d Mon Sep 17 00:00:00 2001 From: Nicco Kunzmann Date: Mon, 29 Sep 2025 13:33:41 +0100 Subject: [PATCH] Test no value deletion for refresh_interval see https://github.com/collective/icalendar/pull/877#discussion_r2372723774 --- src/icalendar/tests/test_rfc_7986.py | 15 +++++++++++++-- 1 file changed, 13 insertions(+), 2 deletions(-) diff --git a/src/icalendar/tests/test_rfc_7986.py b/src/icalendar/tests/test_rfc_7986.py index 60ee02d..9f95dce 100644 --- a/src/icalendar/tests/test_rfc_7986.py +++ b/src/icalendar/tests/test_rfc_7986.py @@ -231,13 +231,24 @@ def test_refresh_interval_default(calendar: Calendar): ) def test_invalid_refresh_interval_type(calendar: Calendar, invalid_value): """Invalid REFRESH-INTERVAL""" + calendar.refresh_interval = timedelta(hours=1) with pytest.raises(TypeError): calendar.refresh_interval = invalid_value + assert calendar.refresh_interval == timedelta(hours=1) -def test_invalid_refresh_interval(calendar: Calendar): +@pytest.mark.parametrize("invalid_value", [timedelta(seconds=-1), timedelta(seconds=0)]) +def test_invalid_refresh_interval(calendar: Calendar, invalid_value): + """Invalid REFRESH-INTERVAL. + + Test: + - value validation + - no deletion of the current value + """ + calendar.refresh_interval = timedelta(hours=2) with pytest.raises(ValueError): - calendar.refresh_interval = timedelta(seconds=-1) + calendar.refresh_interval = invalid_value + assert calendar.refresh_interval == timedelta(hours=2) def test_0_refresh_interval(calendar: Calendar):