Feature/setup vitest testing #39
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
@shruthideepikareddy is attempting to deploy a commit to the Akash Kumar's projects Team on Vercel. A member of the Team first needs to authorize it. |
Member
|
The test setup looks good. While you've included a basic test for the homepage, do you plan to create test cases for all pages? |
Contributor
Author
|
Thanks for the feedback! I'll get right on it and i will add unit tests for the remaining pages like About, Contact, Download, Leaderboard, Privacy, and Terms , i will cover everything we have till now then I'll push the updates. |
Contributor
Author
|
I have finished the updates. |
akash-kumar-dev
approved these changes
Jan 8, 2026
Member
|
Looks good, thanks for your contributions! Please join our Discord server: https://discord.gg/Qtgskdk5 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description:
This PR establishes a robust unit testing foundation for the BrowsePing web application using Vitest and React Testing Library. It includes the necessary configuration for Next.js 15 compatibility, path alias support, and a sample test suite. Additionally, it addresses several pre-existing linting issues and synchronizes the branch with the latest upstream changes.
Type of Change:
Related Issue
Closes #32
Testing Done
Unit Tests: Executed npm test. Result: PASS.
Linting: Executed npm run lint. Result: 0 errors.
Build: Executed npm run build. Result: Successful production build.
Verified in local Windows environment.
Screenshots (if applicable)
N/A (Testing infrastructure and lint fixes)
Checklist
I have read and followed all guidelines in CONTRIBUTING.md
My commits follow Conventional Commits format
I have tested my changes thoroughly
CRITICAL: Only my intended changes are included
I have updated relevant documentation
I have now implemented comprehensive unit tests for the About, Contact, Download, Leaderboard, Privacy, and Terms of Service pages. The localized tests, linting, and build are all passing successfully.