diff options
author | Mitch Curtis <mitch.curtis@qt.io> | 2021-08-06 12:27:35 +0200 |
---|---|---|
committer | Mitch Curtis <mitch.curtis@qt.io> | 2021-09-13 21:17:04 +0200 |
commit | e310dadef779b28845b41fb091634cd001cda9de (patch) | |
tree | 610be234e82cc53752f58521ae3beba4693ff78f /tests/auto/qml/qqmlimport | |
parent | de0c1c8a4c834dcc080d1b374186a82cd86921ca (diff) |
Consolidate test helpers into private libraries
Previously each test would include and build sources from the shared
folder. Now we make those sources a library, build it once, then have
each test link to it instead.
We also take the opportunity to move some helpers that qtquickcontrols2
had added into the quicktestutils library where it makes sense, and
for the helpers that don't make sense to be there, move them into
quickcontrolstestutils.
We add the libraries to src/ so that they are internal modules built as
part of Qt, rather than tests. That way we can use them in a standalone
test outside of qtdeclarative.
Task-number: QTBUG-95621
Pick-to: 6.2
Change-Id: I0a2ab3976fdbff2e4414df7bdc0808f16453b80a
Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io>
Diffstat (limited to 'tests/auto/qml/qqmlimport')
-rw-r--r-- | tests/auto/qml/qqmlimport/CMakeLists.txt | 4 | ||||
-rw-r--r-- | tests/auto/qml/qqmlimport/tst_qqmlimport.cpp | 14 |
2 files changed, 13 insertions, 5 deletions
diff --git a/tests/auto/qml/qqmlimport/CMakeLists.txt b/tests/auto/qml/qqmlimport/CMakeLists.txt index 3d4f01da2c..6392a527a4 100644 --- a/tests/auto/qml/qqmlimport/CMakeLists.txt +++ b/tests/auto/qml/qqmlimport/CMakeLists.txt @@ -24,15 +24,13 @@ list(APPEND test_data ${test_data_glob}) qt_internal_add_test(tst_qqmlimport SOURCES - ../../shared/util.cpp ../../shared/util.h tst_qqmlimport.cpp - INCLUDE_DIRECTORIES - ../../shared PUBLIC_LIBRARIES Qt::Gui Qt::Qml Qt::QmlPrivate Qt::Quick + Qt::QuickTestUtilsPrivate TESTDATA ${test_data} ) diff --git a/tests/auto/qml/qqmlimport/tst_qqmlimport.cpp b/tests/auto/qml/qqmlimport/tst_qqmlimport.cpp index fdf72b83f0..f52b01cf04 100644 --- a/tests/auto/qml/qqmlimport/tst_qqmlimport.cpp +++ b/tests/auto/qml/qqmlimport/tst_qqmlimport.cpp @@ -34,12 +34,15 @@ #include <QtQuick/qquickitem.h> #include <private/qqmlimport_p.h> #include <private/qqmlengine_p.h> -#include "../../shared/util.h" +#include <QtQuickTestUtils/private/qmlutils_p.h> class tst_QQmlImport : public QQmlDataTest { Q_OBJECT +public: + tst_QQmlImport(); + private slots: void importPathOrder(); void testDesignerSupported(); @@ -175,6 +178,11 @@ void tst_QQmlImport::uiFormatLoading() delete test; } +tst_QQmlImport::tst_QQmlImport() + : QQmlDataTest(QT_QMLTEST_DATADIR) +{ +} + void tst_QQmlImport::importPathOrder() { #ifdef Q_OS_ANDROID @@ -358,7 +366,9 @@ void tst_QQmlImport::removeDynamicPlugin() { // Load something that adds a dynamic plugin QQmlComponent component(&engine); - component.setData(QByteArray("import QtTest 1.0; TestResult{}"), QUrl()); + // Make sure to use something other than QtTest here, since the !plugins.isEmpty() + // check will fail if we do. + component.setData(QByteArray("import QtQuick.tooling; Property{}"), QUrl()); QVERIFY2(component.isReady(), qPrintable(component.errorString())); } QQmlImportDatabase *imports = &QQmlEnginePrivate::get(&engine)->importDatabase; |