-
Notifications
You must be signed in to change notification settings - Fork 40
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
Android, build and test documentation #38
Comments
Travis needs to build Android and run Android tests, see https://docs.travis-ci.com/user/languages/android |
@mattheworiordan Whether it is Android or pure Java, JUnit tests will run on local JVM. We might run these tests using AndroidJUnitRunner, but they require a connected (virtual or physical) device. For that, Cloud Test Lab would be a better option |
There is no longer a separate Android build of the library. The
I'm not sure how well that Travis CI support will work for us because it takes over the building of the tests based on things you specify in the .yml instead of using the gradle configuration we provide. But I've not played with it so I don't know how flexible it is.
@gokhanbarisaker if you have experience with that and recommend it, it would be worth trying out. |
Sure, but it is very handy to have one test suite that passes or fails, and I expect if we used Cloud Test Lab we'd need to have a separate job run. If I am wrong, and you can integrate into the Travis CI tests, then please do go ahead, although perhaps we should try Travis first before ruling it out as if it works that should be easiest |
I am guessing this is no longer an issue when #372 is merged? |
I agree, I think we can close this |
@paddybyers please can you update the README ASAP for users to:
The text was updated successfully, but these errors were encountered: