ci: fix scheduling tests in GitHub CI for Nextcloud and Cyrus

Nextcloud: add email addresses for scheduling test users (user1-user3).
Without email addresses, calendar-user-address-set lacks the mailto: entry
required for iTIP invite delivery to work via CalDAV.

Cyrus: copy imapd.conf with virtdomains: off before waiting for CalDAV.
The default virtdomains: userid causes caladdress_lookup() to retain the
full email form (user2@example.com) as the userid, but mailbox ACLs use
the short form (user2), causing 403 when delivering iTIP invites.  The
local docker-compose setup mounts the custom imapd.conf at startup; in CI
we copy it to the running service container and restart it.

Also fix two test failures unrelated to scheduling:

Nextcloud cleanup-regime: calendar deletion goes to a trashbin, so
delete-and-recreate does not produce a fresh empty calendar.  Use
"cleanup-regime: wipe-calendar" to wipe objects instead.

Cyrus testRecurringDateWithExceptionSearch: Cyrus stores exception VEVENTs
as separate calendar object resources, so client-side expansion cannot
produce correct RECURRENCE-ID values.  Mark the feature unsupported and
gate the assertion on it.

prompt: fix the github ci failures for scheduling tests (nextcloud email
addresses for scheduling users, cyrus virtdomains setting); commit all
pending changes together

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: tobixen

.github/workflows/tests.yaml

@@ -154,6 +154,11 @@ jobs:
             docker exec -e OC_PASS="testpass${i}" ${{ job.services.nextcloud.id }} php occ user:add --password-from-env --display-name="User ${i}" "user${i}" || echo "user${i} may already exist"
           done
 
+          # Set email addresses for scheduling users (required for calendar-user-address-set)
+          for i in 1 2 3; do
+            docker exec ${{ job.services.nextcloud.id }} php occ user:setting "user${i}" settings email "user${i}@localhost" || true
+          done
+
           # Enable calendar and contacts apps
           docker exec ${{ job.services.nextcloud.id }} php occ app:enable calendar || true
           docker exec ${{ job.services.nextcloud.id }} php occ app:enable contacts || true
@@ -180,6 +185,17 @@ jobs:
           " || true
 
           echo "Nextcloud is configured!"
+      - name: Configure Cyrus
+        run: |
+          # Copy imapd.conf with virtdomains: off (required for iTIP scheduling delivery).
+          # The default virtdomains: userid setting causes caladdress_lookup() to preserve
+          # the full email form (user2@example.com) while mailbox ACLs use the short form
+          # (user2), resulting in 403 errors when delivering iTIP invites.
+          sed 's/{{DEFAULTDOMAIN}}/example.com/g; s/{{SERVERNAME}}/cyrus-test/g' \
+            tests/docker-test-servers/cyrus/imapd.conf > /tmp/imapd_expanded.conf
+          docker cp /tmp/imapd_expanded.conf ${{ job.services.cyrus.id }}:/srv/cyrus-docker-test-server.git/imapd.conf
+          docker restart ${{ job.services.cyrus.id }}
+          echo "✓ Cyrus reconfigured with virtdomains: off"
       - name: Wait for Cyrus to be ready
         run: |
           echo "Waiting for Cyrus server..."

caldav/compatibility_hints.py

@@ -959,6 +959,9 @@ def dotted_feature_set_list(self, compact=False):
         'behaviour': "deleting a calendar moves it to a trashbin, thrashbin has to be manually 'emptied' from the web-ui before the namespace is freed up",
         'support': 'fragile',
     },
+    # Calendar deletion goes to trashbin so delete-and-recreate doesn't give a
+    # fresh empty calendar.  Wipe objects instead of deleting the calendar itself.
+    "test-calendar": {"cleanup-regime": "wipe-calendar"},
     'search.recurrences.includes-implicit.todo': {'support': 'unsupported'},
     #'save-load.todo.mixed-calendar': {'support': 'unsupported'}, ## Why?  It started complaining about this just recently.
     'principal-search.by-name.self': {'support': 'unsupported'},
@@ -1145,7 +1148,11 @@ def dotted_feature_set_list(self, compact=False):
     # Cyrus changes the Schedule-Tag even on attendee PARTSTAT-only updates,
     # violating RFC6638 section 3.2 which requires the tag to remain stable.
     "scheduling.schedule-tag.stable-partstat": {"support": "unsupported"},
-    # Cyrus may not properly reject wrong passwords in some configurations
+    # Cyrus splits exception VEVENTs (with RECURRENCE-ID) into separate calendar
+    # object resources rather than keeping master+exception together.  Client-side
+    # expansion therefore cannot produce correct RECURRENCE-ID values.
+    "save-load.event.recurrences.exception": {"support": "unsupported"},
+    # Cyrus may not properly reject wrong passwords in some configurations.
     # Cyrus implements server-side automatic scheduling: for cross-user invites,
     # the server both auto-processes the invite into the attendee's calendar
     # AND delivers an iTIP notification copy to the attendee's schedule-inbox.

tests/test_caldav.py

@@ -3969,7 +3969,12 @@ def testRecurringDateWithExceptionSearch(self):
         ):
             assert len(rs) == 2
 
-        asserts_on_results = [r]
+        asserts_on_results = []
+        # Client-side expansion only produces correct RECURRENCE-IDs when the
+        # server keeps master VEVENT + exception VEVENT in the same calendar
+        # object resource.  If the server splits them, skip this assertion.
+        if self.is_supported("save-load.event.recurrences.exception"):
+            asserts_on_results.append(r)
         if self.is_supported("search.recurrences.expanded.exception"):
             asserts_on_results.append(rs)