Ubuntu
unity8 package

indicators_client tests need to run under a unity8 session

Bug #1234736 reported by Michał Sawicz
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
unity8 (Ubuntu)
Triaged
Medium
Allan LeSage

Bug Description

We need to split out indicators_client tests into a separate suite from unity8, so that they're run as "normal" apps. So we'd need a .desktop file with NoDisplay=true and a completely separate "indicators_client" suite.

Related branches

lp://staging/~nick-dedekind/unity8/remove-indicators-client-autopilot
PS Jenkins bot (community): Approve (continuous-integration)
Michał Sawicz: Approve
Diff: 341 lines (+0/-287)
8 files modified
CMakeLists.txt (+0/-1)
debian/control (+0/-1)
tests/autopilot/unity8/indicators_client/__init__.py (+0/-8)
tests/autopilot/unity8/indicators_client/emulators/__init__.py (+0/-31)
tests/autopilot/unity8/indicators_client/emulators/common.py (+0/-14)
tests/autopilot/unity8/indicators_client/emulators/main_window.py (+0/-27)
tests/autopilot/unity8/indicators_client/tests/__init__.py (+0/-91)
tests/autopilot/unity8/indicators_client/tests/test_battery.py (+0/-114)
lp://staging/ubuntu/trusty/unity8
Revision history for this message
Nick Dedekind (nick-dedekind) wrote :

Indicator tests are somewhat useless at the moment.
For now it's probably better that we just remove them.

Changed in unity8:
status: Triaged → In Progress
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

Fix committed into lp:unity8 at revision None, scheduled for release in unity8, milestone ubuntu-13.09

Changed in unity8:
status: In Progress → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package unity8 - 7.82+13.10.20131005-0ubuntu1

---------------
unity8 (7.82+13.10.20131005-0ubuntu1) saucy; urgency=low

  [ Michał Sawicz ]
  * Revert r376 that caused constant CPU usage due to the
    ActivityIndicator.

  [ Nick Dedekind ]
  * Removed indicators-client autopilot tests. (LP: #1234736)

  [ Albert Astals ]
  * Revert r388. (LP: #1235268)

  [ Christopher Lee ]
  * Removes passing -fullscreen to unity8 when on the device (as per bug
    #1235065). (LP: #1235065)

  [ Ubuntu daily release ]
  * Automatic snapshot from revision 396
 -- Ubuntu daily release <email address hidden> Sat, 05 Oct 2013 11:45:14 +0000

Changed in unity8 (Ubuntu):
status: New → Fix Released
Revision history for this message
Michał Sawicz (saviq) wrote :

The merged branch alleviated the issue by removing the current tests, now we need to reintroduce them either as part of the unity8 suite or a separate suite for indicators-client.

Changed in unity8:
assignee: Nick Dedekind (nick-dedekind) → Allan LeSage (allanlesage)
milestone: phone-v1-freeze → none
status: Fix Committed → Triaged
importance: Critical → Medium
Changed in unity8 (Ubuntu):
status: Fix Released → Triaged
Michael Zanetti (mzanetti)
tags: added: needs-ap-test
Michał Sawicz (saviq)
Changed in unity8 (Ubuntu):
assignee: nobody → Allan LeSage (allanlesage)
importance: Undecided → Medium
Michał Sawicz (saviq)
no longer affects: unity8
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.