-
Notifications
You must be signed in to change notification settings - Fork 2
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
add tests to all of our React components, by using JEST #92
Comments
Thanks, Yes I need examples and how I can install jest on my system?
Pooja
…On Sat, Jul 28, 2018 at 3:53 AM, Arthur Tkachenko ***@***.***> wrote:
@poojashukla10 <https://github.com/poojashukla10>
At this folder we have all our major components for homepage:
https://github.com/GroceriStar/homepage/tree/master/src/components
- Add %component%.test.js file for each component and add simple tests
for empty rendering
I can provide an example if you wish.
- Create simple tests for child components inside parent component
In order to do it - you need to fork this repo, apply your changes and
open a pull request in order
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#92>, or mute the thread
<https://github.com/notifications/unsubscribe-auth/Ac5NuL3ScfsFxbWumhLH2eETHZT50sB7ks5uK1OxgaJpZM4VjE6x>
.
|
if you'll clone the project to your computer - and run the react project - then jest installed. this is a begginer guide to jest testing: https://www.sitepoint.com/test-react-components-jest/
tell me if you need help with setting up local instance of nodejs and forking our project |
this is a simple example of test, created with jest: https://github.com/GroceriStar/react-only-intern-23/blob/master/src/__tests__/Calender.test.js |
@poojashukla10
At this folder we have all our major components for homepage: https://github.com/GroceriStar/homepage/tree/master/src/components
I can provide an example if you wish.
In order to do it - you need to fork this repo, apply your changes and open a pull request in order
The text was updated successfully, but these errors were encountered: