-
Notifications
You must be signed in to change notification settings - Fork 307
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
Travis builds are failing again #720
Comments
- Add separate ANDOID_TAG ABI tag variable - Duplicate tools to get the new `repository-11.xml` and to install Android SDK tools 25.1.x - Change system image names to match new Android SDK - Change emulator start command to use new ABI tag variable to specify Google APIs See travis-ci/travis-ci#6122 (comment) for details. See also googlemaps/android-maps-utils#299.
Looks like ABIs for API Level 24 Google APIs are also missing - output of
|
The error was because the google recently renamed the ABIs as mentioned in the following issue OneBusAway/onebusaway-android#720 I have made the following changes 1. Upgraded AndroidTarget and AndroidToolBuild to 24. 2. No manual test being run hence emulator not required (removed emulator). Which has reduced the build time drastically. 3. Removed connectedCheck test because without an emulator there is no point of running the test. 4. Added assembleRelease to check if unsigned APK is being generated or not.
So the API Level 22 emulator should work because the ABIs exist, especially when using the However, with this build: ...I'm now seeing this error:
|
- Add separate ANDOID_TAG ABI tag variable - Duplicate tools to get the new `repository-11.xml` and to install Android SDK tools 25.1.x - Change system image names to match new Android SDK - Change emulator start command to use new ABI tag variable to specify Google APIs See travis-ci/travis-ci#6122 (comment) for details. See also googlemaps/android-maps-utils#299.
- Add separate ANDOID_TAG ABI tag variable - Duplicate tools to get the new `repository-11.xml` and to install Android SDK tools 25.1.x - Change system image names to match new Android SDK - Change emulator start command to use new ABI tag variable to specify Google APIs - Switch to 64-bit emulator image - Remove extra API level emulator image See travis-ci/travis-ci#6122 (comment) for details. See also googlemaps/android-maps-utils#299.
Given that Google still doesn't seem to have resolved the issues with API level 23-25 emulators that prevent them from running on Travis, I'm pushing a temporary workaround to the master branch in 03e262d that builds the projects on Travis, but does not run the unit tests. Builds don't require the emulator, only the unit tests, so this at least gives us the ability to see if builds themselves succeed. |
I opened this issue for failing emulator builds on android-map-utils, and I believe it's the best summary of the current problem with the emulators/SDK: |
@albodelu Thanks for the info! I'll see if I can test this on my projects soon... |
From travis-ci/travis-ci#6122 (comment):
|
-check wether there are default scripts ran by Travis Apparently images for emulators are not available for all API levels. Foolowing this OneBusAway/onebusaway-android#720 (comment) checking on my local install, it seems API level 25 should be good like that.
-check wether there are default scripts ran by Travis Down the Travis rabbit hole https://travis-ci.org/albodelu/Dexter/jobs/220847350/config from OneBusAway/onebusaway-android#720 (comment)
-check wether there are default scripts ran by Travis Down the Travis rabbit hole https://travis-ci.org/albodelu/Dexter/jobs/220847350/config from OneBusAway/onebusaway-android#720 (comment) typo
* Adding social functionality Running OBA style checker * Navigation Drawer expand/collapse chevron recolored and expanded clickable area to include entire row * Lowered min sdk version to 9 and moved check for social functionality to Utils * Updated build config name for Embedded Social api key and added field to agency x & y build flavors * Updated Navigation drawer fragment to get shared preferences from the application class * Remove extra margin from top and bottom of navigation drawer * Updated navigation drawer fragment to include only relevant social items * Removed unused navigation drawer items from Home Activity * Undo changes removing margins from navdrawer * Fixed bug where Action Bar did not appear in ArrivalsListActivity * Moved non-UI strings to DoNotTranslate * Updated comment bubble to be material design icon * Removing unncessary Embedded Social search configurations * Updating SDK version to fix google auth * Fixed a bug where 'University Area Transit Center' would briefly show when loading a social fragment * Use OBA theme color in SDK progress bars * Fixed bug where Pins navdrawer item would appear below the social overflow when the plan trip item was displayed * Updating Embedded Social server url to point to production instance * Updated Google authentication endpoint and removed unnecessary string resource * Fixed bug when opening discussion from ArrivalsListActivity * Update What's New for embedded social * Bump version to prepare for release * Jump several versionCodes over current master to allow for additional production releases which alpha is still deployed. * Fix unit test compilation issue * Updated embedded social library package name in proguard rules * Change to RealtimeService: Intent for BroadcastReceiver in Android 7.0 should only have primitive types in Bundle (ie no get/set Serializable). Fix duplicate calls to RealtimeService * Ensure correct itinerary is selected when app is backgrounded and resumed. * Fix OneBusAway#747 - Only show trip plan report button if we get a server response * Also change dialog message to "check Internet connection" * Fix OneBusAway#671 - Add trip plan start/end markers * Improve trip plan notification text * Make it clearer what is happening when notifiying user that a better itinerary was found, or that their trip had a significant schedule deviation change since when they first planned the trip * Updating ES version for google play store * Bump versionCode We burned versionCode 80 - as after we uploaded and submitted the APK Google rejected it with the following vulnerability message: ------ onReceivedSslError Detected in APK 80 Your app is using an unsafe implementation of WebViewClient.onReceivedSslError handler. Please see this Google Help Center article for details, including the deadline for fixing the vulnerability. Vulnerable classes: • com.microsoft.live.AuthorizationRequest$OAuthDialog$AuthorizationWebViewClient ------ And we can't upload another APK with versionCode 80, so we'll bump to 81. * Bump plugins for Android Studio 2.3 * Temporary workaround for OneBusAway#720 - don't run unit tests on Travis * Added configuartions which hide social search options and the topic gallery view * Update SDK version and add configurations required for push notifications * Removing unnecessary interface and simply overwriting the SDK's color attribute * Updating colors to overwrite SDK values and removing unnecessary objects previously used for coloring the UI * Added analytics to track interactions with the social UI elements * Updating social analytics names and values * Updating SDK version and reverting version information * Fix OneBusAway#770 - Get release password via dialog when Gradle runs as daemon * If gradle is not running as a daemon, get key/keystore password via command line input * Update README for current release process * Bump version number to prepare for release * Fix OneBusAway#697 - Allow users to opt out of "Location not enabled" dialog * Update to OneBusAway#697 - Allow users to opt out of "Location not enabled" dialog * Include Amazon build variant - somehow this didn't get committed with 98135b6 * Updating SDK version to exclude ms auth library * Bump version number to prepare for release * Updating config to exclude user profile tabs which showcase user created topics * Removing margins from navigation drawer list. OBA UI is unaffected but the navigation drawer in social activities loses the extra whitespace * Apply multidex removal to embedded-social branch * Enable Facebook auth * Update Embedded Social SDK version * Enabling Microsoft auth * Bump version for next release * Update What's New * Remove dexOptions from build.gradle * Fixing DB update to avoid failure cases and log attempts to add duplicate columns * Removing unnecessary imports from ObaProvider * Removing unnecessary import from HomeActivity * Update SDK version and proguard rules to avoid any DEX issues * Removing dependency on ES activities * Updating SDK version to use comment feed fragment * Updating config to remove user relations since users cannot post topics * Updating Nevigation Drawer fragment behavior when resuming a paused activity or navigating via the back button * Update behavior plan trip activity's software back button to handle ES activities * Move social options menu to the OBA settings page * Pull newest version of ES SDK * Remove navigation drawer from the ES Options activity * Allow HomeActivity to remain on the back stack when the Help nav drawer option is invoked from an ES activity * Correct nav drawer highlight when a user selects the help option while in an ES activity * Fixed bug where Embedded Social funtion is called without checking if social is enabled * Fix issue where OBA failed to build if EmbeddedSocialApiKey was missing * Added comments to functions handling the ES API key and fixed a method name in EmbeddedSocialUtils * Use PrimaryThemeDark to color the status bar in ES activities * Bump version to prepare for release * Updating SDK version to fix spacing in the action bar * Update SDK version for updated Google auth flow * Include layout for navdrawer separator rather than copy it * Update copyright on files authored by Microsoft * Add copyright to files touched by Microsoft * Updating contributors file * Update the social button to properly handle the fragment back stack and move ES specific logic to EmbeddedSocialUtils * Update copyright * Fix copyright header * Fix copyright header * Fix copyright header * Update AndroidManifest.xml * Remove extra whitespace * Fix copyright header * Fix copyright header * Fix copyright header * Fix copyright header * Fix copyright header * Fix copyright header * Fix copyright headers * Update strings for toasts when signing in or out and when viewing teh settings page. Hide social settings if ES is not enabled * Update toast text for creating user * Update SDK version * Fix bugs related to Navigation drawer * Factor the callback logic for social activities into a named class * Display the pin feed as a fragment rather than as an activity * Display the popular and recent activity feeds as fragments rather than activities * Replace the NavDrawer with an Up arrow in the SignIn activity * Reorder and refactor references to the Popular Feed fragment
* Based on #720 (comment), and solution at https://github.com/albodelu/android-maps-utils/blob/8ab3efe962cfe59da6db748c176108d7e142e4e7/.travis.yml. Change gradlew command to match previous Don't fail builds for Lint errors
@albodelu I'm just now coming back to setting up Android unit tests on Travis again - is the info in the above links still the most recent configuration you're aware of, or do you have a more recent version that still works as of today? |
Possible progress reported here on getting emulator tests running: |
It looks like Butterknife now runs emulator-based tests on Travis: Here's the Travis config - maybe this will work for us: |
This is working on GPSTest: |
A good analysis of speeding up emulator-based Travis builds: |
Now all builds (even not emulator based) are failing due to a license issue - from https://travis-ci.org/OneBusAway/onebusaway-android/builds/557364172:
|
Based on this Travis build comment, it looks like newer build tools broke Travis: Here's the Travis config that apparently now works: |
With the latest changes based on the above config the emulator does start now and tests run 🎉! However, the navigation tests are taking too long and the builds time out at 50 minutes. I'm going to set up just compiling the project in #997, and then work on adding the emulator-based testing again. Here's the statement to run the integration tests on the emulator:
Hopefully we can just disable the NavigationTest tests on CI and have the rest of the tests run. |
Summary:
Seems like we should have a perpetual issue for this - example of failed build:
https://travis-ci.org/OneBusAway/onebusaway-android/builds/178311476
This is related to Google renaming the ABIs again - see my past SO post:
http://stackoverflow.com/questions/28751111/invalid-abi-armeabi-v7a-for-the-selected-target-with-google-apis-21
I believe based on my answer http://stackoverflow.com/a/28751112/937715, we need to update things as specified in the "EDIT Sept 12th, 2016" section. This has worked for these two projects:
The text was updated successfully, but these errors were encountered: