Merge lp:~zsombi/ubuntu-ui-toolkit/fixLitsItemFocus56 into lp:ubuntu-ui-toolkit/staging

Proposed by Zsombor Egri on 2016-10-11
Status: Merged
Approved by: Christian Dywan on 2017-01-27
Approved revision: 2143
Merged at revision: 2165
Proposed branch: lp:~zsombi/ubuntu-ui-toolkit/fixLitsItemFocus56
Merge into: lp:ubuntu-ui-toolkit/staging
Diff against target: 1 lines (+0/-0)
0 files modified
To merge this branch: bzr merge lp:~zsombi/ubuntu-ui-toolkit/fixLitsItemFocus56
Reviewer Review Type Date Requested Status
ubuntu-sdk-build-bot continuous-integration Approve on 2017-01-27
Christian Dywan 2016-10-11 Approve on 2017-01-27
Review via email: mp+308118@code.launchpad.net

Commit Message

Get tst_listitem_focus_13 back alive.

Description of the Change

Get tst_listitem_focus_13 back alive.

To post a comment you must log in.
Christian Dywan (kalikiana) wrote :

Self-fixing bug? Works for me.

review: Approve
Christian Dywan (kalikiana) wrote :

1 unit test failures -- search the full log for string FAIL! to see the context.
FAIL! : components::ListItemFocus::test_refocus_listview_on_last_focused_item(With content) 'Focus frame shown for the item' returned FALSE. ()

review: Needs Fixing
Zsombor Egri (zsombi) wrote :

> 1 unit test failures -- search the full log for string FAIL! to see the
> context.
> FAIL! :
> components::ListItemFocus::test_refocus_listview_on_last_focused_item(With
> content) 'Focus frame shown for the item' returned FALSE. ()

Funny is that the submit passed, then autolanding failed. I couldn't see that failure on my xenial + overlay...

Christian Dywan (kalikiana) wrote :

Let's see if this would pass.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== renamed file 'tests/unit/visual/FIXME-QT56_listitem_focus.13.qml' => 'tests/unit/visual/tst_listitem_focus.13.qml'

Subscribers

People subscribed via source and target branches