style: remove section-label comments from JMAP test files

Author: SashankBhamidi

tests/test_jmap_integration.py

@@ -23,10 +23,6 @@
 from caldav.jmap.constants import CALENDAR_CAPABILITY
 from caldav.jmap.session import fetch_session
 
-# ---------------------------------------------------------------------------
-# Skip the whole module if Cyrus is not reachable
-# ---------------------------------------------------------------------------
-
 CYRUS_HOST = "localhost"
 CYRUS_PORT = 8802
 JMAP_URL = f"http://{CYRUS_HOST}:{CYRUS_PORT}/.well-known/jmap"
@@ -50,10 +46,6 @@ def _cyrus_reachable() -> bool:
     "start it with: docker-compose -f tests/docker-test-servers/cyrus/docker-compose.yml up -d",
 )
 
-# ---------------------------------------------------------------------------
-# Fixtures
-# ---------------------------------------------------------------------------
-
 
 @pytest.fixture(scope="module")
 def client():
@@ -65,11 +57,6 @@ def session():
     return fetch_session(JMAP_URL, auth=HTTPBasicAuth(CYRUS_USERNAME, CYRUS_PASSWORD))
 
 
-# ---------------------------------------------------------------------------
-# Session
-# ---------------------------------------------------------------------------
-
-
 class TestJMAPSessionIntegration:
     def test_session_fetch_returns_api_url(self, session):
         assert session.api_url
@@ -82,11 +69,6 @@ def test_session_has_calendar_capability(self, session):
         assert CALENDAR_CAPABILITY in session.account_capabilities
 
 
-# ---------------------------------------------------------------------------
-# Calendar listing
-# ---------------------------------------------------------------------------
-
-
 class TestJMAPCalendarListIntegration:
     def test_list_calendars_returns_list(self, client):
         calendars = client.get_calendars()

tests/test_jmap_unit.py

@@ -14,19 +14,11 @@
 except ImportError:
     from requests.auth import HTTPBasicAuth  # type: ignore[no-redef]
 
-# ---------------------------------------------------------------------------
-# Shared test fixtures (module-level constants — not hardcoded inline)
-# ---------------------------------------------------------------------------
-
 _JMAP_URL = "http://localhost:8802/.well-known/jmap"
 _API_URL = "http://localhost:8802/jmap/api"
 _USERNAME = "user1"
 _PASSWORD = "x"
 
-# ---------------------------------------------------------------------------
-# Error hierarchy
-# ---------------------------------------------------------------------------
-
 from caldav.jmap.error import (
     JMAPAuthError,
     JMAPCapabilityError,
@@ -89,10 +81,6 @@ def test_jmap_auth_error_catchable_as_authorization_error(self):
             raise JMAPAuthError()
 
 
-# ---------------------------------------------------------------------------
-# Session establishment
-# ---------------------------------------------------------------------------
-
 from caldav.jmap.constants import CALENDAR_CAPABILITY, TASK_CAPABILITY
 from caldav.jmap.session import Session, fetch_session
 
@@ -219,10 +207,6 @@ def test_picks_first_calendar_capable_account(self):
         assert session.account_id == "user_calendar"
 
 
-# ---------------------------------------------------------------------------
-# Calendar domain object
-# ---------------------------------------------------------------------------
-
 from caldav.jmap.objects.calendar import JMAPCalendar
 
 _CALENDAR_JSON_FULL = {
@@ -300,10 +284,6 @@ def test_from_jmap_raises_when_name_missing(self):
             JMAPCalendar.from_jmap({"id": "cal3"})
 
 
-# ---------------------------------------------------------------------------
-# Calendar method builders and parsers
-# ---------------------------------------------------------------------------
-
 from caldav.jmap.methods.calendar import (
     build_calendar_changes,
     build_calendar_get,
@@ -355,10 +335,6 @@ def test_build_calendar_changes_structure(self):
         assert isinstance(call_id, str)
 
 
-# ---------------------------------------------------------------------------
-# JMAPClient
-# ---------------------------------------------------------------------------
-
 from caldav.jmap.client import JMAPClient
 
 _CALENDAR_GET_RESPONSE = {
@@ -471,10 +447,6 @@ def test_request_raises_method_error_on_error_response(self, monkeypatch):
         assert exc_info.value.error_type == "unknownMethod"
 
 
-# ---------------------------------------------------------------------------
-# get_jmap_client factory
-# ---------------------------------------------------------------------------
-
 from caldav.jmap import get_jmap_client
 
 
@@ -500,10 +472,6 @@ def test_strips_caldav_only_keys(self, monkeypatch):
         assert not hasattr(client, "ssl_verify_cert")
 
 
-# ---------------------------------------------------------------------------
-# CalendarEvent domain object
-# ---------------------------------------------------------------------------
-
 from caldav.jmap.objects.event import JMAPEvent
 
 _EVENT_JSON_FULL = {
@@ -746,10 +714,6 @@ def test_from_jmap_explicit_empty_collections_are_empty(self):
         assert ev.recurrence_overrides == {}
 
 
-# ---------------------------------------------------------------------------
-# CalendarEvent method builders and parsers
-# ---------------------------------------------------------------------------
-
 from caldav.jmap.methods.event import (
     build_event_changes,
     build_event_get,
@@ -971,10 +935,6 @@ def test_parse_event_set_partial_failure(self):
     _timedelta_to_duration,
 )
 
-# ---------------------------------------------------------------------------
-# Shared iCal fixtures
-# ---------------------------------------------------------------------------
-
 
 def _make_ical(extra_lines: str = "", uid: str = "test-uid@example.com") -> str:
     return (
@@ -1000,11 +960,6 @@ def _minimal_jscal(**kwargs) -> dict:
     return base
 
 
-# ---------------------------------------------------------------------------
-# TestUtils — shared utility functions
-# ---------------------------------------------------------------------------
-
-
 class TestUtils:
     def test_timedelta_to_duration_hours(self):
         assert _timedelta_to_duration(timedelta(hours=1, minutes=30)) == "PT1H30M"
@@ -1050,11 +1005,6 @@ def test_format_local_dt_date(self):
         assert _format_local_dt(d) == "2024-06-15T00:00:00"
 
 
-# ---------------------------------------------------------------------------
-# TestIcalToJscal
-# ---------------------------------------------------------------------------
-
-
 class TestIcalToJscal:
     def test_minimal_event(self):
         ical = _make_ical("DTSTART:20240615T100000Z\r\nDURATION:PT1H\r\nSUMMARY:Test Event\r\n")
@@ -1321,11 +1271,6 @@ def test_rrule_missing_freq_raises(self):
             ical_to_jscal(ical)
 
 
-# ---------------------------------------------------------------------------
-# TestJscalToIcal
-# ---------------------------------------------------------------------------
-
-
 class TestJscalToIcal:
     def test_minimal_event(self):
         jscal = _minimal_jscal()
@@ -1483,11 +1428,6 @@ def test_floating_datetime_emitted(self):
         assert "TZID" not in result
 
 
-# ---------------------------------------------------------------------------
-# TestRoundTrip
-# ---------------------------------------------------------------------------
-
-
 class TestRoundTrip:
     def _key_fields_survive(self, original_ical: str) -> dict:
         """ical → jscal → ical → parse back and check."""