Skip to content

test(sync): exercise refresh through adapter seam#109

Merged
samzong merged 1 commit into
mainfrom
test/sync-job-adapter-seam
Jul 15, 2026
Merged

test(sync): exercise refresh through adapter seam#109
samzong merged 1 commit into
mainfrom
test/sync-job-adapter-seam

Conversation

@samzong

@samzong samzong commented Jul 15, 2026

Copy link
Copy Markdown
Owner

What changed?

  • Inject the existing store, config, and adapter set into SyncJob so production and tests use the same execution path.
  • Cover changed-session refresh through the adapter-to-store path.
  • Remove three regression tests that did not execute sync behavior.

Why

  • Protect sync orchestration with one behavior-level test without adding a new abstraction or mock framework.

Verification

  • cargo fmt --all -- --check
  • cargo clippy --workspace --all-targets -- -D warnings
  • cargo test -p recall --lib (331 passed)
  • make check reaches the unchanged recall-reflect CLI tests; two fail on macOS because /var and /private/var are compared without path normalization.

Signed-off-by: samzong <samzong.lu@gmail.com>
@samzong
samzong merged commit f746350 into main Jul 15, 2026
1 check passed
@samzong
samzong deleted the test/sync-job-adapter-seam branch July 15, 2026 11:58
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