Merge lp:~elopio/ubuntu-ui-toolkit/fix_tab_switch into lp:ubuntu-ui-toolkit

[Leo Arias]

Fixed the tab switch on autopilot emulators that used to rely on the order of the tree.

There where two places where the autopilot emulators relied on the order of the tree. Now, instead of that, they rely on the index property of the elements.
For this, I had to merge zsombi's branch that added the index property to the Tabs.
Here I'm also adding debug information to the log while switching tabs, hoping that it will help to diagnose other failures that happen sometimes on Jenkins. This logs are temporary. Once we can update to autopilot 1.4 we will have a much more elegant way to log all the user actions simulated by autopilot.

[I Ahmad (iahmad)]

This line import pdb; pdb.set_trace() below will stop the execution if this is picked up by ci. Any reason you are leaving this in your merge proposal?

[Leo Arias (elopio)]

> This line import pdb; pdb.set_trace() below will stop the execution if this
> is picked up by ci. Any reason you are leaving this in your merge proposal?

No, thanks for catching it.

[Zsombor Egri (zsombi)]

Looks good to me. Thanks!

review: Approve

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:776
http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-ci/837/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-saucy-vm/191
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-touch/2056
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-saucy-amd64-ci/694
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-saucy-armhf-ci/694
        deb: http://jenkins.qa.ubuntu.com/job/ubuntu-ui-toolkit-saucy-armhf-ci/694/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/autopilot-testrunner-vm-saucy/141
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-saucy-i386/4339
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-saucy-i386/4339/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-saucy-armhf/2058
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-saucy-armhf/2058/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-maguro/1745
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/1762

Click here to trigger a rebuild:
http://s-jenkins:8080/job/ubuntu-ui-toolkit-ci/837/rebuild

review: Approve (continuous-integration)

[Zsombor Egri (zsombi)]

Had to reject as prerequisite wasn't landing. Merged into prerequisite MR so we can proceed with the release.

Unmerged revisions

=== modified file 'tests/autopilot/ubuntuuitoolkit/emulators.py'
--- tests/autopilot/ubuntuuitoolkit/emulators.py	2013-09-21 19:08:50 +0000
+++ tests/autopilot/ubuntuuitoolkit/emulators.py	2013-10-02 07:16:16 +0000
@@ -14,11 +14,15 @@
 # You should have received a copy of the GNU Lesser General Public License
 # along with this program. If not, see <http://www.gnu.org/licenses/>.
 
+import logging
+
 from autopilot import input, platform
 from autopilot.introspection import dbus
 
 _NO_TABS_ERROR = 'The MainView has no Tabs.'
 
+logger = logging.getLogger(__name__)
+
 
 class ToolkitEmulatorException(Exception):
     """Exception raised when there is an error with the emulator."""
@@ -111,6 +115,7 @@
         :return: The newly opened tab.
 
         """
+        logger.debug('Switch to next tab.')
         self.get_header().switch_to_next_tab()
         current_tab = self.get_tabs().get_current_tab()
         current_tab.visible.wait_for(True)
@@ -123,6 +128,7 @@
         :return: The newly opened tab.
 
         """
+        logger.debug('Switch to tab with index {0}.'.format(index))
         tabs = self.get_tabs()
         number_of_tabs = tabs.get_number_of_tabs()
         if index >= number_of_tabs:
@@ -130,6 +136,8 @@
         current_tab = tabs.get_current_tab()
         number_of_switches = 0
         while not tabs.selectedTabIndex == index:
+            logger.debug(
+                'Current tab index: {0}.'.format(tabs.selectedTabIndex))
             if number_of_switches >= number_of_tabs - 1:
                 # This prevents a loop. But if this error is ever raised, it's
                 # likely there's a bug on the emulator or on the QML Tab.
@@ -162,7 +170,7 @@
         tabs = self.get_tabs()
         for index, tab in enumerate(tabs.select_many('Tab')):
             if tab.objectName == object_name:
-                return self.switch_to_tab_by_index(index)
+                return self.switch_to_tab_by_index(tab.index)
         raise ValueError(
             'Tab with objectName "{0}" not found.'.format(object_name))
 
@@ -221,11 +229,23 @@
 
     def get_current_tab(self):
         """Return the currently selected tab."""
-        return self.select_many('Tab')[self.selectedTabIndex]
+        return self._get_tab(self.selectedTabIndex)
+
+    def _get_tab(self, index):
+        tabs = self._get_tabs()
+        for tab in tabs:
+            if tab.index == index:
+                return tab
+        else:
+            raise ToolkitEmulatorException(
+                'There is no tab with index {0}.'.format(index))
+
+    def _get_tabs(self):
+        return self.select_many('Tab')
 
     def get_number_of_tabs(self):
         """Return the number of tabs."""
-        return len(self.select_many('Tab'))
+        return len(self._get_tabs())
 
 
 class TabBar(UbuntuUIToolkitEmulatorBase):
@@ -234,16 +254,19 @@
     def switch_to_next_tab(self):
         """Open the next tab."""
         # Click the tab bar to switch to selection mode.
+        logger.debug('Click the tab bar to enable selection mode.')
         self.pointing_device.click_object(self)
         if not self.selectionMode:
+            logger.debug('Selection mode not enabled, try again.')
             # in case someone stole the click, like the open toolbar
             self.pointing_device.click_object(self)
+        logger.debug('Click the next tab bar button.')
         self.pointing_device.click_object(self._get_next_tab_button())
 
     def _get_next_tab_button(self):
         current_index = self._get_selected_button_index()
         next_index = (current_index + 1) % self._get_number_of_tab_buttons()
-        return self._get_tab_buttons()[next_index]
+        return self._get_tab_button(next_index)
 
     def _get_selected_button_index(self):
         return self.select_single('QQuickPathView').selectedButtonIndex
@@ -254,6 +277,15 @@
     def _get_tab_buttons(self):
         return self.select_many('AbstractButton')
 
+    def _get_tab_button(self, index):
+        buttons = self._get_tab_buttons()
+        for button in buttons:
+            if button.buttonIndex == index:
+                return button
+        else:
+            raise ToolkitEmulatorException(
+                'There is no tab button with index {0}.'.format(index))
+
 
 class ActionSelectionPopover(UbuntuUIToolkitEmulatorBase):
     """ActionSelectionPopover Autopilot emulator."""