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

Issue 822089 link

Starred by 6 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Mar 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

quawks-paladin: cheets_PlayStoreTest: retry_count: 2, FAIL: Unhandled Exception: Can not find INSTALL button

Project Member Reported by ayatane@chromium.org, Mar 15 2018

Issue description

https://luci-milo.appspot.com/buildbot/chromeos/quawks-paladin/2427

It looks like the test is trying to find the Install button for the Linpack benchmark app, but the app is already installed so there is no Install button.

WTF

Even if this is test flake, it sounds like the test is written in a very risky way.

 
Cc: elijahtaylor@chromium.org victorhsieh@chromium.org khmel@chromium.org
Labels: -Pri-2 OS-Chrome Pri-1
This test is failing on the R65 branch consistently, does this test have a server side dependency that broke? 

Or did we actually merge something bad to 65?

Adding some folks whom have touched this test in the past and may know how serious this is.

We need to clear this as a bug on 65 to push the beta today.

Comment 2 by norvez@chromium.org, Mar 15 2018

Labels: -Pri-1 Pri-0
It's blocking the CQ too: https://luci-milo.appspot.com/buildbot/chromeos/quawks-paladin/2432

I think the test should be disabled.
Let's disable it first to unblock others.  The "fix" should be easy by replacing the target app, which seems to be gone from Play.
I've run the test locally, and found the following quirk:

- it searches for a package by package name (rs.pedjaapps.Linpack)
- it installs the app whose info card matches the app name ('Linpack')
- later (either at the end of a run, or at the start of a future run) it tries to uninstall by package name (again, rs.pedjapps.Linkpack) so the search will [hopefully, see below] yield a package that is not installed.

The implicit assumption is that the search result yields a package with the same name. In my case, the package installed has package name 'org.skynetsoftware.linpack' so future runs will show "Uninstall" / "Open" UI in play store, so the test can't find the "Install" button (and fail)

I have no clue as to why this *recently* broke, <speculation>perhaps the package used to use the rs.pedjapps.Linpack package name.

(Disabling while this gets sorted out SGTM)

Cc: za...@chromium.org jhorwich@chromium.org
Owner: victorhsieh@chromium.org
Status: Started (was: Assigned)
Project Member

Comment 6 by bugdroid1@chromium.org, Mar 15 2018

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chromeos/autotest-cheets/+/c2665a1d18bbeae996e8de68a8b1084184574571

commit c2665a1d18bbeae996e8de68a8b1084184574571
Author: Victor Hsieh <victorhsieh@chromium.org>
Date: Thu Mar 15 17:44:35 2018

Labels: -Pri-0 Pri-1
Next step: fix the test and re-enable it.

Comment 8 by ihf@chromium.org, Mar 15 2018

Cc: ihf@chromium.org
When you remove a test from a blocking bvt suite please move it to bvt-perbuild instead. Otherwise you lose coverage and don't see when to move it back.
Sure, will do it next time.  Though in this case, we'll have to fix the test first, and I'll put them back to bvt-perbuild once it's ready.

Comment 10 by ihf@chromium.org, Mar 15 2018

Sure, it is just a little less work and smoother to send it right to bvt-perbuild.
Project Member

Comment 11 by bugdroid1@chromium.org, Mar 15 2018

Labels: merge-merged-release-R65-10323.B
The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chromeos/autotest-cheets/+/3c9ea292855b176d7b0246b3cab606501f54823c

commit 3c9ea292855b176d7b0246b3cab606501f54823c
Author: Victor Hsieh <victorhsieh@chromium.org>
Date: Thu Mar 15 19:33:31 2018

Project Member

Comment 12 by bugdroid1@chromium.org, Mar 15 2018

Labels: merge-merged-release-R66-10452.B
The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chromeos/autotest-cheets/+/c2e34336cf3c5207e311df48bc6338705e4be279

commit c2e34336cf3c5207e311df48bc6338705e4be279
Author: Victor Hsieh <victorhsieh@chromium.org>
Date: Thu Mar 15 19:33:34 2018

Cc: wzang@chromium.org glevin@chromium.org
 Issue 822151  has been merged into this issue.
NOTE: Test is (was?) also failing on reef-chrome-pfq, cyan-chrome-pfq, veyron_minnie-chrome-pfq, veyron_minnie-tot-chrome-pfq-informational.

Cc: akes...@chromium.org
Project Member

Comment 16 by bugdroid1@chromium.org, Mar 20 2018

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chromeos/autotest-cheets/+/33c7c46a8087473d65135fb2414c37b5af1d5056

commit 33c7c46a8087473d65135fb2414c37b5af1d5056
Author: Victor Hsieh <victorhsieh@chromium.org>
Date: Tue Mar 20 02:21:44 2018

Project Member

Comment 17 by bugdroid1@chromium.org, Mar 22 2018

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chromeos/autotest-cheets/+/00f0b25fd2c23f531f169e6e872c29bfeb2e9515

commit 00f0b25fd2c23f531f169e6e872c29bfeb2e9515
Author: Victor Hsieh <victorhsieh@chromium.org>
Date: Thu Mar 22 06:23:40 2018

Status: Fixed (was: Started)

Sign in to add a comment