[Test] Add Step Completion Boundary Test Case to Weekly Review Test Suite

• Labels: good first issue, testing, difficulty:beginner, area:core
• Target File: tests/weekly-review.test.ts
• Estimated Time: 10–15 minutes
• Difficulty: Level 2 (Beginner)

Why This Matters

Weekly review rituals govern accountability progress. Adding explicit boundary test coverage ensures ritual step transitions work reliably.

Current Behavior

tests/weekly-review.test.ts tests review step navigation, but does not explicitly test the transition from step 5 to completion state.

Requested Change

Add a unit test in tests/weekly-review.test.ts asserting that completing the final step marks the review ritual state as completed.

Acceptance Criteria

• Final step completion test case added to tests/weekly-review.test.ts.
• Verification command passes cleanly: npm run test:file -- tests/weekly-review.test.ts.

Verification

npm run test:file -- tests/weekly-review.test.ts

Contributor Notes

Single-file test addition.

🚀 How to Claim and Implement an Issue

1. Choose an Issue: Browse our open issues and pick one matching your interest and available time.
2. Comment on GitHub: Leave a comment on the corresponding GitHub issue: "I would like to work on this issue. Please assign it to me."
3. Follow the Guide: Review our Beginner's Contribution Guide for step-by-step Git instructions.
4. Verify Locally: Run the specific verification command listed on the issue before opening your PR.
5. Submit PR: Open a pull request against main referencing this issue number!

Open tests/weekly-review.test.ts and read the existing review step navigation tests, then run npm run test:file -- tests/weekly-review.test.ts. Add the boundary case for completing the final step and verify that the ritual state is completed; the file-specific test command should pass cleanly.