1195033 - tap_update(self, return_details=False) is not safe, needs workaround for bug 1109213

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::UI Tests, defect)

Description

[Martijn Wargers (dead)]

See:
http://mxr.mozilla.org/gaia/search?string=.tap_update%28
http://mxr.mozilla.org/gaia/source/tests/python/gaia-ui-tests/gaiatest/apps/contacts/regions/contact_form.py#149

In the case of return_details=False, tap_update is not safe. When I add a time.sleep(1) call after line 163, I get errors like in bug 1109213.
That's it not causing not any troubles currently is only luck, I guess.

Comment 1

[Martijn Wargers (dead)]

Ok, bug 1188924 is now checked in, so we should make use of that here.

Comment 2

[Martijn Wargers (dead)]

(In reply to Martijn Wargers [:mwargers] (QA) from comment #0)
> In the case of return_details=False, tap_update is not safe. When I add a
> time.sleep(1) call after line 163, I get errors like in bug 1109213.
> That's it not causing not any troubles currently is only luck, I guess.

Ok, that it's not failing constantly, is because of the self.apps.displayed_app part, that one is switching to the system frame: http://mxr.mozilla.org/gaia/source/tests/python/gaia-ui-tests/gaiatest/gaia_test.py#111
If you directly move back to the system app (or other app that keeps to be open), then you won't get this error.
But I don't think this is a good situation, currently.

Comment 3

[Martijn Wargers (dead)]

More places that are potentially problematic:
http://mxr.mozilla.org/gaia/search?string=Wait%28self.marionette%29.until%28lambda+m%3A+self.apps.displayed_app.name+!%3D&find=&findi=&filter=^[^\0]*%24&hitlimit=&tree=gaia

http://mxr.mozilla.org/gaia/source/tests/python/gaia-ui-tests/gaiatest/apps/wallpaper/app.py#21
http://mxr.mozilla.org/gaia/source/tests/python/gaia-ui-tests/gaiatest/apps/homescreen/regions/bookmark_menu.py#35
http://mxr.mozilla.org/gaia/source/tests/python/gaia-ui-tests/gaiatest/apps/contacts/regions/contact_form.py#227 (this is a11y_click_done )
http://mxr.mozilla.org/gaia/source/tests/python/gaia-ui-tests/gaiatest/apps/gallery/regions/view_image.py#43
http://mxr.mozilla.org/gaia/source/tests/python/gaia-ui-tests/gaiatest/apps/contacts/app.py#157

Comment 4

[Martijn Wargers (dead)]

Note that if these places make use of the system app, then it's no problem, because the system app is never closed.

Comment 5

[Martijn Wargers (dead)]

Attachment: all.diff

I did some testing with the methods that seemed susceptible to this problem.
With this patch, I was able to reproduce problems in these:


http://mxr.mozilla.org/gaia/source/tests/python/gaia-ui-tests/gaiatest/apps/wallpaper/app.py#21
test_settings_wallpaper.py

http://mxr.mozilla.org/gaia/source/tests/python/gaia-ui-tests/gaiatest/apps/homescreen/regions/bookmark_menu.py#35
test_browser_bookmark.py
test_rocketbar_add_collection_save_bookmark.py

http://mxr.mozilla.org/gaia/source/tests/python/gaia-ui-tests/gaiatest/apps/contacts/app.py#159
test_sms_add_contact.py 

I didn't get a problem with this one:
http://mxr.mozilla.org/gaia/source/tests/python/gaia-ui-tests/gaiatest/apps/contacts/regions/contact_form.py#164
/test_add_to_an_existing_contact_from_messages.py 

tap_back_button usage:
http://mxr.mozilla.org/gaia/search?string=tap_back_button
This one I can't test locally:
http://mxr.mozilla.org/gaia/source/tests/python/gaia-ui-tests/gaiatest/apps/gallery/regions/view_image.py#43
test_sms_with_picture_attached.py
The browser one isn't closing an app at all.
The other tap_back_button functions don't seem to be used at all.

Comment 6

[GitHub Autolander Bot]

Attachment: [gaia] mwargers:1195033 > mozilla-b2g:master

Comment 7

[Martijn Wargers (dead)]

Comment on attachment 8651551 [details] [review]
[gaia] mwargers:1195033 > mozilla-b2g:master

Ok, these places need it.

Comment 8

[Johan Lorenzo [:jlorenzo]]

Comment on attachment 8651551 [details] [review]
[gaia] mwargers:1195033 > mozilla-b2g:master

LGTM

Comment 9

[No-Jun Park [:njpark]]

Comment on attachment 8651551 [details] [review]
[gaia] mwargers:1195033 > mozilla-b2g:master

Aha, so this is how I should handle NoSuchWindows exception. thanks!

Comment 10

[Martijn Wargers (dead)]

Merged: https://github.com/mozilla-b2g/gaia/commit/6a84d5542d34b7a414062750cf022e5b576fc0a0

I guess we should mention bug 1164078 in the centralized workaround. That is currently the bug covering this NoSuchWindow exception.