New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 679949 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner: ----
Closed: Jan 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

"org.chromium.chrome.browser.banners.AppBannerManagerTest#testBannerAppearsThenDoesNotAppearAgainForCustomTime" is flaky

Project Member Reported by chromium...@appspot.gserviceaccount.com, Jan 11 2017

Issue description

"org.chromium.chrome.browser.banners.AppBannerManagerTest#testBannerAppearsThenDoesNotAppearAgainForCustomTime" is flaky.

This issue was created automatically by the chromium-try-flakes app. Please find the right owner to fix the respective test/step and assign this issue to them. If the step/test is infrastructure-related, please add Infra-Troopers label and change issue status to Untriaged. When done, please remove the issue from Sheriff Bug Queue by removing the Sheriff-Chromium label.

We have detected 3 recent flakes. List of all flakes can be found at https://chromium-try-flakes.appspot.com/all_flake_occurrences?key=ahVzfmNocm9taXVtLXRyeS1mbGFrZXNyeAsSBUZsYWtlIm1vcmcuY2hyb21pdW0uY2hyb21lLmJyb3dzZXIuYmFubmVycy5BcHBCYW5uZXJNYW5hZ2VyVGVzdCN0ZXN0QmFubmVyQXBwZWFyc1RoZW5Eb2VzTm90QXBwZWFyQWdhaW5Gb3JDdXN0b21UaW1lDA.

Flaky tests should be disabled within 30 minutes unless culprit CL is found and reverted. Please see more details here: https://sites.google.com/a/chromium.org/dev/developers/tree-sheriffs/sheriffing-bug-queues#triaging-auto-filed-flakiness-bugs
 

Comment 1 by foolip@chromium.org, Jan 11 2017

Owner: dominickn@chromium.org
Status: Assigned (was: Untriaged)
Test was added in https://chromium.googlesource.com/chromium/src/+/598df313ae463915342386163972df774453c570 by dominickn@.

Comment 2 by foolip@chromium.org, Jan 11 2017

The latest failure looks like this:

org.chromium.chrome.browser.banners.AppBannerManagerTest#testBannerAppearsThenDoesNotAppearAgainForCustomTime (run #1):
junit.framework.AssertionFailedError: Criteria not met in allotted time.
	at org.chromium.content.browser.test.util.CriteriaHelper.pollInstrumentationThread(CriteriaHelper.java:79)
	at org.chromium.content.browser.test.util.CriteriaHelper.pollUiThread(CriteriaHelper.java:117)
	at org.chromium.content.browser.test.util.CriteriaHelper.pollUiThread(CriteriaHelper.java:138)
	at org.chromium.chrome.browser.banners.AppBannerManagerTest.waitUntilAppBannerInfoBarAppears(AppBannerManagerTest.java:243)
	at org.chromium.chrome.browser.banners.AppBannerManagerTest.testBannerAppearsThenDoesNotAppearAgainForCustomTime(AppBannerManagerTest.java:437)
	at java.lang.reflect.Method.invokeNative(Native Method)
	at android.test.InstrumentationTestCase.runMethod(InstrumentationTestCase.java:214)
	at android.test.InstrumentationTestCase.runTest(InstrumentationTestCase.java:199)
	at android.test.ActivityInstrumentationTestCase2.runTest(ActivityInstrumentationTestCase2.java:192)
	at org.chromium.chrome.test.ChromeActivityTestCaseBase.runTest(ChromeActivityTestCaseBase.java:752)
	at org.chromium.base.test.BaseTestResult.runParameterized(BaseTestResult.java:161)
	at org.chromium.base.test.BaseTestResult.run(BaseTestResult.java:124)
	at android.test.AndroidTestRunner.runTest(AndroidTestRunner.java:191)
	at android.test.AndroidTestRunner.runTest(AndroidTestRunner.java:176)
	at android.test.InstrumentationTestRunner.onStart(InstrumentationTestRunner.java:554)
	at android.app.Instrumentation$InstrumentationThread.run(Instrumentation.java:1701)
Project Member

Comment 3 by bugdroid1@chromium.org, Jan 11 2017

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

commit 09a3924243cd62fd49cc091b6948f1455963f851
Author: Philip Jägenstedt <foolip@chromium.org>
Date: Wed Jan 11 16:04:39 2017

Disable testBannerAppearsThenDoesNotAppearAgainForCustomTime (flaky)

BUG= 679949 
TBR=dominickn@chromium.org

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

[modify] https://crrev.com/09a3924243cd62fd49cc091b6948f1455963f851/chrome/android/javatests/src/org/chromium/chrome/browser/banners/AppBannerManagerTest.java

Comment 4 by foolip@chromium.org, Jan 11 2017

Labels: -Sheriff-Chromium
Cc: dominickn@chromium.org
Owner: ian...@chromium.org
ianwen@: could https://chromium.googlesource.com/chromium/src/+/7a609035f046b007eb1b0d8302776aeae3b2e174 have caused these flakes?

Otherwise, this test has been green for a long time in the flakiness dashboard.
For context, this test is failing because an infobar isn't being detected as showing.

Comment 7 by ian...@chromium.org, Jan 26 2017

Owner: ----
Status: Available (was: Assigned)
Cc: ian...@chromium.org
ianwen, before you escape: any thoughts about #5?
ianwen's culprit CL was reverted: https://codereview.chromium.org/2665663003/. I'm going to re-enable this test.
Project Member

Comment 10 by bugdroid1@chromium.org, Jan 30 2017

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

commit c336037815019df68c431a22870ce4c122a8ddca
Author: dominickn <dominickn@chromium.org>
Date: Mon Jan 30 19:14:53 2017

Revert of Disable testBannerAppearsThenDoesNotAppearAgainForCustomTime (flaky) (patchset #1 id:1 of https://codereview.chromium.org/2627973002/ )

Reason for revert:
Suspect CL causing failures has been reverted (https://codereview.chromium.org/2665663003/)

Original issue's description:
> Disable testBannerAppearsThenDoesNotAppearAgainForCustomTime (flaky)
>
> BUG= 679949 
> TBR=dominickn@chromium.org
>
> Review-Url: https://codereview.chromium.org/2627973002 .
> Cr-Commit-Position: refs/heads/master@{#442913}
> Committed: https://chromium.googlesource.com/chromium/src/+/09a3924243cd62fd49cc091b6948f1455963f851

TBR=foolip@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG= 679949 

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

[modify] https://crrev.com/c336037815019df68c431a22870ce4c122a8ddca/chrome/android/javatests/src/org/chromium/chrome/browser/banners/AppBannerManagerTest.java

Status: Fixed (was: Available)

Sign in to add a comment