test_note_focus_on_second_click_outside failed on mako #159

Bug #1350361 reported by Brendan Donegan
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
notes-app
Won't Fix
Low
Tim Peeters

Bug Description

This test is failing on mako image #159 with the following error:

Traceback (most recent call last):
  File "/home/phablet/autopilot/notes_app/tests/test_create.py", line 106, in test_note_focus_on_second_click_outside
    self.assertThat(content.activeFocus, Eventually(Equals(True)))
  File "/usr/lib/python3/dist-packages/testtools/testcase.py", line 406, in assertThat
    raise mismatch_error
testtools.matchers._impl.MismatchError: After 10.0 seconds test on MixedEdit.activeFocus failed: True != dbus.Boolean(False, variant_level=1)

After a bit of investigation, the reason for this is that the second click is outside the note altogether (rather than outside the content area, but still inside the note, as intended). This means that the note is not focused when the assertion is made. Autopilot is reporting the size of the content area as being the same as the note, which throws the calculation out.

tags: added: lt-category-noimpact lt-date-20140730 lt-prio-low
tags: added: lt-blocker
Steve Langasek (vorlon)
Changed in notes-app:
importance: Undecided → Critical
importance: Critical → Undecided
Revision history for this message
Pat McGowan (pat-mcgowan) wrote :

Assign to Tim so we at least understand it

Changed in notes-app:
assignee: nobody → Tim Peeters (tpeeters)
status: New → In Progress
Revision history for this message
Tim Peeters (tpeeters) wrote :

I cannot reproduce the failure on my utopic desktop

Revision history for this message
Tim Peeters (tpeeters) wrote :

On mako this test fails for me, even when I push the qml files for older versions of UITK. My first guess was that this could be related to TextField/TextArea changes in UITK but reverting to the older versions doesn't seem to fix the test. It might be related to changes in the cpp for UITK (font sizes?) or to something outside of UITK.

Revision history for this message
Pat McGowan (pat-mcgowan) wrote :

This reportedly started failing on build 154 but nothing relevant in the changelog, but may want to compare latest to 153.

tags: removed: lt-blocker
Revision history for this message
Tim Peeters (tpeeters) wrote :

The test passes on image 153.

I installed 153, and upgraded all the packages that changes in 154 (http://people.canonical.com/~ogra/touch-image-stats/154.changes except the two click packages), and still the test passes (even after rebooting the phone).

Revision history for this message
Tim Peeters (tpeeters) wrote :

I could not reproduce the failure on image 154 (after apt-get update so that I could install notes-app-autopilot).

So the test broke in a later image, or it got fixed in an updated package.

Tim Peeters (tpeeters)
Changed in notes-app:
importance: Undecided → Low
Revision history for this message
Tim Peeters (tpeeters) wrote :

Test fine in: 153, 154
Images 155 and 156 doesn't show unity, cannot run tests
Fails in: 175

Revision history for this message
Tim Peeters (tpeeters) wrote :

Test works in image 157.

Revision history for this message
Tim Peeters (tpeeters) wrote :

So it appears to have broken in image 158 or 159, but I will focus my time on other more important issues now.

Changed in notes-app:
status: In Progress → Won't Fix
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.