Skip to content
This repository has been archived by the owner on Jun 21, 2022. It is now read-only.

Re-enable running instrumentation tests in CI #56

Open
mustafa01ali opened this issue Nov 3, 2017 · 0 comments
Open

Re-enable running instrumentation tests in CI #56

mustafa01ali opened this issue Nov 3, 2017 · 0 comments
Labels
help wanted

Comments

@mustafa01ali
Copy link
Contributor

With the update to Gradle 3.0 and API 26, all builds in CI are failing with the following error -

$ echo no | android create avd --force -n test -t android-19 --abi armeabi-v7a

Error: Target id is not valid. Use 'android list targets' to get the target ids.

The command "echo no | android create avd --force -n test -t android-19 --abi armeabi-v7a" failed and exited with 1 during .

A similar issue was reported to Travis back in May 2017, but they haven't looked into it till now. After trying a lot of different ideas with no luck, we decided to disable running instrumented tests to publish an update.

Moving forward, we need to either find a way to make it work again or migrate this project to a better CI.
@mustafa01ali mustafa01ali mentioned this issue Nov 3, 2017
Merged
mustafa01ali added a commit that referenced this issue Nov 3, 2017
@mustafa01ali
Update project to use Android Gradle plugin v3.x (#55)
b384c0c 
* Updates project to use Android Gradle plugin v3.x and updated all 3rd party deps

* Disables running instrumentation tests in CI, see #56

* Fixes Proguard issue for release builds, updates gradle version

Also, the response builder in OkHttp 3.8 has made the message mandatory which Barricade was not passing, causing IllegalStateExceptions. I've fixed that issue by hardcoding a static message for now. In the future, we could make even the message configurable like we do for status codes.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
help wanted
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@mustafa01ali