985341 - Fix uitest-privileged for test_video_capture_prompt failure

Status: RESOLVED FIXED

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

Description

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

Buri master build
Gaia      c03a6af9028c4b74a84b5a98085bbb0c07261175
Gecko     https://hg.mozilla.org/mozilla-central/rev/082761b7bc54
BuildID   20140318160201
Version   31.0a1
ro.build.version.incremental=eng.archermind.20131114.105818
ro.build.date=Thu Nov 14 10:58:33 CST 2013


TEST-START test_privileged_app_audio_capture_prompt.py

test_audio_capture_prompt (test_privileged_app_audio_capture_prompt.TestPrivilegedAppAudioCapturePrompt) ... ok

test_video_capture_prompt (test_privileged_app_audio_capture_prompt.TestPrivilegedAppAudioCapturePrompt) ... ERROR

======================================================================
ERROR: None
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/var/lib/jenkins/jobs/B2G.master.pvt.hamachi.gaiatest/workspace/tests/python/gaia-ui-tests/.env/local/lib/python2.7/site-packages/marionette_client-0.7.5-py2.7.egg/marionette/marionette_test.py", line 163, in run
    testMethod()
  File "/var/lib/jenkins/jobs/B2G.master.pvt.hamachi.gaiatest/workspace/tests/python/gaia-ui-tests/gaiatest/tests/functional/system/test_privileged_app_audio_capture_prompt.py", line 45, in test_video_capture_prompt
    permission.wait_for_permission_dialog_displayed()
  File "/var/lib/jenkins/jobs/B2G.master.pvt.hamachi.gaiatest/workspace/tests/python/gaia-ui-tests/gaiatest/apps/homescreen/regions/permission_dialog.py", line 17, in wait_for_permission_dialog_displayed
    self.wait_for_element_displayed(*self._permission_dialog_locator)
  File "/var/lib/jenkins/jobs/B2G.master.pvt.hamachi.gaiatest/workspace/tests/python/gaia-ui-tests/gaiatest/apps/base.py", line 42, in wait_for_element_displayed
    lambda m: m.find_element(by, locator).is_displayed())
  File "/var/lib/jenkins/jobs/B2G.master.pvt.hamachi.gaiatest/workspace/tests/python/gaia-ui-tests/.env/local/lib/python2.7/site-packages/marionette_client-0.7.5-py2.7.egg/marionette/wait.py", line 143, in until
    cause=last_exc)
TimeoutException: TimeoutException: Timed out after 30.1082799435 seconds
TEST-UNEXPECTED-FAIL | test_privileged_app_audio_capture_prompt.py test_privileged_app_audio_capture_prompt.TestPrivilegedAppAudioCapturePrompt.test_video_capture_prompt | 
----------------------------------------------------------------------
Ran 2 tests in 117.159s

FAILED (errors=1)

Comment 1

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

Attachment: Screenshot

There is "Permission Denied" message on the screen.

Comment 2

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

### STR
1. launch UI Test-Privileged
2. "getUserMedia > Video1"

### Expected
1. show permission dialog

### Actual
1. Permission Denied, no permission dialog

Comment 3

[[e-mail]]

Attachment: pull request redirect to github

Comment 4

[[:AndreiH]]

Comment on attachment 8393403 [details] [review]
pull request redirect to github

Adding the test in a separate file looking good.
But the test still fails with "Permision Denied"

Comment 5

[[e-mail]]

I think 'The gUM is not complete yet' message was removed in bug 983572 though.

Test it in real device and works fine.

Comment 6

[Florin Strugariu [:Bebe]]

Comment on attachment 8393403 [details] [review]
pull request redirect to github

If we split in 2 files can you also add a xfail with the appropriate bug please

Comment 7

[Zac C (:zac)]

Comment on attachment 8393403 [details] [review]
pull request redirect to github

r-, looks like this either fails on device or is intermittent.

Comment 8

[[e-mail]]

travis green, push to try server
https://tbpl.mozilla.org/?tree=Try&rev=71cddc01f52e

with following options
https://gist.github.com/gasolin/9679032

Comment 9

[[e-mail]]

new PR https://github.com/mozilla-b2g/gaia/pull/17416

Comment 10

[Zac C (:zac)]

Forgot about this..

Still want to take this patch because it separates the test into separate files.

gasolin, can you rebase? thanks!

Comment 11

[[e-mail]]

Attachment: pull request redirect to github 2

rebased and push to try https://tbpl.mozilla.org/?tree=Try&rev=85a056044ba8

Comment 12

[Zac C (:zac)]

Comment on attachment 8411535 [details] [review]
pull request redirect to github 2

r+

The manifest file tag is wrong but I will fix that in a follow up commit.

Comment 13

[Zac C (:zac)]

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

Manifest fix:
https://github.com/mozilla-b2g/gaia/commit/9d0bc2ec3b504a2ce815c48a49335fad33af754b

Comment 14

[[e-mail]]

thanks!