test_lockscreen_unlock_to_camera_with_passcode.TestCameraUnlockWithPasscode is failing after Bug 951978

RESOLVED FIXED

Status

Firefox OS
Gaia::UI Tests
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Bebe, Assigned: Bebe)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
This test started to fail after Bug 951978 - (secure-window) [Window Management] Implement Secure Window (for Secure Camera App on Lockscreen)

test_unlock_to_camera_with_passcode (test_lockscreen_unlock_to_camera_with_passcode.TestCameraUnlockWithPasscode) ... ERROR

======================================================================
ERROR: None
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/florinstrugariu/.virtualenvs/gaia/local/lib/python2.7/site-packages/marionette_client-0.7.3-py2.7.egg/marionette/marionette_test.py", line 143, in run
    testMethod()
  File "/home/florinstrugariu/gaia/gaia/tests/python/gaia-ui-tests/gaiatest/tests/functional/lockscreen/test_lockscreen_unlock_to_camera_with_passcode.py", line 30, in test_unlock_to_camera_with_passcode
    lock_screen.wait_for_lockscreen_not_visible()
  File "/home/florinstrugariu/gaia/gaia/tests/python/gaia-ui-tests/gaiatest/apps/lockscreen/app.py", line 52, in wait_for_lockscreen_not_visible
    self.wait_for_element_not_displayed(*self._lockscreen_locator)
  File "/home/florinstrugariu/gaia/gaia/tests/python/gaia-ui-tests/gaiatest/apps/base.py", line 90, in wait_for_element_not_displayed
    'Element %s still visible after timeout' % locator)
TimeoutException: TimeoutException: Element lockscreen still visible after timeout
TEST-UNEXPECTED-FAIL | test_lockscreen_unlock_to_camera_with_passcode.py test_lockscreen_unlock_to_camera_with_passcode.TestCameraUnlockWithPasscode.test_unlock_to_camera_with_passcode | 
----------------------------------------------------------------------
Ran 1 test in 75.269s


It looks like the lock screen is detected visible after we tap the camera button


Failing on build: 
Gaia      0bc0e703df197d46dfffb9ac65cb85d2e3e10c4a
Gecko     http://hg.mozilla.org/mozilla-central/rev/bf49e4428906
BuildID   20140130040201
Version   29.0a1
ro.build.version.incremental=eng.archermind.20131114.105818
ro.build.date=Thu Nov 14 10:58:33 CST 2013
(Assignee)

Comment 1

5 years ago
Greg  is this expected ? Should we fix the test?
Flags: needinfo?(gweng)

Comment 2

5 years ago
Bebe, if this works manually then the test needs to be updated.
On my commit, I had left the Travis link to indicate that my patch passed all test jobs, include the Gaia UI test:

https://travis-ci.org/mozilla-b2g/gaia/jobs/17738183

And I found that the test you mentioned was TEST-SKIP. So I don't know whether the test is now set to not skip after my patch (and thus an error occurs), or the test just only be skipped on Travis and make me can't have a clue about this until you now fired the bug.
Flags: needinfo?(gweng)
(Assignee)

Comment 4

5 years ago
Greg:
The test is not run on Travis because it opens the camera app (Travis uses a desktop build).

Zac:
The test woks fine manually and the test might need updating.

My question here is is normal for the lockscreen locator (#lockscreen) to be detected visible if we tap the camera icon with a pass key on.

Comment 5

5 years ago
This is probably a Marionette bug then.
(Assignee)

Comment 6

5 years ago
Mdas Can you take a look over this issue.

We think it might be a marionette issue
Flags: needinfo?(mdas)
(Assignee)

Comment 7

5 years ago
Created attachment 8368532 [details] [review]
pull request
Attachment #8368532 - Flags: review?(zcampbell)
Attachment #8368532 - Flags: review?(robert.chira)

Comment 8

5 years ago
Comment on attachment 8368532 [details] [review]
pull request

r+
Attachment #8368532 - Flags: review?(zcampbell)
Attachment #8368532 - Flags: review?(robert.chira)
Attachment #8368532 - Flags: review+
(Assignee)

Updated

5 years ago
Assignee: nobody → florin.strugariu
So...would this bug should be marked as RESOLVED FIXED?
Flags: needinfo?(florin.strugariu)

Comment 11

5 years ago
Yes this is Fixed.

We've modified the test case.
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Flags: needinfo?(mdas)
Flags: needinfo?(florin.strugariu)
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.