1096379 - TEST-UNEXPECTED-FAIL | apps/email/test/marionette/search_test.js | email message list edit mode "before each" hook

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::E-Mail, defect)

Description

[Julien Wajsberg [:julienw]]

06:06:23 INFO - TEST-UNEXPECTED-FAIL | build/test/integration/node_modules.test.js | Node modules tests make node_modules from git mirror
06:06:23 INFO - Error: timeout of 300000ms exceeded
06:06:23 INFO - at null.<anonymous> (/builds/slave/test/gaia/node_modules/mocha/lib/runnable.js:139:19)
06:06:23 INFO - at Timer.listOnTimeout [as ontimeout] (timers.js:110:15) 

https://treeherder.mozilla.org/ui/logviewer.html#?job_id=292255&repo=gaia-try

Comment 1

[Legacy TBPL/Treeherder Robot]

submit_timestamp: 2014-11-10T08:19:40
log: https://treeherder.mozilla.org/ui/logviewer.html#?repo=gaia-try&job_id=290895
repository: gaia-try
who: jwajsberg[at]mozilla[dot]com
machine: tst-linux64-spot-743
buildname: b2g_ubuntu64_vm gaia-try opt test gaia-js-integration-1
revision: 9fb8636f8f6f

TEST-UNEXPECTED-FAIL | apps/email/test/marionette/search_test.js | email message list edit mode "before each" hook
make: *** [test-integration-test] Error 1
Return code: 2
Tests exited with return code 2: harness failures
# TBPL FAILURE #

Comment 2

[Andrew Sutherland [:asuth] (he/him)]

Note that while comment 0 is referencing a different test failure (no sign of search_test.js anywhere in its output), comment 1 is a real failure but is of course mysterious and sheds no light on what's going on, not that I trust the current integration tests anyways.

I propose we just disable this test in favor of the giant "re-enable the tests in a form we have faith in" bug.  James?

Comment 3

[James Burke [:jrburke]]

Attachment: GitHub pull request

I thought all the email tests were already disabled from running. I am guessing that was under the travis reign. I think I found what tbpl uses, and it is curious that it was missing 4 tests, so I just added them all, and best to wait for bug 975588 for them all to be renabled.

Note that I do periodically run the tests locally and for bigger email front end changes, but agree it is not worth stopping others from merging their non-email related code.

Comment 4

[James Burke [:jrburke]]

Merged in master:
https://github.com/mozilla-b2g/gaia/commit/a98528f9a69dae06cbeba9b602c3d9839724250d

from pull request:
https://github.com/mozilla-b2g/gaia/pull/26009

Comment 5

[Julien Wajsberg [:julienw]]

Mmm yeah sorry, I pasted the wrong log in comment 0 :/ comment 1 is definitely what I intended to post.