Merge lp:~elopio/ubuntu-ui-toolkit/fix_tab_switch into lp:ubuntu-ui-toolkit
Status: | Rejected | ||||||||
---|---|---|---|---|---|---|---|---|---|
Rejected by: | Zsombor Egri | ||||||||
Proposed branch: | lp:~elopio/ubuntu-ui-toolkit/fix_tab_switch | ||||||||
Merge into: | lp:ubuntu-ui-toolkit | ||||||||
Prerequisite: | lp:~zsombi/ubuntu-ui-toolkit/tab-index | ||||||||
Diff against target: |
116 lines (+36/-4) 1 file modified
tests/autopilot/ubuntuuitoolkit/emulators.py (+36/-4) |
||||||||
To merge this branch: | bzr merge lp:~elopio/ubuntu-ui-toolkit/fix_tab_switch | ||||||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
PS Jenkins bot | continuous-integration | Approve | |
Zsombor Egri | Approve | ||
Review via email: mp+188774@code.launchpad.net |
This proposal supersedes a proposal from 2013-09-30.
Commit message
Fixed the tab switch on autopilot emulators that used to rely on the order of the tree.
Description of the change
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.
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?