This seems to be something we're going to be doing a lot, so please just add a property to the Hud class that gets you the correct icon in all situations. THen these 6 lines become one:
This is ugly - and it's my fault. Please could you patch self.start_app(...) so it returns the BamfApplication instance? It shouldn't be hard to do, since we launch the application in the Bamf Emulator. THen you can replace all that code with this:
Hi,
Great branch - a few small issues with the AP test though:
99 + self.keybinding ("dash/ reveal" , 0.1)
100 + sleep(1)
Please use self.dash. ensure_ visible( ) for this - it's safer, since it doesn't require you to sleep() after calling it.
105 + if self.hud. is_locked_ launcher: hud_launcher_ icon() (hud_launcher_ icon.icon_ name, Equals(calc.icon)) get_embedded_ icon() (hud_embedded_ icon.icon_ name, Equals(calc.icon))
106 + hud_launcher_icon = self.get_
107 + self.assertThat
108 + else:
109 + hud_embedded_icon = self.hud.
110 + self.assertThat
This seems to be something we're going to be doing a lot, so please just add a property to the Hud class that gets you the correct icon in all situations. THen these 6 lines become one:
self.assertThat (self.hud. icon.name, Equals(calc.icon))
... or something similar.
114 + self.start_ app("Calculator ") app_instances( "Calculator" ) (len(calctools) , GreaterThan(0)) (calc.is_ active)
115 + calctools = self.get_
116 + self.assertThat
117 + calc = calctools[0]
118 + self.assertTrue
This is ugly - and it's my fault. Please could you patch self.start_app(...) so it returns the BamfApplication instance? It shouldn't be hard to do, since we launch the application in the Bamf Emulator. THen you can replace all that code with this:
calc = self.start_ app("Calculator ")
Cheers,