ci: fix xandikos test server#35
Closed
tobixen wants to merge 1 commit into
Closed
Conversation
* Newer xandikos versions renamed XandikosBackend to SingleUserFilesystemBackend. Use a try/except fallback matching the approach already used in the caldav library's testing.py. * Lychee failures in CHANGELOG.md: update tobixen/plann issue links to pycalendar/plann (issues 14 and 16 return 404 after repo rename) * .lychee.toml: xandikos.org has been excluded (502 from CloudFlare, not under our control, though this is likely to be fixed soon) * tests/test_functional.py: catch NotImplementedError from journal search (xandikos does not support CalDAV REPORT for VJOURNAL) * tests/test_functional.py: reload todo1/todo2 before _adjust_relations reset to get fresh ETags after _check_for_panic modified those objects on the server prompt: The github checks failed on PR33, please investigate. Note that another branch is currently checked out, so either wait editing any files or create a separate worktree. [follow-up] The tests fail locally also. The code for setting up the xandikos test server does not work with the latest xandikos versions. Check ~/caldav on how things are done there. [follow-up] This fix should probably go in as a separate pull request, forked off the main branch. prompt: pull request 35 still fails Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
tobixen
force-pushed
the
fix/xandikos-backend-rename
branch
from
005cb99 to
b1fb235
Compare
tobixen
added a commit
that referenced
this pull request
May 21, 2026
The journal test (_select(journal=True)) was silently broken in CI: caldav>=1.5.0 is the first release with journal search support (May 2025). The bug was in caldav, not xandikos — xandikos supports VJOURNAL REPORT fine. Dropping Python 3.9 follows from the caldav>=1.5.0 requirement; caldav does not officially support 3.9 from that version onwards. Python 3.14 is added to the matrix en passant. Other changes: - Fix xandikos test for SingleUserFilesystemBackend rename (replaces the try/except workaround that was in the discarded PR #35) - Reload todo1/todo2 before _adjust_relations reset: _check_for_panic with fix_timeline modifies these objects on the server, staling ETags - Exclude xandikos.org from lychee (intermittent 502, not our fault) - Fix caldav deprecation warning in plann/lib.py Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Member
Author
|
Apparently those changes was already made in March in pull request #30. Hm! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Newer xandikos versions renamed XandikosBackend to
SingleUserFilesystemBackend. Use a try/except fallback matching
the approach already used in the caldav library's testing.py.
prompt: The github checks failed on PR33, please investigate. Note that
another branch is currently checked out, so either wait editing any files
or create a separate worktree. [follow-up] The tests fail locally also.
The code for setting up the xandikos test server does not work with the
latest xandikos versions. Check ~/caldav on how things are done there.
[follow-up] This fix should probably go in as a separate pull request,
forked off the main branch.
Co-Authored-By: Claude Sonnet 4.6 noreply@anthropic.com