Closed Bug 1223849 Opened 6 years ago Closed 6 years ago

ID of the settings->bluetooth page have been changed from 'bluetooth_v2' to 'bluetooth'

Categories

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

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: njpark, Assigned: njpark)

Details

Attachments

(2 files)

Found by following error:
TEST-START | test_settings_NC_RTL.py TestSettingsRTLNC.test_settings_app
TEST-UNEXPECTED-ERROR | test_settings_NC_RTL.py TestSettingsRTLNC.test_settings_app | NoSuchElementException: NoSuchElementException: Unable to locate element: bluetooth_v2


Traceback (most recent call last):
  File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/marionette_client-1.0.0-py2.7.egg/marionette/marionette_test.py", line 296, in run
    testMethod()
  File "/Users/g5njpark/GitRepo/gaia/tests/python/gaia-ui-tests/gaiatest/tests/graphics/RTL/test_settings_NC_RTL.py", line 202, in test_settings_app
    bluetooth_page = settings.open_bluetooth()
  File "/Users/g5njpark/GitRepo/gaia/tests/python/gaia-ui-tests/gaiatest/apps/settings/app.py", line 194, in open_bluetooth
    return self._open_subpage(self._bluetooth_menu_item_locator, 'bluetooth', 'Bluetooth')
  File "/Users/g5njpark/GitRepo/gaia/tests/python/gaia-ui-tests/gaiatest/apps/settings/app.py", line 293, in _open_subpage
    return class_object(self.marionette)
  File "/Users/g5njpark/GitRepo/gaia/tests/python/gaia-ui-tests/gaiatest/apps/settings/regions/bluetooth.py", line 28, in __init__
    root = marionette.find_element(*self._root_locator)
  File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/marionette_driver-1.0.0-py2.7.egg/marionette_driver/marionette.py", line 1713, in find_element
    return self._send_message("findElement", body, key="value")
  File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/marionette_driver-1.0.0-py2.7.egg/marionette_driver/decorators.py", line 36, in _
    return func(*args, **kwargs)
  File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/marionette_driver-1.0.0-py2.7.egg/marionette_driver/marionette.py", line 716, in _send_message
    self._handle_error(resp)
  File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/marionette_driver-1.0.0-py2.7.egg/marionette_driver/marionette.py", line 770, in _handle_error
    raise errors.lookup(error)(message, stacktrace=stacktrace)
TEST-INFO took 343949ms
Assignee: nobody → npark
Attachment #8686125 - Flags: review+
Merged:
https://github.com/mozilla-b2g/gaia/commit/fdb66f75963fa9255f707af87f405d54892e5e7d
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Attached patch bluetooth.diffSplinter Review
Currently, pvtbuilds don't contain the bluetooth_v2 to bluetooth change, which caused very confusing errors in this area.
Anyway, I think the __init__ code in this file needs to be improved here, because it's too generic. It needs to be improved like in this patch.
You need to log in before you can comment on or make changes to this bug.