Closed Bug 1233525 Opened 4 years ago Closed 4 years ago

Intermittent TEST-UNEXPECTED-FAIL | apps/homescreen/test/marionette/app_uninstall_pending_test.js | Homescreen - App uninstall while pending uninstall the app

Categories

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

ARM
Gonk (Firefox OS)
defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
2.6 S5 - 1/15

People

(Reporter: mikehenrty, Assigned: cwiiis)

References

Details

(Keywords: intermittent-failure, Whiteboard: [MJS])

Attachments

(2 files)

TEST-UNEXPECTED-FAIL | apps/homescreen/test/marionette/app_uninstall_pending_test.js | Homescreen - App uninstall while pending uninstall the app
Error: timeout exceeded!
    at Object.Client.waitForSync (node_modules/marionette-client/lib/marionette/client.js:760:16)
    at Object.Client.waitFor (node_modules/marionette-client/lib/marionette/client.js:726:60)
    at Object.MarionetteHelper.waitForElement (node_modules/marionette-helper/index.js:142:12)
    at Object.removeButton (apps/homescreen/test/marionette/lib/homescreen.js:65:31)
    at Context.<anonymous> (apps/homescreen/test/marionette/app_uninstall_pending_test.js:62:9)
    at Test.MarionetteTest.run (node_modules/marionette-js-runner/lib/ui.js:25:31)
    at apps/homescreen/test/marionette/app_uninstall_pending_test.js:23:7
    at ChildProcess.<anonymous> (apps/homescreen/test/marionette/server/parent.js:187:5)
    at handleMessage (internal/child_process.js:686:10)
    at Pipe.channel.onread (internal/child_process.js:440:11)
TEST-END | apps/homescreen/test/marionette/app_uninstall_pending_test.js | Homescreen - App uninstall while pending uninstall the app took 63174 ms
Assignee: nobody → chrislord.net
Target Milestone: --- → 2.6 S5 - 1/15
This I guess could either be not waiting for the app to be fully loaded (fixed by the patch in bug 1223321), the edit bar not being fully visible, or some other thing with the events screwing up somehow...

I'm going to assume it's the first and second, and will write a patch to address the second.
Status: NEW → ASSIGNED
Depends on: 1223321
Comment on attachment 8704638 [details] [review]
[gaia] Cwiiis:bug1233525-homescreen-app-uininstall-pending-test > mozilla-b2g:master

50 consecutive greens: https://treeherder.mozilla.org/#/jobs?repo=gaia&revision=144d9e12906752a547eb4f5b477c4ccd79315578
Attachment #8704638 - Flags: review?(gmarty)
Comment on attachment 8704638 [details] [review]
[gaia] Cwiiis:bug1233525-homescreen-app-uininstall-pending-test > mozilla-b2g:master

Looks good!
Attachment #8704638 - Flags: review?(gmarty) → review+
Merged: https://github.com/mozilla-b2g/gaia/commit/2f0602ec0c54bbe6361286e9213c24646dd74a2a
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.