zulip-android-legacy icon indicating copy to clipboard operation
zulip-android-legacy copied to clipboard

Setup zulip server and install android sdk to run tests on travis.

Open Sam1301 opened this issue 8 years ago • 10 comments

This pr addresses #271 Since dist trusty is not compatible with android on travis we need to install android sdk and other support libraries manually in order to make use of trusty to run zulip dev environment.

The existing tests are flaked (verified by me and kunal) so they have been commented out for now.

Sam1301 avatar Jan 12 '17 20:01 Sam1301

Automated message from Dropbox CLA bot

@Sam1301, it looks like you've already signed the Dropbox CLA. Thanks!

smarx avatar Jan 12 '17 20:01 smarx

@saketkumar95 @kunall17 @vishwesh3 can you give this a review?

timabbott avatar Jan 31 '17 23:01 timabbott

@timabbott I can do it too if you agree.

yadav-rahul avatar Feb 01 '17 08:02 yadav-rahul

@yadav-rahul Yeah sure do it. :+1:

saketkumar avatar Feb 01 '17 09:02 saketkumar

@saketkumar95 Thanks for your concern. Waiting for @timabbott reiview. :)

yadav-rahul avatar Feb 01 '17 12:02 yadav-rahul

@yadav-rahul you don't need my permission to do a code review; just review it and post your feedback :)

timabbott avatar Feb 01 '17 18:02 timabbott

@timabbott OK :)

yadav-rahul avatar Feb 01 '17 18:02 yadav-rahul

@timabbott This won't work now because the espresso test's are flaky and they work sometimes and sometimes don't! https://github.com/zulip/zulip-android/issues/342

So we need to have some test's for the travis to run!

kunall17 avatar Feb 01 '17 18:02 kunall17

Well we can do the infrastructure setup for Travis CI to be able to run a single do-nothing test with the app built, and get that debugged, while in parallel folks work on #342. So I think this is still useful to work on.

timabbott avatar Feb 01 '17 19:02 timabbott

@Sam1301, your pull request has developed a merge conflict! Please review the most recent commit (f791888) for conflicting changes and resolve your pull request's merge conflicts.

zulipbot avatar May 04 '17 18:05 zulipbot