test(api/github): Enhance mock-integrations coverage for async service layer and cache

[Robo-Synaptic-Vaish]

Fixes #2591

Description

Adds test coverage for asynchronous service layer mocking and local cache stubs, as requested in #2591.

The original ticket named components/dashboard/RadarChart.tsx as the target file, but that component is a pure presentational component — it receives all its data via props and has no cache, service call, or async logic to test. After tracing where RadarChart actually gets its data from, the real async service layer + local cache behavior (checking cache before fetching, timing out on a hung request, syncing the cache after a successful refresh) lives in app/api/github/route.ts. This PR adds the requested test coverage there instead, since that's where the behavior actually exists.

Tests added (app/api/github/route.mock-integrations.test.tsx):

• Cache is checked before a fresh service call is made (bypassCache: false by default)
• An explicit refresh (?refresh=true) bypasses the cache
• A hung request times out via AbortController and falls back with a 504
• A repeated refresh within the cooldown window falls back to serving cache instead of bypassing again
• A successful stale-cache read triggers exactly one background cache sync (backgroundRefresh.triggerRefresh)

Pillar

• 🎨 Pillar 1 — New Theme Design
• 📐 Pillar 2 — Geometric SVG Improvement
• 🕐 Pillar 3 — Timezone Logic Optimization
• 🛠️ Other (Bug fix, refactoring, docs)

Visual Preview

N/A — this PR adds test coverage only, no UI/visual changes.

Checklist before requesting a review:

• I have read the CONTRIBUTING.md file.
• I have tested these changes locally (localhost:3000/api/streak?user=YOUR_USERNAME).
• I have run npm run format and npm run lint locally and resolved all errors (CI will fail otherwise).
• My commits follow the Conventional Commits format (e.g., feat(themes): ..., fix(calculate): ...).
• I have updated README.md if I added a new theme or URL parameter.
• I have started the repo.
• I have made sure that i have only one commit to merge in this PR.
• The SVG output matches the CommitPulse "premium quality" aesthetic standard (no raw elements, smooth animations, correct fonts).
• (Recommended) I joined the CommitPulse Discord community for contributor discussions, mentorship, and faster PR support.

[retenta-bot]

This pull request enhances the testing framework by adding mock-integrations coverage for the async service layer and local cache, which is crucial for ensuring the reliability of these components. This approach aligns with previous decisions to improve test coverage for similar functionalities, such as the ControlsPanel and other routes. 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]

Solid mock integration tests for the github API route. The async service layer stubbing with getFullDashboardData mock is well done, and the timeout/hang simulation using fake timers demonstrates good edge case coverage. The beforeEach cleanup of quotaMonitor, refreshPolicy, refreshRateLimiter, and backgroundRefresh is thorough.

CI is pending — once green, this is ready to merge.

Labels: level:beginner (1 file, focused test coverage), 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.