Skip to content

test: wait for search-cache only as long as needed - #775

Merged
tobixen merged 1 commit into
masterfrom
test/search-cache-delay-only-filtered
Oct 14, 2026
Merged

tobixen merged 1 commit into
masterfrom
test/search-cache-delay-only-filtered

Conversation

@tobixen

@tobixen tobixen commented Oct 14, 2026

Copy link
Copy Markdown
Member

Commit message:

test: wait for search-cache only as long as needed

The integration tests slept the full search-cache delay before every
search, 180 s on Purelymail.  A search now waits only for what is left
of the delay since the last write.  A new filtered-only key on
search-cache, set for Purelymail, also lets plain listings skip it:
there only time-range and text searches lag (probed 2026-10-14), while
Bedework 3 lags in plain listings too.  Bedework 3's search tests pass
in half the time.

Prompt: The purelymail speedup fix, was it only implemented in the caldav-server-tester, or also in the caldav test code?  If the first, then please fix it also in the caldav test code
Followup-Prompt: (chose a per-server search-cache key exempting plain listings, set on purelymail only, after Bedework 3 failed with plain listings exempt everywhere)

Assisted-By: Claude Opus 5.5 <noreply@anthropic.com>
Reviewed-by: Tobias Brox <tobias@redpill-linpro.com>

The caldav-server-tester does the same around CalDAVSearcher.search in python-caldav/caldav-server-tester#25, reading the same filtered-only key.

🤖 Generated with Claude Code

The integration tests slept the full search-cache delay before every
search, 180 s on Purelymail.  A search now waits only for what is left
of the delay since the last write.  A new filtered-only key on
search-cache, set for Purelymail, also lets plain listings skip it:
there only time-range and text searches lag (probed 2026-10-14), while
Bedework 3 lags in plain listings too.  Bedework 3's search tests pass
in half the time.

Prompt: The purelymail speedup fix, was it only implemented in the caldav-server-tester, or also in the caldav test code?  If the first, then please fix it also in the caldav test code
Followup-Prompt: (chose a per-server search-cache key exempting plain listings, set on purelymail only, after Bedework 3 failed with plain listings exempt everywhere)

Assisted-By: Claude Opus 5.5 <noreply@anthropic.com>
Reviewed-by: Tobias Brox <tobias@redpill-linpro.com>
@tobixen
tobixen merged commit 723d85e into master Oct 14, 2026
17 checks passed
@tobixen
tobixen deleted the test/search-cache-delay-only-filtered branch October 14, 2026 17:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant