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

Jest testing setup #34

Merged
merged 14 commits into from
Oct 4, 2024
Merged

Conversation

prathameshkoshti
Copy link
Contributor

Changes made in this PR:

  • Setup of jest along with react testing library
  • Configuration of jest with next.js

@prathameshkoshti
Copy link
Contributor Author

@lucifercr07, as per our earlier discussion from #29, I have setup the jest for next.js, please review the same.

@lucifercr07
Copy link
Contributor

@prathameshkoshti please resolve conflicts.

@prathameshkoshti
Copy link
Contributor Author

@lucifercr07 conflicts resolved.

@lucifercr07 lucifercr07 changed the title Chore/jest setup Jest testing setup Oct 4, 2024
Copy link
Contributor

@lucifercr07 lucifercr07 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@prathameshkoshti thanks for contributing, looks good to me.
Please add this to Github actions workflow, and also update the README with how to run the test.

@prathameshkoshti
Copy link
Contributor Author

@lucifercr07, added github workflow and updated readme with necessary instructions. Please check.

@lucifercr07 lucifercr07 merged commit a284410 into DiceDB:master Oct 4, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants