test_go_to_root/home_must_open_the_root/home_directory fails since mako #132

[Alan Pope 🍺🐧🐱 🦄]

http://ci.ubuntu.com/smokeng/utopic/touch/mako/132:20140715.1:20140712/9071/filemanager/

test_go_to_root_must_open_the_root_directory(with touch)
test_go_home_must_open_the_home_directory(with touch)

Traceback (most recent call last):
  File "/home/phablet/autopilot/filemanager/tests/test_places.py", line 37, in test_go_to_root_must_open_the_root_directory
    self.main_view.go_to_place('placeFileSystem')
  File "/usr/lib/python3/dist-packages/autopilot/logging.py", line 46, in inner
    return f(instance, *args, **kwargs)
  File "/home/phablet/autopilot/filemanager/emulators.py", line 46, in go_to_place
    self._go_to_place_from_popover(object_name)
  File "/home/phablet/autopilot/filemanager/emulators.py", line 60, in _go_to_place_from_popover
    place = popover.select_single('Standard', objectName=object_name)
  File "/usr/lib/python3/dist-packages/autopilot/introspection/dbus.py", line 248, in select_single
    raise StateNotFoundError(type_name_str, **kwargs)
autopilot.exceptions.StateNotFoundError: Object not found with name 'Standard' and properties {'objectName': 'placeFileSystem'}.

Tips on minimizing the occurrence of this failureare available here:
http://developer.ubuntu.com/api/devel/ubuntu-14.10/python/autopilot/faq/troubleshooting.html

[Arto Jalkanen (ajalkane)]

Balloons tip on fixing this problem:

change this line to a wait_select_single
place = popover.select_single('Standard', objectName=object_name)

[Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot)]

Fix committed into lp:ubuntu-filemanager-app at revision 225, scheduled for release in ubuntu-filemanager-app, milestone alpha-1

[Brendan Donegan (brendan-donegan)]

Sorry - but this doesn't fix the issue. Still failing in todays smoke testing. I checked that the fix is present: http://ci.ubuntu.com/smokeng/utopic/touch/mako/135:20140717:20140715.2/9101/filemanager/

[Brendan Donegan (brendan-donegan)]

The basic issue appears to be that PlacesPopover.qml uses the following code to fill the objectName -

objectName: model.objectName

At runtime this evaluates to nothing.

In contrast, the PlacesSidebar.qml component does have objectNames set for placeHome and placeFileSystem, and it uses this code -

objectName: "place" + folderDisplayName(path).replace(/ /g,'')

[Brendan Donegan (brendan-donegan)]

FWIW, placesmodel.h does not define 'objectName' as a property or method.

[Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot)]

Fix committed into lp:ubuntu-filemanager-app at revision 232, scheduled for release in ubuntu-filemanager-app, milestone alpha-1