fix(caldav): ignorer les événements gérés sans UID

Closes #47 Co-authored-by: Codex <codex@antoineve.me>
2026-09-13 00:13:29 +02:00
parent 50c3696b06
commit e6f0659cbf
3 changed files with 38 additions and 3 deletions
@@ -327,6 +327,37 @@ def test_list_managed_events_returns_only_managed(
    assert str(vevent.get("UID")) == "test-uid-123"


+def test_list_managed_events_ignores_managed_event_without_uid(
+    caldav_settings: CalDAVSettings,
+    mock_client_factory: MagicMock,
+    caplog: LogCaptureFixture,
+) -> None:
+    """Ignore un événement géré sans UID et ne le transmet pas au planificateur.
+
+    :param caldav_settings: Paramètres CalDAV valides.
+    :param mock_client_factory: Usine de clients CalDAV mockée.
+    :param caplog: Capture des journaux de diagnostic.
+    :return: None
+    """
+    gateway = CalDAVGateway(caldav_settings, client_factory=mock_client_factory)
+    gateway.connect()
+
+    malformed_event = Event()
+    malformed_event.add("SUMMARY", "Événement sans identifiant")
+    malformed_event.add(MANAGED_PROPERTY, MANAGED_VALUE)
+    remote_event = MagicMock()
+    remote_event.icalendar_component = Calendar()
+    remote_event.icalendar_component.add_component(malformed_event)
+    calendar = mock_client_factory.return_value.principal.return_value.calendars.return_value[0]
+    calendar.search.return_value = [remote_event]
+
+    with caplog.at_level(logging.WARNING):
+        result = gateway.list_managed_events(datetime(2026, 1, 1), datetime(2026, 12, 31))
+
+    assert result == []
+    assert "sans UID ignoré" in caplog.text
+
+
 def test_list_managed_events_not_connected_raises(
    caldav_settings: CalDAVSettings,
 ) -> None: