Priority: P2: Important
Affects Version/s: None
Fix Version/s: None
Component/s: Quick: Core Declarative QML
Now that https://codereview.qt-project.org/c/qt/qtbase/+/375915 is merged, we can start preventing new warnings from creeping into qtdeclarative.
My proposal is to do this:
This would make every QQmlDataTest-based test function in qtdeclarative fail if a qWarning is encountered. It would require a bit of work up front: a few tests would probably require a bunch of QTest::ignoreMessages, and
QTBUG-98439 would need to be fixed first.
In addition, we should add a bool failOnWarnings to TestCase, defaulting it to the value of QT_QUICK_TEST_FAIL_ON_WARNINGS if it is set, otherwise false. This will allow us to enable it for all QML tests in the least disruptive way.
This won't cover every test, but it's a good start.
|For Gerrit Dashboard: QTBUG-98718|
|383346,19||WIP: Fix and ignore warnings in tests in preparation for failOnWarnings||dev||qt/qtdeclarative||Status: NEW||-2||0|
|414644,1||WIP: make some tests fail on warnings||dev||qt/qtdeclarative||Status: NEW||-2||0|
|420204,1||tst_qquicklistview: fail on warnings||6.3||qt/qtdeclarative||Status: NEW||+2||0|