test: add responsive breakpoint tests for stats route #8080
Conversation
|
🚨 Hey @AMAN194701, the CI Pipeline is failing on this PR and it has been marked as 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 passes2. Auto-fix common issues: npm run format # Auto-fix formatting with Prettier
npm run lint -- --fix # Auto-fix lint errors where possible3. Check the full failure log here: Once you push a fix and the CI passes, the |
Aamod007
left a comment
There was a problem hiding this comment.
Good job implementing the responsive breakpoint tests for the stats API route. However, the Vercel check is failing. Please resolve the issues and rerun the CI pipeline.
Aamod007
left a comment
There was a problem hiding this comment.
Upon re-evaluation, I realized I previously misinterpreted the Vercel deployment failure as an issue with this PR. The Vercel build failure seems unrelated to the changes introduced here, so I will ignore it moving forward. The rest of the PR looks good, so I'm approving it!
Description
Closes #6784
Added a dedicated responsive breakpoint test suite for the Stats route.
Changes
app/api/stats/route.responsive-breakpoints.test.tsPillar
Visual Preview
N/A (Test-only changes)
Checklist before requesting a review:
CONTRIBUTING.mdfile..feat(themes): ...,fix(calculate): ...).