What I'm seeing here:
- the problem appears when cleanupTestCase() tries to destroy the components
- cleanupTestCase() destroys Tabs
- TabBar is somehow destroyed before its style.
- tabsModel in TabBarStyle becomes null because styledItem becomes null
- isVisible() in TabBarStyle tries  to access tabsModel.count and that triggers 
the error.

Adding 
"if (!tabsModel) return false"
is enough to fix the test, although I'm not sure that's the way we want to go. 
We should investigate why that wasn't needed before Qt5.6 and why, for 
instance, the "selected" property binding (which still relies on 
tabsModel.count) does not fire any error.

-- 
You received this bug notification because you are a member of Ubuntu
Touch seeded packages, which is subscribed to ubuntu-ui-toolkit in
Ubuntu.
https://bugs.launchpad.net/bugs/1622577

Title:
  tst_pagestack_deprecated_toolbar unit test warnings with qt56

Status in ubuntu-ui-toolkit package in Ubuntu:
  Confirmed

Bug description:
  ubuntu@xenial1:~/dev/ubuntu-ui-toolkit/staging/tests/unit/components$ 
qmltestrunner -import ../../../qml/ -input tst_pagestack_deprecated_toolbar.qml 
  qml: Page.tools is a deprecated property. Please use Page.head instead.
  qml: Page.tools is a deprecated property. Please use Page.head instead.
  qml: WARNING! Do not put Page/Tabs/PageStack inside another Page because that 
causes confusion which is the active page that sets the title and actions.
  qml: WARNING! Do not put Page/Tabs/PageStack inside another Page because that 
causes confusion which is the active page that sets the title and actions.
  ********* Start testing of qmltestrunner *********
  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   : qmltestrunner::PageStackAPI::initTestCase()
  PASS   : qmltestrunner::PageStackAPI::test_active_bug1260116()
  QDEBUG : qmltestrunner::PageStackAPI::test_active_bug1260116() qml: 
Page.tools is a deprecated property. Please use Page.head instead.
  PASS   : qmltestrunner::PageStackAPI::test_currentPage()
  PASS   : qmltestrunner::PageStackAPI::test_depth()
  PASS   : qmltestrunner::PageStackAPI::test_pop_to_tabs_bug1316736()
  PASS   : qmltestrunner::PageStackAPI::test_tabs_inside_stack_bug1187850()
  PASS   : qmltestrunner::PageStackAPI::test_title_bug1143345_bug1317902()
  PASS   : qmltestrunner::PageStackAPI::test_tools_bug1126197()
  QDEBUG : qmltestrunner::PageStackAPI::test_tools_bug1126197() qml: Page.tools 
is a deprecated property. Please use Page.head instead.
  PASS   : qmltestrunner::PageStackAPI::cleanupTestCase()
  QWARN  : qmltestrunner::UnknownTestFunc() 
file:///home/ubuntu/dev/ubuntu-ui-toolkit/staging/qml/Ubuntu/Components/Themes/Ambiance/1.2/TabBarStyle.qml:144:
 TypeError: Cannot read property 'count' of null
  QWARN  : qmltestrunner::UnknownTestFunc() 
file:///home/ubuntu/dev/ubuntu-ui-toolkit/staging/qml/Ubuntu/Components/Themes/Ambiance/1.2/TabBarStyle.qml:144:
 TypeError: Cannot read property 'count' of null
  QWARN  : qmltestrunner::UnknownTestFunc() 
file:///home/ubuntu/dev/ubuntu-ui-toolkit/staging/qml/Ubuntu/Components/Themes/Ambiance/1.2/TabBarStyle.qml:144:
 TypeError: Cannot read property 'count' of null
  QWARN  : qmltestrunner::UnknownTestFunc() 
file:///home/ubuntu/dev/ubuntu-ui-toolkit/staging/qml/Ubuntu/Components/Themes/Ambiance/1.2/TabBarStyle.qml:144:
 TypeError: Cannot read property 'count' of null
  QWARN  : qmltestrunner::UnknownTestFunc() 
file:///home/ubuntu/dev/ubuntu-ui-toolkit/staging/qml/Ubuntu/Components/Themes/Ambiance/1.2/TabBarStyle.qml:144:
 TypeError: Cannot read property 'count' of null
  QWARN  : qmltestrunner::UnknownTestFunc() 
file:///home/ubuntu/dev/ubuntu-ui-toolkit/staging/qml/Ubuntu/Components/Themes/Ambiance/1.2/TabBarStyle.qml:144:
 TypeError: Cannot read property 'count' of null
  Totals: 9 passed, 0 failed, 0 skipped, 0 blacklisted
  ********* Finished testing of qmltestrunner *********

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/ubuntu-ui-toolkit/+bug/1622577/+subscriptions

-- 
Mailing list: https://launchpad.net/~touch-packages
Post to     : touch-packages@lists.launchpad.net
Unsubscribe : https://launchpad.net/~touch-packages
More help   : https://help.launchpad.net/ListHelp

Reply via email to