William Hsu [:whsu] Reporter
	Description • 10 years ago

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
-------

	Zac C (:zac)
	Comment 1 • 10 years ago

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.

	Zac C (:zac)
	Comment 2 • 10 years ago

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}]

	Florin Strugariu [:Bebe]
	Comment 4 • 10 years ago

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?

	Jason Smith [:jsmith]
	Comment 5 • 10 years ago

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.

	Kevin Grandon :kgrandon
	Comment 6 • 10 years ago

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.

	Kevin Grandon :kgrandon
	Comment 7 • 10 years ago

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.

	Kevin Grandon :kgrandon
	Comment 8 • 10 years ago

Added a patch we can land here if it helps.

	William Hsu [:whsu] Reporter
	Comment 9 • 10 years ago

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

	Robert Chira [:RobertC]
	Comment 10 • 10 years ago

Comment on attachment 8477455 [details]
Github pull request

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

	William Hsu [:whsu] Reporter
	Comment 11 • 10 years ago

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