Add Travis matrix for multiple emulators #645
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Getting the Voice recognition running is only possible, if at least one handler handles the RecognitionService.SERVICE_INTERFACE intent. The easiest way to achive this is to run the instrumented test on an emulator which actually has the Google Play services installed. This commit changes the travis-ci configuration so that the emulator started has the Play services installed. It also adds some extras, which are already pre-installed, but accoridng to the travis documentation[1] should be explicitly mentioned to ensure build stability. However, for the current build they don't change anything. The extras are namely: extra-android-support extra-google-google_play_services extra-google-m2repository extra-android-m2repository For some reason, also the -no-audio switch does not work anymore, however, currently there's no need to completely disable audio in our build and test. [1] https://docs.travis-ci.com/user/languages/android/#Pre-installed-components Signed-off-by: Florian <[email protected]>
Signed-off-by: Florian <[email protected]>
android-27 can not be run on travis, as there's no arm abi anymore, and x86 requires KVM support, which is not possible in travis containers atm, see travis-ci/travis-ci#1419 Use latest android image provided by travis (which has an arm abi), android-25, and also does not have a non-google-apis image anymore. Signed-off-by: Florian <[email protected]>
Signed-off-by: Florian <[email protected]>
Signed-off-by: mueller-ma <[email protected]>
Codecov Report
@@ Coverage Diff @@
## master #645 +/- ##
==========================================
+ Coverage 31.36% 31.53% +0.16%
==========================================
Files 64 64
Lines 4903 4903
Branches 658 658
==========================================
+ Hits 1538 1546 +8
+ Misses 3144 3136 -8
Partials 221 221
Continue to review full report at Codecov.
|
|
@mueller-ma feel free to add it to my existing PR. However, your changes still skip the voice test, so this problem is not resolved with that. |
|
I wonder why the tests pass for api 16. I run kitkat on an avd and that crashed instantly due to a multidex issue (#619) |
|
Ok, the test fails with api 19, but not with api 16. I have checked my fix #619 on an avd, so I'm going to merge that one. @FlorianSW I'm going to add api 19 to your PR. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
@FlorianSW I just want to see if api 14 works without messing up your PR.
If it works I can either merge this one and you make a new one, or you include this change in #611