test_tz_list_initially_empty started failing in image #29

Bug #1319711 reported by Timo Jyrinki
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ubuntu-system-settings (Ubuntu)
Fix Released
High
Brendan Donegan

Bug Description

As seen on http://ci.ubuntu.com/smokeng/utopic/touch/mako/30:20140515:20140513.2/8026/ubuntu_system_settings/ , test_tz_list_initially_empty has started failing.

Maybe related to new tzdata that went in to #29?

Related branches

Revision history for this message
Iain Lane (laney) wrote :

Unlikely. If you look at what the test does it's simply checking that you see the correct empty list, which is quite unrelated to tzdata.

I also can't reproduce it. Can you?

Revision history for this message
Timo Jyrinki (timo-jyrinki) wrote :

Indeed I was not able to reproduce it either, but it seems to happen always on test infra. I'm assuming some sort of environment difference.

Changed in ubuntu-system-settings (Ubuntu):
status: New → Incomplete
Revision history for this message
Iain Lane (laney) wrote :

It's flaky, since the latest CI run passed too.

Victor, do you think you can have a look and see if the latest AP changes might have introduced this somewhere? Like a select that needs to be wait_select or something. Thanks!

Changed in ubuntu-system-settings (Ubuntu):
assignee: nobody → Víctor R. Ruiz (vrruiz)
Revision history for this message
Brendan Donegan (brendan-donegan) wrote :

+65 labelVisible = self.system_settings.main_view.select_single(
            objectName='nothingLabel').visible

This should probably be wait_select_single. Obviously since we can't reproduce the issue locally

I'll push this change up and see if it makes a difference.

Changed in ubuntu-system-settings (Ubuntu):
assignee: Víctor R. Ruiz (vrruiz) → Brendan Donegan (brendan-donegan)
status: Incomplete → In Progress
importance: Undecided → High
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubuntu-system-settings - 0.1+14.10.20140606-0ubuntu1

---------------
ubuntu-system-settings (0.1+14.10.20140606-0ubuntu1) utopic; urgency=low

  [ Michael Terry ]
  * Use XDG_GREETER_DATA_DIR to store background files in a folder that
    the greeter can also read. This fixes the recent regression in not
    being able to choose a ContentHub file for the welcome screen. (LP:
    #1325505)

  [ Iain Lane ]
  * Inline setUpImages(), otherwise the function call doesn't do
    anything.

  [ Brendan Donegan ]
  * Make test_tz_list_initially_empty more robust by using
    wait_select_single instead of select_single to get the label. (LP:
    #1319711)
 -- Ubuntu daily release <email address hidden> Fri, 06 Jun 2014 14:39:39 +0000

Changed in ubuntu-system-settings (Ubuntu):
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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