Skip to content

test(remote): qualify daemon async reuse - #853

Merged
jleni merged 1 commit into
mainfrom
test/daemon-remote-reuse-696
Aug 26, 2026
Merged

test(remote): qualify daemon async reuse#853
jleni merged 1 commit into
mainfrom
test/daemon-remote-reuse-696

Conversation

@jleni

@jleni jleni commented Aug 26, 2026

Copy link
Copy Markdown
Member

Summary

  • replace the explicit sync --push shortcut with a process-level daemon upload → independent client remote HIT test
  • use separate source trees, caches, configs, runtimes, and outputs; require matching keys, zero consumer compiler runs, and byte-identical artifacts
  • pin the fixture config against inherited KACHE_* overrides and make daemon cleanup wait on the portable lifetime lock
  • update the E2E coverage inventory now that the async production path is qualified

Falsification

I temporarily replaced the daemon worker's do_upload call with an injected error. The revised test failed at the remote publication boundary with no manifest or pack present; restoring the worker made it green. The fault was fully reverted before commit.

Validation

  • cargo fmt --all -- --check
  • git diff --check
  • cargo clippy --workspace --all-targets --all-features -- -D warnings
  • ./scripts/with-test-resources.sh cargo test --workspace --all-features
  • cargo test --test filesystem_remote_test -- --nocapture (7/7)
  • poisoned-env run with KACHE_REMOTE_READONLY=1 KACHE_LOCAL_ONLY=1 KACHE_DISABLED=1
  • independent static review; all findings resolved

Closes #696

@jleni
jleni marked this pull request as ready for review August 26, 2026 14:03
@jleni
jleni merged commit 2c28ffa into main Aug 26, 2026
19 checks passed
@jleni
jleni deleted the test/daemon-remote-reuse-696 branch August 26, 2026 14:04
@jleni jleni mentioned this pull request Aug 27, 2026
4 tasks
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.

test(e2e): qualify daemon remote reuse across isolated caches

1 participant