Closed Bug 965806 Opened 10 years ago Closed 10 years ago

test_lockscreen_unlock_to_camera_with_passcode.TestCameraUnlockWithPasscode is failing after Bug 951978

Categories

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

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: Bebe, Assigned: Bebe)

Details

Attachments

(1 file)

46 bytes, text/x-github-pull-request
zcampbell
: review+
Details | Review
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
Greg  is this expected ? Should we fix the test?
Flags: needinfo?(gweng)
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)
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.
This is probably a Marionette bug then.
Mdas Can you take a look over this issue.

We think it might be a marionette issue
Flags: needinfo?(mdas)
Attached file pull request
Attachment #8368532 - Flags: review?(zcampbell)
Attachment #8368532 - Flags: review?(robert.chira)
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: nobody → florin.strugariu
So...would this bug should be marked as RESOLVED FIXED?
Flags: needinfo?(florin.strugariu)
Yes this is Fixed.

We've modified the test case.
Status: NEW → RESOLVED
Closed: 10 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.

Attachment

General

Created:
Updated:
Size: