test: add comprehensive unit tests for streak calculation utilities by Shanidhya01 · Pull Request #2746 · Priyanshu-byte-coder/devtrack

Shanidhya01 · 2026-06-23T14:15:34Z

Summary

Adds comprehensive unit tests for streak calculation utility functions to improve reliability and prevent regressions in DevTrack's streak tracking system.

Closes #1069

Changes Made

Added unit tests for current streak calculation
Added unit tests for longest streak calculation
Added unit tests for streak-at-risk detection
Added tests for streak reset behavior
Added tests for empty commit history
Added tests for date boundary edge cases
Added tests for leap year handling
Added tests for month transitions (28/29/30/31 days)
Added tests for timezone-related scenarios
Added tests for commits made after midnight

Test Coverage

Covered scenarios:

Returns 0 for empty commit history
Counts consecutive days correctly
Resets streak on missed day
Handles streak freeze functionality (if implemented)
Calculates longest streak accurately
Detects streak at risk when no commit exists today
Handles timezone boundaries correctly
Counts today correctly for post-midnight commits
Handles leap year dates (Feb 29)
Handles month length variations

Type of Change

Test addition
Bug fix
New feature
Breaking change

Checklist

Tests added
Existing tests pass
No production behavior changed
Code follows project conventions

github-actions · 2026-06-23T14:15:53Z

GSSoC Label Checklist 🏷️

@Umbrella-io — please apply the appropriate labels before merging:

Difficulty (pick one):

level:beginner — 20 pts
level:intermediate — 35 pts
level:advanced — 55 pts
level:critical — 80 pts

Quality (optional):

quality:clean — ×1.2 multiplier
quality:exceptional — ×1.5 multiplier

Validation (required to score):

gssoc:approved — counts for points
gssoc:invalid / gssoc:spam / gssoc:ai-slop — does not score

Type labels (type:*) are auto-detected from files and title. Review and adjust if needed.
Points formula: (difficulty × quality_multiplier) + type_bonus

Legit-Ox

Test coverage is solid — good edge case variety across leap years, month transitions, timezone boundaries, and freeze dates. Three issues before merging:

rolldown added to production dependencies — unrelated to this PR and shouldn't be there at all
One test name directly contradicts its assertion
package-lock.json accidentally narrows the Node engine range

Shanidhya01 · 2026-06-24T06:32:35Z

Ok will fix it by the end of the day

…est/streak-calculation-1069

Swap hardcoded stats table for dynamic shields.io badges (stars, forks), add komarev view counter, and remove good-first-issues badge.

- Remove duplicate ThemeToggle from hero section (navbar-only now) - Fix hardcoded Priyanshu-byte-coder#222 footer color → use MUTED CSS variable for dark mode - Add missing lnd-footer-link CSS class - Deduplicate lnd-cta-secondary CSS rule - Remove unused ThemeToggle import

- Remove duplicate widget section that rendered everything twice - Move orphaned sidebar into proper 2-column grid (1fr + 320px) - Standardize spacing with space-y-8 wrapper - Fix duplicate id="goals" → id="goals-insights" - Fix StreakTracker break-all → tabular-nums (numbers no longer break mid-digit) - Fix PRMetrics lg:grid-cols-6 → lg:grid-cols-3 (readable stat cards) - Make CodingActivityInsightsCard chart height responsive - Remove 7 dead imports and 2 unused dynamic components

…e range, fix test name - Remove rolldown from dependencies (build tool, not a runtime lib) - Restore engines.node to >=20.0.0 to avoid excluding Node 22/24 - Rename misleading test that said does not treat Feb 28 → Mar 1 as consecutive when the assertion proves they are consecutive in a non-leap year

All 11 widgets that were rendered directly in page.tsx are already handled by CustomizableDashboard with better skeletons, error boundaries, and ssr:false. Removed duplicates: - RepoAnalyticsExplorer, PRMetrics, CommunityMetrics - PRBreakdownChart, CommitTimeChart, ActivityRingChart - CodingActivityInsightsCard, PRReviewTrendChart - StreakTracker, LocalCodingTime, CodingTimeWidget Kept RoastHypeWidget (not in CustomizableDashboard). Deleted DashboardWidgets.tsx (dead code, zero importers). page.tsx: 245 → 152 lines.

…r#2709) Bumps [actions/setup-node](https://github.com/actions/setup-node) from 4 to 6. - [Release notes](https://github.com/actions/setup-node/releases) - [Commits](actions/setup-node@v4...v6) --- updated-dependencies: - dependency-name: actions/setup-node dependency-version: '6' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

