test: Add Timezone Boundary Tests for AdvancedSettingsPanel - #8311
test: Add Timezone Boundary Tests for AdvancedSettingsPanel#8311deepsikha-dash wants to merge 2 commits into
Conversation
|
@deepsikha-dash is attempting to deploy a commit to the jhasourav07's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
This PR introduces important tests for the timezone logic, which is crucial for ensuring accurate functionality in the AdvancedSettingsPanel. It aligns with previous decisions made to enhance code quality, such as the recent fix for dead code in the calculateStreak function. Ensuring that our timezone logic is robust will help maintain the integrity of user contributions. Thank you for your contributions, and I look forward to your updates! |
📦 Next.js Bundle Size Report (Gzipped Sizes)✨ No significant bundle size changes detected. 📊 Summary of Totals
|
Aamod007
left a comment
There was a problem hiding this comment.
Timezone boundary tests for AdvancedSettingsPanel — 102 lines of focused test coverage. CI failing, please fix.
Labels: level:beginner (1 file), quality:clean, type:testing, mentor:Aamod007
Aamod007
left a comment
There was a problem hiding this comment.
Re-reviewing after noting the only CI failure is Vercel authorization (not code-related). All other checks (Format · Lint · Typecheck · Test, Production Build, CodeQL) pass. Approving — the Vercel deploy auth is a repo-level config issue, not a code problem.
Description
Fixes #6865
Pillar
Visual Preview
N/A
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).feat(themes): ...,fix(calculate): ...).README.mdif I added a new theme or URL parameter.