From f3fa86f065eb7f066875fd024beb68910d2571e2 Mon Sep 17 00:00:00 2001 From: Mathieu Virbel Date: Thu, 11 Sep 2025 15:49:21 -0600 Subject: [PATCH] fix: update room ICS API tests to include required webhook fields and correct URL - Add webhook_url and webhook_secret fields to room creation tests - Fix room URL matching in ICS sync test to use UI_BASE_URL instead of BASE_URL - Aligns test with actual API requirements and ICS sync service implementation --- server/tests/test_room_ics_api.py | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/server/tests/test_room_ics_api.py b/server/tests/test_room_ics_api.py index 976fb915..27a784d7 100644 --- a/server/tests/test_room_ics_api.py +++ b/server/tests/test_room_ics_api.py @@ -36,6 +36,8 @@ async def test_create_room_with_ics_fields(authenticated_client): "recording_type": "cloud", "recording_trigger": "automatic-2nd-participant", "is_shared": False, + "webhook_url": "", + "webhook_secret": "", "ics_url": "https://calendar.example.com/test.ics", "ics_fetch_interval": 600, "ics_enabled": True, @@ -64,6 +66,8 @@ async def test_update_room_ics_configuration(authenticated_client): "recording_type": "cloud", "recording_trigger": "automatic-2nd-participant", "is_shared": False, + "webhook_url": "", + "webhook_secret": "", }, ) assert response.status_code == 200 @@ -108,7 +112,7 @@ async def test_trigger_ics_sync(authenticated_client): event.add("summary", "API Test Meeting") from reflector.settings import settings - event.add("location", f"{settings.BASE_URL}/{room.name}") + event.add("location", f"{settings.UI_BASE_URL}/{room.name}") now = datetime.now(timezone.utc) event.add("dtstart", now + timedelta(hours=1)) event.add("dtend", now + timedelta(hours=2))