Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: add vitest as test tool and increase test coverage #34

Closed
wants to merge 4 commits into from

Conversation

varshith257
Copy link

Closes #12
/claim #12
@quest-bot loot #12

Test coverage increased to 85%. The coverage is majorly left out in the Login.tsx and Policy.tsx. There are errors in the flows of this files and needed fixes which I count as the out of scope of this bounty.

@varshith257
Copy link
Author

@quest-bot loot #12

@quest-bot quest-bot bot added the ⚔️ Quest Tracks quest-bot quests label Nov 20, 2024
Copy link

quest-bot bot commented Nov 20, 2024

Quest PR submitted! image Quest PR submitted!

@varshith257, you are attempting to solve the issue and loot this Quest. Will you be successful?


Questions? Check out the docs.

@varshith257
Copy link
Author

@gemanor My Bad! I have replied to our conversations through the mail so I haven't seen the message to remove renaming and I have only seen the ai/cursor changes message :)

Anyway, here's the clean PR

@gemanor
Copy link
Collaborator

gemanor commented Nov 24, 2024

Hey, @varshith257, I must close this PR because it violates our rules and uses blindly AI-generated code. As proof, even the package.json file isn't valid and the npm i command isn't working.
I'm kindly asking you to avoid any further work on this repository.

@gemanor gemanor closed this Nov 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🙋 Bounty claim ⚔️ Quest Tracks quest-bot quests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Proper Unit Test Coverage
2 participants