Autopilot tests (try to) stop maliit-server
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Ubuntu Calculator App |
Fix Released
|
High
|
Unassigned | ||
Ubuntu Calendar App |
Fix Released
|
High
|
Unassigned | ||
Ubuntu Clock App |
Fix Released
|
High
|
Unassigned | ||
Ubuntu File Manager App |
Fix Released
|
High
|
Unassigned | ||
Ubuntu Shorts App |
Fix Released
|
High
|
Unassigned | ||
Ubuntu Weather App |
Fix Released
|
High
|
Unassigned |
Bug Description
Since revision 168 in lp:ubuntu-calendar-app, the following code was added to the setUp() method of the base test case for autopilot calendar app tests:
#turn off the OSK so it doesn't block screen elements
if model() != 'Desktop':
#adding cleanup step seems to restart service immeadiately
This feels and looks wrong. If we prevent the OSK from appearing, then we’re not really testing the app in normal conditions, are we?
Besides, it simply doesn’t work when I try to run the tests on my Galaxy Nexus, all the tests fail.
Finally, the commented out cleanup doesn’t work because addCleanup expects a function object, not a call. So it should be something like: self.addCleanup
Related branches
- Brendan Donegan: Approve
- Ubuntu Phone Apps Jenkins Bot: Approve (continuous-integration)
- Leo Arias (community): Approve
-
Diff: 15 lines (+0/-5)1 file modifiedtests/autopilot/filemanager/tests/__init__.py (+0/-5)
Changed in ubuntu-calendar-app: | |
status: | New → Confirmed |
importance: | Undecided → High |
tags: | added: elopio |
Changed in ubuntu-calculator-app: | |
status: | New → Confirmed |
Changed in ubuntu-clock-app: | |
status: | New → Confirmed |
Changed in ubuntu-filemanager-app: | |
status: | New → Confirmed |
Changed in ubuntu-rssreader-app: | |
status: | New → Confirmed |
Changed in ubuntu-weather-app: | |
status: | New → Confirmed |
Changed in ubuntu-calculator-app: | |
importance: | Undecided → High |
Changed in ubuntu-clock-app: | |
importance: | Undecided → High |
Changed in ubuntu-filemanager-app: | |
importance: | Undecided → High |
Changed in ubuntu-rssreader-app: | |
importance: | Undecided → High |
Changed in ubuntu-weather-app: | |
importance: | Undecided → High |
Changed in ubuntu-filemanager-app: | |
status: | Confirmed → Fix Released |
Changed in ubuntu-calculator-app: | |
status: | Confirmed → Fix Released |
Changed in ubuntu-calendar-app: | |
status: | Confirmed → Fix Released |
Changed in ubuntu-weather-app: | |
status: | Confirmed → Fix Released |
Changed in ubuntu-clock-app: | |
status: | Confirmed → Fix Released |
Changed in ubuntu-rssreader-app: | |
status: | Confirmed → Fix Released |
tags: | added: needs-autopilot-test |
This is not the only place this code is found.. Let's add the other affected apps. I should know, it's leftover hackiness from me. ::ducks::