diff options
author | Ulf Hermann <ulf.hermann@qt.io> | 2019-11-11 18:18:04 +0100 |
---|---|---|
committer | Ulf Hermann <ulf.hermann@qt.io> | 2019-11-18 10:39:23 +0100 |
commit | 6fb335ebcec9c5efc6a2d7786e7bbba23c583d48 (patch) | |
tree | 8cc161f7423757e6820d9db40a943464f98fc3aa /tests/auto/qml/qmllint/tst_qmllint.cpp | |
parent | 61d7713ead80a6f5b972778d2e47843af756039e (diff) |
qmllint: Cleanup
qmllint needed to be refactored before we can add any new functionality:
* Fix all the (C++) linter warnings
* Remove pointless namespaces
* Merge ScopeTree and FakeMetaObject into one class
* Remove the "Fake" from class and variable names
* Remove dead code
* Add "We mean it" warnings everywhere
* Unify #include style
This also "accidentally" fixes the automatic matching of signal handlers
in Connections elements to signals in their parent scopes.
Change-Id: Idf8daae34dfd6c0ee00da28b017f921da3eba25c
Reviewed-by: Simon Hausmann <simon.hausmann@qt.io>
Diffstat (limited to 'tests/auto/qml/qmllint/tst_qmllint.cpp')
-rw-r--r-- | tests/auto/qml/qmllint/tst_qmllint.cpp | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/tests/auto/qml/qmllint/tst_qmllint.cpp b/tests/auto/qml/qmllint/tst_qmllint.cpp index e3c297fa0f..4328f7fe80 100644 --- a/tests/auto/qml/qmllint/tst_qmllint.cpp +++ b/tests/auto/qml/qmllint/tst_qmllint.cpp @@ -128,10 +128,10 @@ void TestQmllint::dirtyQmlCode_data() << QStringLiteral("failure1.js") << QStringLiteral("failure1.js:4 : Expected token `;'") << QString(); - QTest::newRow("UnmatchedSignalHandler") - << QStringLiteral("UnmatchedSignalHandler.qml") - << QString("Warning: no matching signal found for handler \"onClicked\" at 12:13") - << QStringLiteral("onMouseXChanged"); + QTest::newRow("AutomatchedSignalHandler") + << QStringLiteral("AutomatchedSignalHandler.qml") + << QString("Warning: unqualified access at 12:36") + << QStringLiteral("no matching signal found"); } void TestQmllint::dirtyQmlCode() |