PopoverTests::test_popover_follows_pointerTarget_bug1199502(Moving pointerTarget) FAILS when setting GRID_UNIT_PX.

Bug #1569794 reported by Tim Peeters
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
ubuntu-ui-toolkit (Ubuntu)
Fix Released
High
Cris Dywan

Bug Description

tim@tim-mbp:~/dev/ubuntu-ui-toolkit/m/freeze/tests/unit_x11/tst_components$ GRID_UNIT_PX=16 qmltestrunner -import ../../../qml/ -input tst_popover13.qml
********* Start testing of qmltestrunner *********
Config: Using QtTest library 5.5.1, Qt 5.5.1 (x86_64-little_endian-lp64 shared (dynamic) release build; by GCC 5.3.1 20160330)
PASS : qmltestrunner::PopoverTests::initTestCase()
PASS : qmltestrunner::PopoverTests::test_dismiss_on_click(row 0)
PASS : qmltestrunner::PopoverTests::test_dismiss_on_click(row 1)
PASS : qmltestrunner::PopoverTests::test_dismiss_on_click(row 2)
PASS : qmltestrunner::PopoverTests::test_dismiss_on_key()
PASS : qmltestrunner::PopoverTests::test_popover_consumes_clicks_bug1488540(Left-click)
PASS : qmltestrunner::PopoverTests::test_popover_consumes_clicks_bug1488540(Right-click)
PASS : qmltestrunner::PopoverTests::test_popover_consumes_clicks_bug1488540(Middle-click)
FAIL! : qmltestrunner::PopoverTests::test_popover_follows_pointerTarget_bug1199502(Moving pointerTarget) Popover isn't pointing at the caller
   Actual (): 636
   Expected (): 318
   Loc: [/home/tim/dev/ubuntu-ui-toolkit/m/freeze/tests/unit_x11/tst_components/tst_popover13.qml(182)]
PASS : qmltestrunner::PopoverTests::cleanupTestCase()
Totals: 9 passed, 1 failed, 0 skipped, 0 blacklisted
********* Finished testing of qmltestrunner *********

It does not fail when I have GRID_UNIT_PX=8.

Related branches

Tim Peeters (tpeeters)
description: updated
summary: PopoverTests::test_popover_follows_pointerTarget_bug1199502(Moving
- pointerTarget) FAILS
+ pointerTarget) FAILS when setting GRID_UNIT_PX.
Revision history for this message
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in ubuntu-ui-toolkit (Ubuntu):
status: New → Confirmed
Cris Dywan (kalikiana)
Changed in ubuntu-ui-toolkit (Ubuntu):
assignee: nobody → Christian Dywan (kalikiana)
Tim Peeters (tpeeters)
Changed in ubuntu-ui-toolkit (Ubuntu):
importance: Undecided → High
Cris Dywan (kalikiana)
Changed in ubuntu-ui-toolkit (Ubuntu):
status: Confirmed → In Progress
Cris Dywan (kalikiana)
Changed in ubuntu-ui-toolkit (Ubuntu):
status: In Progress → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubuntu-ui-toolkit - 1.3.1984+16.10.20160527.2

---------------
ubuntu-ui-toolkit (1.3.1984+16.10.20160527.2) yakkety; urgency=medium

  [ Christian Dywan ]
  * Slimmer frame for TextFields: 0.5dp. Fixes LP: #1578190.

  [ Albert Astals Cid ]
  * Add override
    The override specifier (since C++11) specifies that a virtual function
    overrides another virtual function. In a member function declaration or
    definition, override ensures that the function is virtual and is overriding
    a virtual function from the base class.

  [ Tim Peeters ]
  * Fix reference error in PullToRefreshStyle. Fixes LP: #1582843
  * Mark Tab, Tabs, TabBar, PageHeadConfiguration, PageHeadSections,
    PageHeadState, ToolbarButton, ToolbarItems as deprecated in the
    documentation. Fixes LP: #1566735, LP: #1566741.

  [ CI Train Bot ]
  * Resync trunk.

 -- Zoltan Balogh <email address hidden> Fri, 27 May 2016 07:08:44 +0000

Changed in ubuntu-ui-toolkit (Ubuntu):
status: Fix Committed → 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.