Skip to content

[codex] Fix backup concurrency CI test#170

Merged
chuanman2707 merged 1 commit into
mainfrom
codex/fix-backup-runner-ci
May 19, 2026
Merged

[codex] Fix backup concurrency CI test#170
chuanman2707 merged 1 commit into
mainfrom
codex/fix-backup-runner-ci

Conversation

@chuanman2707
Copy link
Copy Markdown
Owner

Summary

  • keep the backup concurrency/collision test timestamp inside the active retention window
  • compute the expected backup filename stem from the runtime timestamp instead of a stale fixed date

Why

CI failed on main after #169 because the test used 2026-04-18; on 2026-05-19 that manual backup is older than the 30-day retention window, so pruning correctly removed the base file and kept only the collision-suffix safety-floor file.

Validation

  • cargo test --manifest-path mhm/src-tauri/Cargo.toml backup::runner::tests::run_backup_once_serializes_concurrent_backups -- --nocapture
  • cargo test --manifest-path mhm/src-tauri/Cargo.toml
  • cargo check (mhm/src-tauri)
  • cargo clippy --all-targets -- -D warnings (mhm/src-tauri)
  • rustfmt --edition 2024 --check mhm/src-tauri/src/backup/runner.rs

@chuanman2707 chuanman2707 merged commit e8f431b into main May 19, 2026
4 checks passed
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