1058362 - Perma-failures on b-i, ime_uninstallation_test.js, uninstallation_test.js

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia, defect)

Description

[Kevin Grandon :kgrandon]

Suspecting this is caused by bug 1042797. https://tbpl.mozilla.org/php/getParsedLog.php?id=46702680&tree=B2g-Inbound

Comment 1

[Kevin Grandon :kgrandon]

Ted - can you try running these tests locally to verify that we can still uninstall keyboards?

The failing tests are: apps/keyboard/test/marionette/uninstallation_test.js
apps/settings/test/marionette/tests/ime_uninstallation_test.js

Comment 2

[Kevin Grandon :kgrandon]

And it appears the whole world started failing like crazy on the latest b2g-inbound: https://tbpl.mozilla.org/php/getParsedLog.php?id=46741568&tree=B2g-Inbound

Comment 3

[Rudy Lu [:rudyl] (inactive after 2015/7/3)]

Attachment: Patch V1

It seems some test fixing patches were not landed correctly, see bug 1000315 comment 65.

This is a patch to fix those tests, and could run locally.
Let's see how it runs on Gaia-Try.

Comment 4

[Kevin Grandon :kgrandon]

Comment on attachment 8478793 [details] [review]
Patch V1

Argh, R+ assuming we get a green try run. Thanks so much for tracking this down.

Comment 5

[Kevin Grandon :kgrandon]

Just an update that Ted thinks this patch won't fix it due to some changes last week. If we don't get a green try run from it, and Ted can't fix it, then we will backout the regressing patches.

Comment 6

[Kevin Grandon :kgrandon]

If we backout, we would backout all of these patches: https://tbpl.mozilla.org/?tree=B2g-Inbound&showall=1&rev=4b37cddaed77

As well as gaia changes here: https://hg.mozilla.org/integration/b2g-inbound/rev/a840609deb46

Comment 7

[Ted Clancy [:tedders1]]

Attachment: Bug 1058362 - Fix failing gaia integration tests. r=timdream

Comment 8

[Tim Guan-tin Chien [:timdream] (please needinfo)]

(In reply to Kevin Grandon :kgrandon from comment #5)
> Just an update that Ted thinks this patch won't fix it due to some changes
> last week. If we don't get a green try run from it, and Ted can't fix it,
> then we will backout the regressing patches.

It turned out Rudy's patch turned green (not really green, just failed on some other stuff and not the one to be fixed.) but after discussion with Rudy he agreed we should land Ted's fix as apparently it's formatted in a better way.

So, this is the Try run we are now waiting on:

https://tbpl.mozilla.org/?rev=27ba765e270d58138fbb5f2134944566a24b0925&tree=Gaia-Try

Comment 9

[Tim Guan-tin Chien [:timdream] (please needinfo)]

master: https://github.com/mozilla-b2g/gaia/commit/c710c7fac2987bc0f3ea76a2361fe16e995468a0