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

add: unit test added #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

avinash-tiwari
Copy link

Description Added unit-test for all the components present using react-testing-library.
Changes Made changes in how the components were placed. Now, I am using a folder structure to place the components to contain the testing files within the component code itself.
The new structure is:-
ComponentName

  • ComponentName.js (component file)
  • index.js (import file)
  • __test__ (test folder)
    • ComponentName.test.js (test file)

Do let me know your insights, we can discuss things accordingly.
Thanks.

@avinash-tiwari
Copy link
Author

Will create a PR tomorrow, need to add some changes.
Thanks

@avinash-tiwari avinash-tiwari reopened this Oct 1, 2021
@avinash-tiwari
Copy link
Author

Please check, and we can discuss it accordingly if found anything.
Thanks.

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.

1 participant