test: align bootstrap check with shared backup discovery - #115
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe test assertion for the repository clone failure fallback now checks for ChangesBackup Manifest Validation
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~2 minutes Change: Other Merge Risk: ⚪ Minimal · up to The bootstrap fallback test now checks the current helper reference, with no concrete merge-blocking risk identified. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
The Pester workflow failed because the bootstrap fallback check still expected
backup-manifest.jsoninbootstrap.ps1after discovery moved to the shared module. Update the assertion to check theFind-BackupManifestcall.Validation: reproduced the original failure, then passed all 288 tests with
Invoke-Pester -Path tests -CI.git diff --checkpassed.Summary by CodeRabbit