Several issues here:
1) The assert should be at the start of this method, not half way through.
2) You don't need the sleep() statements, please remove them.
3) You don't need to specify the click duration. Please just use "mouse.click()".
66 + for icon in self.launcher.model.get_launcher_icons():
867 + if (icon.tooltip_text != self.ql_app.name):
868 + self.ql_launcher.key_nav_next()
869 + else:
870 + self.keybinding("launcher/keynav/open-quicklist")
871 + self.addCleanup(self.keybinding, "launcher/keynav/close-quicklist")
872 + break
Instead of iterating over the model icons, please use the method on the launcher model, like this:
Hi,
There are still some issues with this I'm afraid:
672 + """Items that can be selected in the quicklist"""
Needs a full-stop at the end of the docstring.
727 + def mouse_click(self, button=1): debug(" Clicking on quicklist item %r", self) move_to( ) self.visible) click(press_ duration= .1)
728 + logger.
729 + self.mouse_
730 + sleep(.25)
731 + assert(
732 + self._mouse.
733 + sleep(.25)
Several issues here:
1) The assert should be at the start of this method, not half way through.
2) You don't need the sleep() statements, please remove them.
3) You don't need to specify the click duration. Please just use "mouse.click()".
66 + for icon in self.launcher. model.get_ launcher_ icons() : launcher. key_nav_ next() ("launcher/ keynav/ open-quicklist" ) (self.keybindin g, "launcher/ keynav/ close-quicklist ")
867 + if (icon.tooltip_text != self.ql_app.name):
868 + self.ql_
869 + else:
870 + self.keybinding
871 + self.addCleanup
872 + break
Instead of iterating over the model icons, please use the method on the launcher model, like this:
icon = self.launcher. model.get_ icon_by_ desktop_ id(...)
If you use the desktop Id then we won't have issues in didfferent locales.
885 + self.assertThat (self.quicklist .selected_ item.id, Equals( expected_ item.id) ) (self.quicklist .selected_ item.id, Equals( expected_ item.id) ) (self.quicklist .selected_ item.id, Equals( expected_ item.id) ) (self.quicklist .selected_ item.id, Equals( expected_ item.id) ) (self.quicklist .selected_ item.id, Equals( expected_ item.id) ) (self.quicklist .selected_ item.id, Equals( expected_ item.id) ) (self.quicklist .selected_ item.id, Equals( expected_ item.id) ) (item.selected) (self.quicklist .selected_ item.id, Equals(item.id)) (mouse_ item.selected) (expected_ item.selected) (self.quicklist .selected_ item.id, Equals( expected_ item.id) ) (mouse_ item.selected) (expected_ item.selected) (self.quicklist .selected_ item.id, Equals( expected_ item.id) ) (mouse_ item.selected) (key_item. selected) (self.quicklist .selected_ item.id, Equals( key_item. id)) (self.quicklist .selected_ item.id, Equals( key_item. id)) (self.quicklist .selected_ item.id, Equals( key_item. id)) (mouse_ item.selected) (self.quicklist .selected_ item.id, Equals( mouse_item. id))
899 + self.assertThat
908 + self.assertThat
917 + self.assertThat
931 + self.assertThat
940 + self.assertThat
949 + self.assertThat
958 + self.assertTrue
959 + self.assertThat
979 + self.assertTrue
984 + self.assertTrue
985 + self.assertThat
995 + self.assertTrue
1000 + self.assertTrue
1001 + self.assertThat
1010 + self.assertTrue
1017 + self.assertTrue
1018 + self.assertThat
1023 + self.assertThat
1027 + self.assertThat
1032 + self.assertTrue
1033 + self.assertThat
Please use the new Eventually() matcher for all of these. To import it:
from autopilot.matchers import Eventually
to use it, take this:
self.assertThat (self.quicklist .selected_ item.id, Equals( mouse_item. id))
and turn it into this:
self.assertThat (self.quicklist .selected_ item.id, Eventually( Equals( mouse_item. id)))
Eventually takes any mater, so you can also do things like this:
self.assertThat (self.quicklist .selected_ item.id, Eventually( LessThan( mouse_item. id)))
Finally, please remove the sleep() statements. Using the Eventually(...) matcher removes the need for random sleep statements in your tests.
Thanks