test(api route): verify 400 error response body is valid JSON

[meetparmar392005]

Description

Fixes #745

Updated the 400 test in app/api/streak/route.test.ts to properly verify
the response body is valid JSON with the correct structure.

Changes Made:

• Replaced response.text() with response.json()
• Added assertion: body.error === 'Invalid parameters'
• Added assertion: typeof body.details === 'object'

Pillar

🧪 Testing — Improving test quality and coverage

Checklist

• I have read CONTRIBUTING.md
• npm run lint passes locally
• npm run test passes locally (477 tests ✅)
• No SVG output changes (test-only PR)

[github-actions]

👋 Hey @meetparmar392005, welcome to CommitPulse! 🎉

Thanks for opening your first pull request — this is a big deal and we appreciate the effort!

While you wait for a review, please double-check:

• ✅ You've read the CONTRIBUTING.md checklist
• ✅ npm run lint, npm run format, and npm run test all pass locally
• ✅ Your PR has a visual preview if it touches any SVG output
• 💬 You've joined our Discord for faster PR feedback

A maintainer will review your PR shortly. Hang tight! 🚀

[github-actions]

👋 Hey @meetparmar392005, it looks like you didn't use our PR template!

The section ## Description is missing from your PR description.

Please update your PR description to include all required sections so we can review this properly:

• ## Description — What does this PR do? Which issue does it fix?
• ## Pillar — Which contribution pillar does this fall under?
• ## Checklist — Have you ticked off the quality checklist?

You can find the full template in CONTRIBUTING.md. Just edit your PR description and the needs-details label will be removed automatically. 🙌

[Copilot]

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Updates the GET /api/streak test to assert a structured JSON error response (instead of plain text) for invalid requests.

Changes:

• Switches error-body parsing from response.text() to response.json()
• Verifies error and details fields on the JSON error payload

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[github-actions]

🚨 Hey @meetparmar392005, the CI Pipeline is failing on this PR and it has been marked as status:blocked.

Please fix the issues before this can be reviewed. Here's how:

1. Run checks locally before pushing:

npm run format:check   # Check Prettier formatting
npm run lint           # Run ESLint
npm run typecheck      # TypeScript type check
npm run test           # Run unit tests (Vitest)
npm run build          # Verify production build passes

2. Auto-fix common issues:

npm run format         # Auto-fix formatting with Prettier
npm run lint -- --fix  # Auto-fix lint errors where possible

3. Check the full failure log here:
👉 View CI Run

Once you push a fix and the CI passes, the status:blocked label will be removed automatically. 💪

[vercel]

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

A member of the Team first needs to authorize it.

Make PR according to guidelines

[github-actions]

🎉 Congratulations @meetparmar392005! Your PR has been successfully merged. 🚀

Thank you for contributing to CommitPulse. Your work helps us build a better tool for the community.

⚠️ Important for GSSoC Contributors:
You are strictly advised to join our Discord Server as it is mandatory for all GSSoC participants. All important announcements, point claims, and community discussions happen there.

Keep building! 💻✨