Closed
Bug 1142558
Opened 9 years ago
Closed 6 years ago
Intermittent failing test, TEST-UNEXPECTED-FAIL | /builds/slave/test/gaia/apps/communications/contacts/test/marionette/ice_test.js | Contacts > ICE contacts ICE settings Check ICE settings transition
Categories
(Firefox OS Graveyard :: Gaia::Contacts, defect)
Tracking
(b2g-master affected)
RESOLVED
WONTFIX
2.2 S9 (3apr)
Tracking | Status | |
---|---|---|
b2g-master | --- | affected |
People
(Reporter: apastor, Assigned: hola)
References
Details
(Keywords: intermittent-failure, leave-open)
Attachments
(1 file)
1) Contacts > ICE contacts ICE settings Check ICE settings transition: 11:35:30 INFO - Error: timeout exceeded! 11:35:30 INFO - at Object.Client.waitForSync (/builds/slave/test/gaia/node_modules/marionette-client/lib/marionette/client.js:693:16) 11:35:30 INFO - at Object.Client.waitFor (/builds/slave/test/gaia/node_modules/marionette-client/lib/marionette/client.js:661:60) 11:35:30 INFO - at Object.MarionetteHelper.waitForElement (/builds/slave/test/gaia/node_modules/marionette-helper/index.js:142:12) 11:35:30 INFO - at clickBackArrow (/builds/slave/test/gaia/apps/communications/contacts/test/marionette/ice_test.js:69:23) 11:35:30 INFO - at Context.<anonymous> (/builds/slave/test/gaia/apps/communications/contacts/test/marionette/ice_test.js:159:7) 11:35:30 INFO - at callFn (/builds/slave/test/gaia/node_modules/mocha/lib/runnable.js:223:21) 11:35:30 INFO - at Test.Runnable.run (/builds/slave/test/gaia/node_modules/mocha/lib/runnable.js:216:7) 11:35:30 INFO - at Runner.runTest (/builds/slave/test/gaia/node_modules/mocha/lib/runner.js:373:10) 11:35:30 INFO - at /builds/slave/test/gaia/node_modules/mocha/lib/runner.js:451:12 11:35:30 INFO - at next (/builds/slave/test/gaia/node_modules/mocha/lib/runner.js:298:14) 11:35:30 INFO - at /builds/slave/test/gaia/node_modules/mocha/lib/runner.js:308:7 11:35:30 INFO - at next (/builds/slave/test/gaia/node_modules/mocha/lib/runner.js:246:23) 11:35:30 INFO - at Object._onImmediate (/builds/slave/test/gaia/node_modules/mocha/lib/runner.js:275:5) 11:35:30 INFO - at processImmediate [as _immediateCallback] (timers.js:330:15)
Reporter | ||
Comment 1•9 years ago
|
||
It seems that this test is failing frequently in 2.2. Francisco, any idea? https://treeherder.mozilla.org/#/jobs?repo=gaia-try&revision=c5365b9f2476 http://ftp.mozilla.org/pub/mozilla.org/b2g/try-builds/gaiabld-c5365b9f2476/gaia-try-linux64_gecko/gaia-try_ubuntu64_vm-b2gdt_test-gaia-js-integration-1-bm123-tests1-linux64-build247.txt.gz
Flags: needinfo?(francisco)
Updated•9 years ago
|
Assignee: nobody → hola
Comment 2•9 years ago
|
||
I think the problems with this test suite have to do with the fact that the ICE Contacts a re set through the UI and not through the data layer. We need to change the way ICE Contacts are set by creating a method that updates the datastore of ICE Contacts directly, instead of doing it through the UI. Assigning it to Adrian
Flags: needinfo?(francisco)
Reporter | ||
Comment 3•9 years ago
|
||
Note that the test is only failing in 2.2 branch
Comment 4•9 years ago
|
||
Assignee | ||
Comment 5•9 years ago
|
||
Comment on attachment 8579223 [details] [review] [gaia] ADLR-es:intermittent-ice > mozilla-b2g:master The error was because the code to close ICE settings was being used to close the contact list when picking an ICE contact. I fixed that, used contacts_data.js to avoid errors when creating contacts and fixed some code duplication.
Attachment #8579223 -
Flags: review?(jmcf)
Comment 6•9 years ago
|
||
Comment on attachment 8579223 [details] [review] [gaia] ADLR-es:intermittent-ice > mozilla-b2g:master thanks Adrian, please check the comments on GH. Once you have the tests passing consider the possibility of asking for an approval as those tests were failing on 2.2
Attachment #8579223 -
Flags: review?(jmcf) → review+
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Assignee | ||
Updated•9 years ago
|
Keywords: checkin-needed
Updated•9 years ago
|
Keywords: checkin-needed
Comment 11•9 years ago
|
||
Pull request has landed in master: https://github.com/mozilla-b2g/gaia/commit/4e651c910b4a48cf5415c8355a9465166b546da4
Updated•9 years ago
|
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Updated•9 years ago
|
status-b2g-master:
--- → fixed
Target Milestone: --- → 2.2 S9 (3apr)
Comment 12•9 years ago
|
||
Gaia-try is looking good, but unfortunately we're having trouble on b2g-inbound. We are currently transitioning b-i to use the same runner as gaia-try (currently using TC). It is a bit painful right now having two different systems, but we hope to fix this soon. Maybe see if the below logs are helpful at all. Revert: https://github.com/mozilla-b2g/gaia/commit/d4f10cd33bb222b38e32c16d86ae98bfe3bbbbcb Log: https://treeherder.mozilla.org/logviewer.html#?job_id=1570995&repo=b2g-inbound
Status: RESOLVED → REOPENED
Flags: needinfo?(hola)
Resolution: FIXED → ---
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Updated•9 years ago
|
Comment hidden (Legacy TBPL/Treeherder Robot) |
Assignee | ||
Comment 27•9 years ago
|
||
The error is from a different file than the one I edited in this patch. Also, I haven't been able to reproduce it in v2.2, with or without my patch. Maybe this is another intermittent test that should be fixed in a new bug, because I don't think it is related to this patch. If possible, running tests again might give us some clues. Thanks.
Flags: needinfo?(hola) → needinfo?(kgrandon)
Comment 28•9 years ago
|
||
Well we can fix it in a new bug, but the problem IIRC is that this patch made it more intermittent on b-i? This may no longer be an issue now that we're running TC everywhere. Now that things are more stable it may be worth submitting a new pull request with the patch again to see where we stand.
Flags: needinfo?(kgrandon)
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment hidden (Legacy TBPL/Treeherder Robot) |
Comment 39•7 years ago
|
||
Closing all intermittent test failures for Firefox OS (since we're not focusing on it anymore). Please reopen if my search included your bug by mistake.
Comment 40•6 years ago
|
||
Firefox OS is not being worked on
Status: REOPENED → RESOLVED
Closed: 9 years ago → 6 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•