react-otp-input icon indicating copy to clipboard operation
react-otp-input copied to clipboard

Setup unit tests

Open xitter opened this issue 6 years ago • 10 comments

  • Describe the Feature Request Add safeguard unit tests to the repository

First time complitation

  • Describe Preferred Solution I would prefer having https://github.com/testing-library/react-testing-library instead of enzyme here, see post. Thoughts are welcome in comments.

xitter avatar Oct 04 '19 07:10 xitter

react-testing-library is what I was thinking as well to begin with. 👍

apollonian avatar Oct 04 '19 08:10 apollonian

Awesome, I will work on this.

xitter avatar Oct 04 '19 10:10 xitter

@xitter how's your test going? I also would like to work on this issue @apollonian.

Buyantogtokh avatar Oct 06 '19 10:10 Buyantogtokh

I will be soon raising a PR for unit test setup, you are welcome to add more tests once it gets merged.

xitter avatar Oct 06 '19 14:10 xitter

This issue will help be getting started with unit testing, I want to work on it!

ankiiitraj avatar Oct 01 '20 08:10 ankiiitraj

@ankiiitraj Sure, we could use your help!

apollonian avatar Oct 01 '20 09:10 apollonian

working on it! stuck somewhere!

ankiiitraj avatar Oct 02 '20 12:10 ankiiitraj

@ankiiitraj Let me know how I could be helpful!

apollonian avatar Oct 02 '20 14:10 apollonian

I will draft a PR today! Thanks

ankiiitraj avatar Oct 03 '20 15:10 ankiiitraj

PR with jest and tests for all props and events: https://github.com/devfolioco/react-otp-input/pull/337

alex-parra avatar Mar 14 '22 14:03 alex-parra