New issue
Advanced search Search tips

Issue 638927 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 638946
Owner:
Closed: Aug 2016
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

chrome_public_test_apk failing on chromium.android/Jelly Bean Tester

Project Member Reported by torne@chromium.org, Aug 18 2016

Issue description

chrome_public_test_apk failing on chromium.android/Jelly Bean Tester

Type: build-failure

Builders failed on: 
- Jelly Bean Tester: 
  https://build.chromium.org/p/chromium.android/builders/Jelly%20Bean%20Tester

Reasons: 
https://build.chromium.org/p/chromium.android/builders/Jelly%20Bean%20Tester/builds/5538/steps/chrome_public_test_apk
Tests:
* org.chromium.chrome.browser.preferences.website.ManageSpaceActivityTest#testClearUnimporantWithoutChromeStart
* org.chromium.chrome.browser.preferences.website.ManageSpaceActivityTest#testClearUnimportantOnly
* org.chromium.chrome.browser.preferences.website.ManageSpaceActivityTest#testLaunchActivity
* org.chromium.webapk.shell_apk.DexLoaderTest#testLoadFromRemoteDataDir


ManageSpaceActivity is only supposed to run on KK and above but the test was missing the required annotation; I'm adding it in https://codereview.chromium.org/2256213002/ so that's easy.

The DexLoaderTest failure is less obvious to me - assigning this bug to a likely owner to investigate.
 
Mergedinto: 638946
Status: Duplicate (was: Available)
Project Member

Comment 2 by bugdroid1@chromium.org, Aug 18 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e4f29ff78ccfdb1243d07e4c88b4812833d06770

commit e4f29ff78ccfdb1243d07e4c88b4812833d06770
Author: torne <torne@chromium.org>
Date: Thu Aug 18 15:29:43 2016

Set min SDK for ManageSpaceActivityTest.

ManageSpaceActivity is only supported on K and above, but didn't have
the correct annotation to tell the test runner to skip it on JB devices.
We hadn't noticed this until recently as the JB test bot was incorrectly
provisioned with K devices :(

BUG= 638927 

Review-Url: https://codereview.chromium.org/2256213002
Cr-Commit-Position: refs/heads/master@{#412833}

[modify] https://crrev.com/e4f29ff78ccfdb1243d07e4c88b4812833d06770/chrome/android/javatests/src/org/chromium/chrome/browser/preferences/website/ManageSpaceActivityTest.java

Sign in to add a comment