986960 - Intermittent erroring test, fullscreen_image_test.js

Status: RESOLVED FIXED

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

Description

[Kevin Grandon :kgrandon]

The following test is failing at a rate of about 3% since being re-enabled. As we can't afford to have intermittent failures creep up on us, I'm disabling the test for now. This one will probably be fairly tricky to debug, I would probably recommend using some screenshots to debug.

Error details:


the gallery

...

◦ should flick through images in fullscreen mode:

No output has been received in the last 10 minutes, this potentially indicates a stalled build or something wrong with the build itself.

The build has been terminated

Comment 1

[Kevin Grandon :kgrandon]

Disabled in: https://github.com/mozilla-b2g/gaia/commit/3286e2a0cbcac52e38c0494ab614bac23cdc229f

Comment 2

[No-Jun Park [:njpark]]

Attachment: Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/30527

Comment 3

[No-Jun Park [:njpark]]

Changed the component to Gaia::Gallery (the test concerns the gallery app)

Comment 4

[No-Jun Park [:njpark]]

Comment on attachment 8620537 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/30527

This fix seems to consistently pass locally.  Previously Save button was being pressed although there is no changes to the picture.  (The latest behavior is that the save button will be greyed out unless there is a change to the file)  Enabled auto-enhance to make the change.  The older script also assumed that the app will exit to the thumbnail view after saving the picture, which is no longer the case.  Let me know if I missed anything.

Comment 5

[Punam Dahiya [:pdahiya]]

Thanks No-Jun for updating test as per the new flow and adding appropriate wait calls. Patch looks good and has my r+. One nit noted in github and with that patch looks good to land

Comment 6

[No-Jun Park [:njpark]]

Thanks!  Merged in Gaia:

https://github.com/mozilla-b2g/gaia/commit/9f36b711af7597a6a32471c3305cf1e2d6947d39

Comment 7

[Firefox Bug Husbandry Bot]

Removing leave-open keyword from resolved bugs, per :sylvestre.