…der#2711) Bumps [github/codeql-action](https://github.com/github/codeql-action) from 3 to 4. - [Release notes](https://github.com/github/codeql-action/releases) - [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md) - [Commits](github/codeql-action@v3...v4) --- updated-dependencies: - dependency-name: github/codeql-action dependency-version: '4' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

…2710) Bumps [actions/checkout](https://github.com/actions/checkout) from 4 to 7. - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](actions/checkout@v4...v7) --- updated-dependencies: - dependency-name: actions/checkout dependency-version: '7' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

…der users (Priyanshu-byte-coder#2783) Adds a visually-hidden skip navigation link as the first focusable element in the root layout, targeting #main-content so keyboard-only users can bypass the navbar. Wraps page children with the skip target div and promotes the dashboard's top-level container to a semantic <main> element.

…hu-byte-coder#2782)

) Co-authored-by: tmdeveloper007 <tmdeveloper007@users.noreply.github.com>

…r#2777) Co-authored-by: tmdeveloper007 <tmdeveloper007@users.noreply.github.com>

Co-authored-by: tmdeveloper007 <tmdeveloper007@users.noreply.github.com>

…anshu-byte-coder#2725) Co-authored-by: tmdeveloper007 <tmdeveloper007@users.noreply.github.com>

…hu-byte-coder#2722) (Priyanshu-byte-coder#2733) Co-authored-by: DESIREDDY MOHITH REDDY <desireddymohithreddy@MRD-2.local>

…anshu-byte-coder#2603) (Priyanshu-byte-coder#2741) WIDGET_SPAN_CLASSES uses xl:col-span-2 on several widgets but the parent section grids had no xl:grid-cols-N, so those col-span rules had no grid context at viewports >=1280px. The rendered layout therefore diverged from the skeleton (which has no span classes) at ultra-wide viewports, causing the visible layout shift on hydration. - Add xl:grid-cols-3 to activity and goals sections (which contain xl:col-span-2 widgets) so column spans are meaningful at xl breakpoints - analytics already had xl:grid-cols-3 (correct) - Add min-h-[280px] to ChartSkeleton and min-h-[200px] to SkeletonCard so skeleton height is stable on auto-rows-max grids before hydration - Fix sectionWidgets indentation inside .map() callback (formatting only)

…Others (Priyanshu-byte-coder#2605) (Priyanshu-byte-coder#2740) - Languages with < 1% share are collapsed into a single 'Others' bucket - Named languages are capped at 6 maximum; any beyond rank 6 also collapse into 'Others' regardless of their share - Donut chart centre label now shows the true total language count from the API, not the capped display count, so users can see they have more - Rounding remainder is absorbed into 'Others' to keep the donut complete - Prevents the legend overflowing its card container on 10+ language profiles

…rt (Priyanshu-byte-coder#2473) (Priyanshu-byte-coder#2768)

…fill (Priyanshu-byte-coder#2728)

…u-byte-coder#2610) (Priyanshu-byte-coder#2743)

Priyanshu-byte-coder · 2026-06-24T20:24:12Z

The test coverage here is genuinely strong — the timezone edge cases, leap year handling, and streak-utils coverage are valuable. A few things to fix before merge:

Remove the package.json change — changing engines from 20.x to >=20.0.0 is unrelated to tests and belongs in a separate PR.
package-lock.json changes — the added resolved/integrity fields for rolldown are also unrelated. Please drop these.
File conflict with test : added unit tests for streak.ts #2776 — both PRs create test/streak.test.ts. Since test : added unit tests for streak.ts #2776 has already been merged, please either rename your file (e.g. test/streak-comprehensive.test.ts) or consolidate the tests into the existing file to avoid conflicts.

Fix these and the test quality is solid enough to merge.

- sanitize.ts: run stripHtml iteratively until stable, preventing nested/split-tag bypass (CodeQL js/incomplete-multi-character-sanitization) - project-tutor/route.ts: encodeURIComponent on owner/repo before embedding in GitHub API URLs (CodeQL js/request-forgery Priyanshu-byte-coder#6-8) - MembersPanel.tsx: encodeURIComponent on github_username in img src (CodeQL js/xss-through-dom Priyanshu-byte-coder#17) - README.md: replace all stale Umbrella-io/devtrack refs with Priyanshu-byte-coder/devtrack

Move security overrides into pnpm.overrides (the overrides field was npm-only and ignored by pnpm). Forces: - serialize-javascript >=7.0.3 (HIGH RCE via RegExp, GHSA-5c6j-r48x-rmvq) - postcss >=8.4.31 (XSS via unescaped </style>, GHSA-qx2v-qp2m-jg93) - uuid >=9.0.1 (buffer bounds, GHSA-w5hq-g745-h8pq) - js-yaml >=4.1.0 (quadratic DoS, GHSA-h67p-54hq-rp68)

…errides pnpm 11 moved overrides from package.json to pnpm-workspace.yaml. Lockfile regenerated — vulnerable old versions eliminated: - serialize-javascript 4.0.0/6.0.2 -> 7.0.6 (HIGH RCE GHSA-5c6j-r48x-rmvq) - js-yaml 3.14.2 removed (quadratic DoS GHSA-h67p-54hq-rp68) - postcss/uuid already at patched versions Also approve needed native build deps in allowBuilds.

…shu-byte-coder#2790)

…iyanshu-byte-coder#2706) * fix: replace npm with pnpm in README and ignore package-lock.json * ci: update github workflows to use pnpm * ci: fix missing pnpm/action-setup in github workflows * chore: approve native build scripts for pnpm * ci: remove global pnpm install to use setup action version --------- Co-authored-by: DESIREDDY MOHITH REDDY <desireddymohithreddy@MRD-2.local>

…coder#2703) Co-authored-by: tmdeveloper007 <tmdeveloper007@users.noreply.github.com>

…coder#2702) Co-authored-by: tmdeveloper007 <tmdeveloper007@users.noreply.github.com>

…yanshu-byte-coder#2700) Co-authored-by: tmdeveloper007 <tmdeveloper007@users.noreply.github.com>

…error

pnpm 9 does not read overrides from pnpm-workspace.yaml, causing ERR_PNPM_LOCKFILE_CONFIG_MISMATCH on frozen install. pnpm 11 is the version in use locally and is the correct version for this lockfile.

…lite)

…op EOL Node 18

- Accept package-lock.json deletion (project uses pnpm) - Revert engines.node to "20.x" (no scope change) - Resolve test/streak.test.ts conflict by keeping upstream version (Priyanshu-byte-coder#2776) - Add test/streak-comprehensive.test.ts with our timezone/leap-year/freeze tests

Our comprehensive tests live in test/streak-comprehensive.test.ts.

test: add unit tests for streak calculation utilities

37a1d15

Shanidhya01 requested a review from Priyanshu-byte-coder as a code owner June 23, 2026 14:15

github-actions Bot added gssoc26 GSSoC 2026 contribution type:feature GSSoC type bonus: new feature type:testing GSSoC type bonus: tests (+10 pts) and removed gssoc26 GSSoC 2026 contribution type:feature GSSoC type bonus: new feature labels Jun 23, 2026

github-actions Bot assigned Shanidhya01 Jun 23, 2026

Legit-Ox reviewed Jun 23, 2026

View reviewed changes

Comment thread package.json Outdated

Comment thread test/streak.test.ts Outdated

Comment thread package-lock.json Outdated

Shanidhya01 and others added 5 commits June 24, 2026 15:15

Merge branch 'main' of https://github.com/Shanidhya01/devtrack into t…

fbed90e

…est/streak-calculation-1069

docs: replace static project stats with live badges and add view counter

d8ae809

Swap hardcoded stats table for dynamic shields.io badges (stars, forks), add komarev view counter, and remove good-first-issues badge.

github-actions Bot added gssoc26 GSSoC 2026 contribution type:feature GSSoC type bonus: new feature labels Jun 24, 2026

Shanidhya01 and others added 3 commits June 24, 2026 17:08

Merge branch 'main' into test/streak-calculation-1069

3df84bf

Merge branch 'main' into test/streak-calculation-1069

dd123d7

Shanidhya01 requested a review from Legit-Ox June 24, 2026 15:14

dependabot Bot and others added 8 commits June 24, 2026 23:29

fix: respect prefers-reduced-motion accessibility preference (Priyans…

0c605d1

…hu-byte-coder#2782)

test: add unit tests for utils.ts cn function (Priyanshu-byte-coder#2778

9b27bf4

) Co-authored-by: tmdeveloper007 <tmdeveloper007@users.noreply.github.com>

test: add unit tests for personality-analysis.ts (Priyanshu-byte-code…

4b258d5

…r#2777) Co-authored-by: tmdeveloper007 <tmdeveloper007@users.noreply.github.com>

test: add unit tests for streak.ts (Priyanshu-byte-coder#2776)

84f5ed5

Co-authored-by: tmdeveloper007 <tmdeveloper007@users.noreply.github.com>

tmdeveloper007 and others added 8 commits June 24, 2026 23:32

test: add unit tests for wrapped year-summary utility functions (Priy…

796e371

…anshu-byte-coder#2725) Co-authored-by: tmdeveloper007 <tmdeveloper007@users.noreply.github.com>

test: add unit tests for memory rate-limit utility functions (Priyans…

9efd68a

…hu-byte-coder#2722) (Priyanshu-byte-coder#2733) Co-authored-by: DESIREDDY MOHITH REDDY <desireddymohithreddy@MRD-2.local>

fix: correct database column names for streak milestones in data-expo…

50200f5

…rt (Priyanshu-byte-coder#2473) (Priyanshu-byte-coder#2768)

fix(local-coding): implement plaintext api key fallback and hash back…

603fb09

…fill (Priyanshu-byte-coder#2728)

docs: add local development setup and troubleshooting guide (Priyansh…

6cc85d5

…u-byte-coder#2610) (Priyanshu-byte-coder#2743)

Merge branch 'main' into test/streak-calculation-1069

c9106b7

Priyanshu-byte-coder and others added 14 commits June 25, 2026 14:19

docs: add missing API Documentation link to Table of Contents (Priyan…

94ddddf

…shu-byte-coder#2790)

test: add unit tests for csv serialization utilities (Priyanshu-byte-…

7b8508e

…coder#2703) Co-authored-by: tmdeveloper007 <tmdeveloper007@users.noreply.github.com>

test: add unit tests for github-rate-limit utilities (Priyanshu-byte-…

ff10e3e

…coder#2702) Co-authored-by: tmdeveloper007 <tmdeveloper007@users.noreply.github.com>

test: add unit tests for achievement-estimators helper functions (Pri…

a3bd9d7

…yanshu-byte-coder#2700) Co-authored-by: tmdeveloper007 <tmdeveloper007@users.noreply.github.com>

fix(ci): add packages field to pnpm-workspace.yaml to fix store path …

15bd4af

…error

fix(ci): upgrade pnpm from v9 to v11 to match local dev environment

d2de87c

pnpm 9 does not read overrides from pnpm-workspace.yaml, causing ERR_PNPM_LOCKFILE_CONFIG_MISMATCH on frozen install. pnpm 11 is the version in use locally and is the correct version for this lockfile.

fix(ci): use Node.js 22 to satisfy pnpm 11 requirement (needs node:sq…

f8150c1

…lite)

fix(ci): upgrade automated-tests workflow to pnpm 11 + Node.js 22, dr…

1311ebc

…op EOL Node 18

chore: resolve streak.test.ts conflict — keep upstream basic version

7df0840

Our comprehensive tests live in test/streak-comprehensive.test.ts.

Shanidhya01 force-pushed the test/streak-calculation-1069 branch from 30a9a28 to 7df0840 Compare June 25, 2026 14:25

github-actions Bot added type:design GSSoC type bonus: UI/design (+10 pts) type:devops GSSoC type bonus: devops (+15 pts) type:docs GSSoC type bonus: documentation (+5 pts) type:bug GSSoC type bonus: bug fix type:performance GSSoC type bonus: performance (+15 pts) labels Jun 25, 2026

Merge branch 'main' into test/streak-calculation-1069

4325cd1

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

test: add comprehensive unit tests for streak calculation utilities#2746

test: add comprehensive unit tests for streak calculation utilities#2746
Shanidhya01 wants to merge 43 commits into
Priyanshu-byte-coder:mainfrom
Shanidhya01:test/streak-calculation-1069

Shanidhya01 commented Jun 23, 2026

Uh oh!

github-actions Bot commented Jun 23, 2026

Uh oh!

Legit-Ox left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Shanidhya01 commented Jun 24, 2026

Uh oh!

Priyanshu-byte-coder commented Jun 24, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

10 participants

Conversation

Shanidhya01 commented Jun 23, 2026

Summary

Changes Made

Test Coverage

Type of Change

Checklist

Uh oh!

github-actions Bot commented Jun 23, 2026

GSSoC Label Checklist 🏷️

Uh oh!

Legit-Ox left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Shanidhya01 commented Jun 24, 2026

Uh oh!

Priyanshu-byte-coder commented Jun 24, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

10 participants