941727 - Fix up test_unlock_to_camera

Status: RESOLVED FIXED

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

Description

[Zac C (:zac)]

This test is failing after a change to the lockscreen yesterday.

The functionality seems to be working fine.

However it is now running into a Marionette limitation in that Marionette cannot judge the visibility of the frames properly.

I think we should change this entirely to use `switch_to_displayed_app()` and be sure that the Camera app comes to the fore after unlocking to camera.

Comment 1

[Askeing Yen[:askeing][:fyen]]

test_lockscreen_unlock_to_camera.py
##### Before Unlock to Camera #####
(Pdb) ls = self.marionette.find_element('id', 'lockscreen')
(Pdb) lsic = self.marionette.find_element('id', 'lockscreen-icon-container')
(Pdb) ls.location
{u'y': 0, u'x': 0}
(Pdb) lsic.location
{u'y': 379, u'x': 0}
##### After Unlock to Camera #####
(Pdb) ls.location
{u'y': 0, u'x': 0}
(Pdb) lsic.location
{u'y': 0, u'x': 0}


test_lockscreen_unlock_to_camera_with_passcode.py
##### Before Unlock to Camera #####
(Pdb) ls = self.marionette.find_element('id', 'lockscreen')
(Pdb) lsic = self.marionette.find_element('id', 'lockscreen-icon-container')
(Pdb) ls.location
{u'y': 0, u'x': 0}
(Pdb) lsic.location
{u'y': 379, u'x': 0}
##### After Unlock to Camera #####
(Pdb) ls.location
{u'y': -240, u'x': -160}
(Pdb) lsic.location
{u'y': 518, u'x': -160}

Comment 2

[Askeing Yen[:askeing][:fyen]]

Base on comment 2, the gaiatest/apps/lockscreen/app.py Line 51 doesn't work without passcode.

def wait_for_lockscreen_not_visible(self):
    self.wait_for_condition(lambda m: not self.marionette.find_element(*self._lockscreen_locator).location['x'] == 0, message="Lockscreen still visible after unlock")

Comment 3

[Zac C (:zac)]

So yeah we don't really want to use that style of code anymore. 

We want to remove any check that the lockscreen has moved and we can replace it with checking that camera is the displayed app.

So something like:

# after slide to unlock
self.wait_for_condition(lambda m: self.apps.displayed_app == camera.name)

This will make sure the camera is in the foreground.

We could then check the window.Lockscreen.locked javascript to check whether b2G thinks it is unlocked or not.

Comment 4

[[:AndreiH]]

Attachment: Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/13950

Pointer to Github pull-request

Comment 5

[Zac C (:zac)]

I filed this bug about how we can't get the camera as the active app:
https://bugzilla.mozilla.org/show_bug.cgi?id=942230

Comment 6

[Zac C (:zac)]

Comment on attachment 8336743 [details]
Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/13950

r+ tests work nicely.
Not the solution wanted but it is good enough for now.

Comment 7

[Zac C (:zac)]

Merged:
https://github.com/mozilla-b2g/gaia/commit/7949f05ac66443047dd8890334555c7449e2f7c7

The source of this failure is this bug:
https://bugzilla.mozilla.org/show_bug.cgi?id=937630

thus marking v1.2 affected.

Comment 8

[[:AndreiH]]

Attachment: Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/14013

Pointer to Github pull-request

Comment 9

[Viorela Ioia [:viorela]]

Comment on attachment 8337698 [details]
Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/14013

Test works nice. Just update the link.

Comment 10

[Florin Strugariu [:Bebe]]

Comment on attachment 8337698 [details]
Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/14013

except for the nit all looks ok

Comment 11

[[:AndreiH]]

https://github.com/mozilla-b2g/gaia/commit/b2733c5f18af45b78462a90e3002c67680a0b9cd

Comment 12

[Bob Silverberg [:bsilverberg]]

When a commit lands on v1.2, please specify that in the comment (the branch) and please update the status-b2g-v1.2 flag to "fixed".