Add Comprehensive Test Coverage for EducationalCurveTracker Component

[nishupr]

Summary

Closes #8324

EducationalCurveTracker.tsx was added in the latest upstream merge with only a basic .test.tsx and a single .accessibility.test.tsx. This PR adds the 4 missing test variants to bring it in line with the full test matrix every other dashboard component follows.

Test Files Added

EducationalCurveTracker.error-resilience.test.tsx - 7 tests

• Renders null when API returns success: false
• Renders null when API returns success: false with no error message
• Renders null when a network error is thrown
• Renders null when fetch throws a non-Error exception (e.g. string)
• Does not crash when timeline is an empty array
• Renders null when data is null despite success: true
• Survives being wrapped in an error boundary without triggering it on valid data

EducationalCurveTracker.empty-fallback.test.tsx - 8 tests

• Shows animate-pulse skeleton while data is being fetched
• Skeleton contains exactly 2 placeholder blocks
• Skeleton disappears after data loads
• Shows "No educational commits in the recent window." when timeline is empty
• Does not render any bar columns when timeline is empty
• Renders null when data is null after a successful response
• Renders null when success is false
• Renders "Last 30 Days" label in the chart card header

EducationalCurveTracker.mouse-interactivity.test.tsx - 10 tests

• Renders exactly one bar column per timeline entry
• Bar columns have hover:bg-indigo-400, transition-colors, and group classes
• Tooltip renders commit count for each bar
• Tooltip renders date for each bar
• Tooltip container has group-hover:block class (hidden by default)
• Card 1 has hover:-translate-y-1 and transition-transform classes
• Card 2 has hover:-translate-y-1 and transition-transform classes

EducationalCurveTracker.responsive-breakpoints.test.tsx - 9 tests

• Outer grid has grid-cols-1 and md:grid-cols-3 classes
• Outer grid has gap-6 class
• Card 1 (Current Focus) uses col-span-1
• Card 2 (Syllabus Momentum) uses col-span-2
• Wrapper has max-w-7xl and responsive padding (px-4, sm:px-6, lg:px-8)
• Loading skeleton uses same grid-cols-1 md:grid-cols-3 layout
• Skeleton card 1 uses col-span-1
• Skeleton card 2 uses col-span-2

Checklist

• All 34 tests pass locally (includes existing 3 tests in .test.tsx and 1 in .accessibility.test.tsx)
• global.fetch mocked via vi.fn() consistent with existing EducationalCurveTracker.test.tsx
• No changes to production code
• Branch created off main

[vercel]

Someone is attempting to deploy a commit to the jhasourav07's projects Team on Vercel.

A member of the Team first needs to authorize it.

[retenta-bot]

This pull request significantly improves the test coverage for the EducationalCurveTracker component, ensuring it handles various scenarios effectively. This aligns with our previous decision to add comprehensive test variants for this component to prevent regressions. Thank you for your contribution, and I look forward to your updates!

[github-actions]

📦 Next.js Bundle Size Report (Gzipped Sizes)

✨ No significant bundle size changes detected.

📊 Summary of Totals

Category	PR Size	Base Size	Difference
Total JS	4052.50 KB	4052.50 KB	0 B
Total CSS	338.94 KB	338.94 KB	0 B

[Aamod007]

Comprehensive test coverage for EducationalCurveTracker — empty fallback, error resilience, mouse interactivity, and responsive breakpoints. Well structured across 4 test files. CI failing, please fix.

Labels: level:intermediate (4 files, 570 lines), quality:clean, type:testing, mentor:Aamod007

[Aamod007]

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.