test(mountsync): cover write-only to mirror mode-flip backfill - #396
test(mountsync): cover write-only to mirror mode-flip backfill#396khaliqgant wants to merge 1 commit into
Conversation
Adds the missing regression for a write-only mount later flipped to mirror mode: performs a real write-only cycle, recreates the syncer in mirror mode against the same state dir, and proves pre-existing remote history is tree-listed/read/materialized instead of being skipped by the restart fast-path. The underlying runtime fix (persist SyncMode and reset BootstrapComplete/bootstrap checkpoints in loadState on write-only to mirror transition) already exists on main — verified this test passes against a clean origin/main checkout with no production code changes. Closes #262. Verified independently: - go test ./internal/mountsync/... -run TestSyncOnceWriteOnlyToMirrorBackfillsExistingRemoteFiles -v -count=1 - go test ./internal/mountsync/... -count=1
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Relayfile Eval ReviewRun: Passed: 4 | Needs human: 0 | Reviewable: 0 | Missing output: 0 | Failed: 0 | Skipped: 0 Human Review CasesNo reviewable human-review cases captured Relayfile output. |
1 participant
Closes #262.
Adds the missing regression for a write-only mount later flipped to mirror mode: performs a real write-only cycle, recreates the syncer in mirror mode against the same state dir, and proves pre-existing remote history is tree-listed/read/materialized instead of being skipped by the restart fast-path.
Test-only PR, no production code change. The underlying runtime fix (persist
SyncModeand resetBootstrapComplete/bootstrap checkpoints inloadStateon write-only→mirror transition) already exists onmain— independently verified this test passes cleanly against a freshorigin/maincheckout with no other changes, before opening this PR.Test plan
go test ./internal/mountsync/... -run TestSyncOnceWriteOnlyToMirrorBackfillsExistingRemoteFiles -v -count=1go test ./internal/mountsync/... -count=1