Ubuntu
ubuntu-ui-toolkit package

Scrollbar tests fail on Qt 5.6.

Bug #1624337 reported by Tim Peeters
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ubuntu-ui-toolkit (Ubuntu)
Fix Committed
Critical
Andrea Bernabei

Bug Description

FAIL! : components::Scrollbar::test_defaultStylingValues(vertical scrollbar) Wrong styling property default value.
   Actual (): #5d5d5d
   Expected (): #111111
   Loc: [/tmp/buildd/ubuntu-ui-toolkit-1.3.2085+16.10.20160831.4/tests/unit/visual/tst_scrollbar.13.qml(1205)]
FAIL! : components::Scrollbar::test_defaultStylingValues(horizontal scrollbar) Wrong styling property default value.
   Actual (): #5d5d5d
   Expected (): #111111
   Loc: [/tmp/buildd/ubuntu-ui-toolkit-1.3.2085+16.10.20160831.4/tests/unit/visual/tst_scrollbar.13.qml(1205)]
FAIL! : components::Scrollbar::test_dragThumbAndCheckStyling(vertical scrollbar) Uncaught exception: Type error
   Loc: [/tmp/buildd/ubuntu-ui-toolkit-1.3.2085+16.10.20160831.4/tests/unit/visual/tst_scrollbar.13.qml(800)]
FAIL! : components::Scrollbar::test_dragThumbAndCheckStyling(horizontal scrollbar) Uncaught exception: Type error
   Loc: [/tmp/buildd/ubuntu-ui-toolkit-1.3.2085+16.10.20160831.4/tests/unit/visual/tst_scrollbar.13.qml(838)]

and

********* Start testing of components *********
Config: Using QtTest library 5.6.1, Qt 5.6.1 (x86_64-little_endian-lp64 shared (dynamic) release build; by GCC 5.4.0 20160609)
PASS : components::Scrollbar::initTestCase()
FAIL! : components::Scrollbar::test_handlingOfMovingHeader(Standard header) Uncaught exception: Type error
   Loc: [/tmp/buildd/ubuntu-ui-toolkit-1.3.2085+16.10.20160831.4/tests/unit/visual/tst_scrollbar_header.13.qml(230)]
FAIL! : components::Scrollbar::test_handlingOfMovingHeader(Edit header (with extensions)) Uncaught exception: Type error
   Loc: [/tmp/buildd/ubuntu-ui-toolkit-1.3.2085+16.10.20160831.4/tests/unit/visual/tst_scrollbar_header.13.qml(230)]
PASS : components::Scrollbar::cleanupTestCase()
Totals: 2 passed, 2 failed, 0 skipped, 0 blacklisted
********* Finished testing of components *********

See https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-ci-amd64-devel/1201/consoleFull

Tags: qt5.6

Related branches

lp://qastaging/~faenil/ubuntu-ui-toolkit/fix_scrollbar_tests_qt56
ubuntu-sdk-build-bot: Approve (continuous-integration)
Zsombor Egri: Approve
Cris Dywan: Pending requested
Diff: 43 lines (+4/-4)
2 files modified
tests/unit/visual/tst_scrollbar.13.qml (+3/-3)
tests/unit/visual/tst_scrollbar_header.13.qml (+1/-1)
Andrea Bernabei (faenil)
Changed in ubuntu-ui-toolkit (Ubuntu):
assignee: nobody → Andrea Bernabei (faenil)
importance: Undecided → Critical
status: New → In Progress
Revision history for this message
Andrea Bernabei (faenil) wrote :

We were previously misusing mapToItem/mapFromItem by not passing x/y parameters
and relying on JS to initialize them to 0.

Qt5.6 fixes QTBUG-41686 and requires x/y to be explicitly passed.

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