Skip to content

Commit 95060f1

Browse files
test(suspension): fix test_is_suspended_normal and add before-load warning test
- Add explicit source='none' to test_is_suspended_normal assertion to verify the loaded non-holiday path (not the coincidentally-matching unloaded path) - Add test_is_suspended_before_load_warns to assert the warning fires and source='none' is returned when is_suspended() is called before load() Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent e09cd60 commit 95060f1

1 file changed

Lines changed: 12 additions & 1 deletion

File tree

tests/test_suspension.py

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,18 @@ def test_is_suspended_normal(ics_bytes: bytes) -> None:
6666
cal._holidays = _parse_ics(ics_bytes)
6767
cal._loaded = True
6868
result = cal.is_suspended(date(2026, 6, 3))
69-
assert result == SuspensionInfo(is_suspended=False, reason=None)
69+
assert result == SuspensionInfo(is_suspended=False, reason=None, source="none")
70+
71+
72+
def test_is_suspended_before_load_warns(caplog: pytest.LogCaptureFixture) -> None:
73+
"""is_suspended() before load() logs a warning and returns source='none'."""
74+
import logging
75+
76+
cal = HolidayCalendar()
77+
with caplog.at_level(logging.WARNING, logger="gps2asp.suspension"):
78+
result = cal.is_suspended(date(2026, 1, 1))
79+
assert any("before load()" in r.message for r in caplog.records)
80+
assert result == SuspensionInfo(is_suspended=False, reason=None, source="none")
7081

7182

7283
# --- load() with mocked httpx ---

0 commit comments

Comments
 (0)