test: add streak calculation tests for Monday-to-Friday#1956
Conversation
|
@abhilashgedela28-lang is attempting to deploy a commit to the jhasourav07's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
If you review this as soon as possible it will be helpful me to solve the others also. I was getting errors when performing different issues and submitting PRs of same repo. Once this is closed I will again put PR on the other two. |
Aamod007
left a comment
There was a problem hiding this comment.
Adds a Monday-Friday streak test with 4 evaluation dates: Friday (active 5-day streak), Saturday (grace keeps streak alive), Sunday (streak broken), Wednesday (mid-week streak). 55/55 tests pass.
|
So if you merge this I can able to do the others also . |
|
🎉 Congratulations @abhilashgedela28-lang! Your PR has been successfully merged. 🚀 Thank you for contributing to CommitPulse. Your work helps us build a better tool for the community.
Keep building! 💻✨ |
Description
Fixes streak computation test verification for custom weekday contribution patterns, ensuring no off-by-one errors when managing calendar offsets and date boundaries.
fixes #1505
Pillar
Visual Preview
N/A (This is a backend unit test change and does not affect the visual presentation of the SVG).
Checklist before requesting a review:
CONTRIBUTING.mdfile.localhost:3000/api/streak?user=YOUR_USERNAME).npm run formatandnpm run lintlocally and resolved all errors (CI will fail otherwise).test: add unit test for Monday-Friday exclusive commit streaks).README.mdif I added a new theme or URL parameter.