Repository navigation
Conversation
caldav now ships its test-server registry in caldav.testing, so the conftest imports it instead of loading tests/test_servers by path from a caldav checkout, and CALDAV_SOURCE is gone. The registry also merges all its config sources, finds the checkout's yaml whatever the working directory, and probes external servers itself, so the conftest's workarounds for those go too. python-caldav/caldav#817 python-caldav/caldav#824 Prompt: Now that we have #817 done on the local caldav master (please confirm), please fix the calendaring library to use this, and run the integration tests towards all the docker servers plus all the external servers defined under [a caldav checkout holding the test servers' passwords] [#817: python-caldav/caldav#817] Assisted-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Reviewed-by: Tobias Brox <tobias@redpill-linpro.com>
caldav's Zimbra test server is configured with ssl_verify_cert: false, and urllib3's "Unverified HTTPS request" warning then failed every Zimbra test under filterwarnings = error. The filter matches on the message, so it needs no urllib3 import. Prompt: Now that we have #817 done on the local caldav master (please confirm), please fix the calendaring library to use this, and run the integration tests towards all the docker servers plus all the external servers defined under [a caldav checkout holding the test servers' passwords] [#817: python-caldav/caldav#817; this fixes what the Zimbra run turned up] Assisted-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Reviewed-by: Tobias Brox <tobias@redpill-linpro.com>
A run on every caldav test server took hours, most of them on the slowest server. Under pytest -n N --dist loadgroup each server's tests now stay on one worker, so the servers run in parallel and keep one connection and one session lock each; pytest-xdist joins the dev extra. --caldav-servers-config adds the servers of a test-server yaml outside the caldav checkout, its entries winning on a name clash. Prompt: The biggest problem is the purelymail runs [due to test sleeps due to purelymail having an async search cache], a single run can easily take several hoursl If we cannot do that with pytest-xdist, then it's needed to do aome similar lgoic as in [a personal script that runs caldav's tests in parallel, one tmux window per slow server] Followup-Prompt: Do the needful ensuring all tests are run towardes all servers - if ecloud failed on the last run, then it's a good reason to retry, [(sic)] Assisted-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Reviewed-by: Tobias Brox <tobias@redpill-linpro.com>
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
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.
Disclaimer: AI-generated code and text below the line, generated by Claude Opus 5.5, based on a wish to run functional tests for the calendaring library.
caldav now ships its test-server registry in
caldav.testing(python-caldav/caldav#824). The conftest imports it from there instead of loadingtests/test_serversby path from a caldav checkout, and the workarounds for caldav's old config handling are gone.A run on every caldav test server took many hours, because the servers were tested one after another. Under
pytest -n N --dist loadgroup, each server's tests now stay on one worker, so the servers run in parallel.--caldav-servers-config=PATHadds the servers of a test-server yaml kept outside the caldav checkout. A warning filter lets the test servers configured without TLS verification (caldav's Zimbra) run underfilterwarnings = error.Tests only: the default suite passes serially and under
-n 8 --dist loadgroup(2817 passed, 433 skipped, 4 xfailed). The first run on all servers turned up many divergences, which are left for roadmap 3.2.🤖 Generated with Claude Code