-
Notifications
You must be signed in to change notification settings - Fork 56
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
Testing Suite #56
Comments
@joshuapinter Are you still thinking about this? Would you be using Jest? I have no experience in testing, but I am interested in learning Jest and Enzyme. Maybe you can get something installed and configured the way you like and then I can help write tests? |
Absolutely. I've been making improvements to Android recently, including SelectContact, which is really slick. The more I think about testing, the more I think that Jest and Enzyme won't cut it. We use those in a project at work but mostly to render Components. In this library, we mostly need to test the access and response of the native Contacts on various device/OS combinations. The only way I can think of to test this is through emulators, via Travis or CircleCI or something, where we can specify a testing matrix and confirm all supported OS and versions are passing the tests. Maybe we use Jest or something else for the actual tests but we'll need to have them run in an emulator to confirm they're working in the device environment. i.e. it can't just be standalone testing. What do you think? |
I think the two obvious ones are CircleCI and Travis CI. Here is an example CircleCI project for React Native: https:/CircleCI-Public/circleci-demo-react-native Might be a good starting place. Plus, they offer free services for Open Source projects:
|
Very interesting. I'd like to learn this stuff. It's a bit over me at the moment but I'm willing to follow along and help out here and there where I can. I agree that testing in a real/emulated device is superior as most of our problems have stemmed from weirdness on device. |
Sounds good. Well, I'll get things setup and get the first couple tests done so that there is a pattern and let you finish it off from there. |
Started playing around with this on the tests branch: https:/joshuapinter/react-native-unified-contacts/tree/tests Much more difficult than anticipated. Will require a whole weekend to get this working right so have to put this off until more time opens up. It's important, though, so I'm keen to get it up and running eventually. |
Awesome. Yep, there is only so much time in a week! |
Currently we don't have any automated testing to help protect against regressions. We should add tests.
The text was updated successfully, but these errors were encountered: