1057273 - [Flame][v2.1] Cannot set app permission when executing test_everythingme_search.py

Status: VERIFIED FIXED

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

Description

[William Hsu [:whsu]]

When Gaia UI Test executing test_everythingme_search.py, it cannot set app permission and cause failure.

@ Build Information:
 - Gaia      c8e93dadeaec6a2617b4db8b7d65ba14aa8db378
 - Gecko     https://hg.mozilla.org/mozilla-central/rev/c14e5feadc61
 - BuildID   20140821160203
 - Version   34.0a1

@ ERROR MESSAGE:
starting httpd
running webserver on http://10.247.27.5:57091/
SUITE-START | Running 1 tests
TEST-START | test_everythingme_search.py TestEverythingMeSearch.test_launch_everything_me_search
TEST-UNEXPECTED-ERROR | test_everythingme_search.py TestEverythingMeSearch.test_launch_everything_me_search | JavascriptException: JavascriptException: NS_ERROR_UNEXPECTED: 
stacktrace:
	execute_async_script @gaia_test.py, line 50
	inline javascript, line 80
	src: "      mozPerms.set("
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/marionette_client-0.8.2-py2.7.egg/marionette/marionette_test.py", line 153, in run
    self.setUp()
  File "/home/william/Workspace_B2G/Gaia_Repo/Gaia_Master/tests/python/gaia-ui-tests/gaiatest/tests/functional/everythingme/test_everythingme_search.py", line 14, in setUp
    self.apps.set_permission('Search', 'geolocation', 'deny')
  File "/home/william/Workspace_B2G/Gaia_Repo/Gaia_Master/tests/python/gaia-ui-tests/gaiatest/gaia_test.py", line 50, in set_permission
    (app_name, permission_name, value))
  File "/usr/local/lib/python2.7/dist-packages/marionette_client-0.8.2-py2.7.egg/marionette/marionette.py", line 1247, in execute_async_script
    filename=os.path.basename(frame[0]))
  File "/usr/local/lib/python2.7/dist-packages/marionette_client-0.8.2-py2.7.egg/marionette/decorators.py", line 35, in _
    return func(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/marionette_client-0.8.2-py2.7.egg/marionette/marionette.py", line 634, in _send_message
    self._handle_error(response)
  File "/usr/local/lib/python2.7/dist-packages/marionette_client-0.8.2-py2.7.egg/marionette/marionette.py", line 682, in _handle_error
    raise errors.JavascriptException(message=message, status=status, stacktrace=stacktrace)
TEST-INFO expected PASS | took 35885ms

SUMMARY
-------
passed: 0
failed: 1
todo: 0

FAILED TESTS
-------

Comment 1

[Zac C (:zac)]

According to b2g-i builds this bug started with:
https://tbpl.mozilla.org/?tree=B2g-Inbound&showall=1&rev=45771209ca06

but it doesn't look suspicious.

Let's run it locally with and without that commit.

Comment 2

[Zac C (:zac)]

I'm getting this in my logcat:

08-22 02:31:23.799 E/GeckoConsole( 1499): [JavaScript Error: "NS_ERROR_MALFORMED_URI: Component returned failure code: 0x804b000a (NS_ERROR_MALFORMED_URI) [nsIIOService2.newURI]" {file: "jar:file:///system/b2g/omni.ja!/components/PermissionSettings.js" line: 39}]

Comment 4

[Florin Strugariu [:Bebe]]

The app name has been changed and this is whty we see this fail.

The new app name is:  "Browser2"

https://github.com/mozilla-b2g/gaia/commit/3cae1cc87e8de8adabe006ff115b9e32006d1818#diff-d592df8babed869ec056e5b240f57e8aR13



Should we update the test?

Comment 5

[Jason Smith [:jsmith]]

Kevin - Do you know if the app name change above was expected? Just want to double check if we need to update the tests here.

Comment 6

[Kevin Grandon :kgrandon]

I'm a bit confused. Right now we have a "Browser" and a "Browser2" app. Can you describe what the test is doing? Since the Browser2 app is so new, I don't think we should have any python tests for it yet.

In the next few weeks we will be replacing the Browser app with the Browser2 app, and re-naming it.

Comment 7

[Kevin Grandon :kgrandon]

Oh, I understand now... Hmm is there no way to do this based on manifest URL and not test name? We are going to be updating the test in the next week or so, but go ahead and update the expected app name for now and we should be good.

Comment 8

[Kevin Grandon :kgrandon]

Attachment: Github pull request

Added a patch we can land here if it helps.

Comment 9

[William Hsu [:whsu]]

Thanks Kevin!
I will try this patch to see if it works a expected.
ETA: 8/25 (UTC+8)

Comment 10

[Robert Chira [:RobertC]]

Comment on attachment 8477455 [details] [review]
Github pull request

Merged to master in:
https://github.com/mozilla-b2g/gaia/commit/7d125324bf7df769e41d8f173b01177413a1aa3e

Comment 11

[William Hsu [:whsu]]

Thanks Kevin and Robert! :)
The patch works as expected on my local branch.

---------------------------- LOG ----------------------------
starting httpd
running webserver on http://10.247.27.5:44436/
SUITE-START | Running 1 tests
TEST-START | test_everythingme_launch_link.py TestEverythingMeLaunchLink.test_launch_everything_me_link
TEST-PASS | test_everythingme_launch_link.py TestEverythingMeLaunchLink.test_launch_everything_me_link | took 55149ms

SUMMARY
-------
passed: 1
failed: 0
todo: 0
SUITE-END | took 55s