1033449 - Gaia unit tests are perma-failing on b2g30_v1_4 since bug 874510 was merged into production

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::TestAgent, defect)

Description

[Ryan VanderMeulen [:RyanVM]]

b2g30 is closed until this is resolved.

https://tbpl.mozilla.org/php/getParsedLog.php?id=42950417&tree=Mozilla-B2g30-v1.4

b2g_ubuntu64_vm mozilla-b2g30_v1_4 opt test gaia-unit on 2014-07-02 09:04:13 PDT for push d597b94be592
slave: tst-linux64-spot-809

09:13:31     INFO -  gaia-unit-tests TEST-START | keyboard/test/unit/jspinyin_test.js |
09:13:31     INFO -  gaia-unit-tests TEST-START | keyboard/test/unit/jspinyin_test.js | jspinyin
09:13:31     INFO -  gaia-unit-tests TEST-UNEXPECTED-FAIL | keyboard/test/unit/jspinyin_test.js | jspinyin load | expected #{this} to equal #{exp}
09:13:31     INFO -  gaia-unit-tests INFO       | stack trace:
09:13:31     INFO -      Error: expected #{this} to equal #{exp}
09:13:31     INFO -          at chaiAssert (http://keyboard.gaiamobile.org:8080/common/test/helper.js:33:1)
09:13:31     INFO -          at assertEqual (http://keyboard.gaiamobile.org:8080/common/vendor/chai/chai.js:648:1)
09:13:31     INFO -          at (anonymous) (http://keyboard.gaiamobile.org:8080/common/vendor/chai/chai.js:2724:1)
09:13:31     INFO -          at isDefined (http://keyboard.gaiamobile.org:8080/common/vendor/chai/chai.js:1843:9)
09:13:31     INFO -          at (anonymous) (http://keyboard.gaiamobile.org:8080/test/unit/jspinyin_test.js:156:5)
09:13:31     INFO -          at wrapper (http://keyboard.gaiamobile.org:8080/common/test/mocha_generators.js:62:13)
09:13:31     INFO -          at run (http://keyboard.gaiamobile.org:8080/common/vendor/mocha/mocha.js:3709:7)
09:13:31     INFO -          at runTest (http://keyboard.gaiamobile.org:8080/common/vendor/mocha/mocha.js:4087:5)
09:13:31     INFO -          at (anonymous) (http://keyboard.gaiamobile.org:8080/common/vendor/mocha/mocha.js:4133:7)
09:13:31     INFO -          at next (http://keyboard.gaiamobile.org:8080/common/vendor/mocha/mocha.js:4013:7)
09:13:31     INFO -          at (anonymous) (http://keyboard.gaiamobile.org:8080/common/vendor/mocha/mocha.js:4022:7)
09:13:31     INFO -          at next (http://keyboard.gaiamobile.org:8080/common/vendor/mocha/mocha.js:3970:16)
09:13:31     INFO -          at (anonymous) (http://keyboard.gaiamobile.org:8080/common/vendor/mocha/mocha.js:3990:5)
09:13:31     INFO -          at (anonymous) (http://keyboard.gaiamobile.org:8080/common/vendor/mocha/mocha.js:4938:28)
09:13:31     INFO -  gaia-unit-tests TEST-UNEXPECTED-FAIL | keyboard/test/unit/jspinyin_test.js | jspinyin init | jspinyin is undefined

etc...

Comment 1

[Jonathan Griffin (:jgriffin)]

Looking for help from the Gaia team; this problem has caused 1.4 to be closed.

Comment 2

[Jonathan Griffin (:jgriffin)]

FYI, I can revert the mozharness change from bug 874510, if that's what's causing the problem, but it isn't clear to me that it is, other than the timing of the change.

Comment 3

[Ryan VanderMeulen [:RyanVM]]

I already tried reverting the revision it started on and that didn't improve things (and I didn't expect it to based on what it changed). If you want more convincing that it's from the mozharness merge to production, I can kick off a run on a previously-green push as well.

Comment 4

[John Ford [:jhford] CET/CEST Berlin Time]

So, I am controlling the version of Mozharness that's being used on Gaia-Try and it's running against http://hg.mozilla.org/users/jford_mozilla.com/mozharness/default

Does that branch have the mozharness merge to production that you're concerned by?  If it doesn't, we could kick off tests on the latest v1.4 commit using the latest v1.4 gecko builds to see if an older mozharness passes tests.

Comment 5

[Ryan VanderMeulen [:RyanVM]]

Looks like it does.

Comment 6

[John Ford [:jhford] CET/CEST Berlin Time]

As long as I can continue to run my special commits, we could alter it to not have that merge

Comment 7

[John Ford [:jhford] CET/CEST Berlin Time]

Oh, which merge do you suspect?

Comment 8

[John Ford [:jhford] CET/CEST Berlin Time]

I've submitted

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

This is testing reverting 35d2981a6365, which is what Ryan is suspecting is causing the breakage

Comment 9

[Kevin Grandon :kgrandon]

Thanks for trying that revert John. I'll keep an eye on this thread and your run and will step-in if needed.

Comment 10

[Julien Wajsberg [:julienw]]

Uplifted bug 998759 which wasn't on v1.4. Sorry about this, we should have done this long ago.

Comment 11

[Julien Wajsberg [:julienw]]

Let's see what https://tbpl.mozilla.org/?rev=2717b77b1e1f7f5597513d4c78a579372eb0fa11&tree=Gaia-Try will do.

Comment 12

[Ryan VanderMeulen [:RyanVM]]

(In reply to Ryan VanderMeulen [:RyanVM UTC-4] from comment #3)
> I can kick off a run on a previously-green push as well.

Retriggered and failed in the same manner.

Comment 13

[Julien Wajsberg [:julienw]]

Looks green now !