feedc0de/qt-creator
1
0
Fork 0
forked from qt-creator/qt-creator
Code Pull Requests Activity

QmlDesigner: Fix version scanning

Browse Source 
Switch was missing.

Change-Id: I5d87e624f698b9f59fe57cf49cafef4e6d08eca2
Reviewed-by: Marco Bubke <marco.bubke@qt.io>
Reviewed-by: Aleksei German <aleksei.german@qt.io>
(cherry picked from commit 45cf905b29)
Reviewed-by: Tim Jenssen <tim.jenssen@qt.io>
This commit is contained in:
Marco Bubke
2023-04-25 11:30:58 +02:00
committed by Tim Jenssen
parent 4761b76a1c
commit c855a6bd48
2 changed files with 18 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
src/plugins/qmldesigner/designercore/projectstorage/modulescanner.cpp
View File

@@ -70,8 +70,11 @@ void ModuleScanner::scan([[maybe_unused]] std::string_view modulePath)
if (moduleName.isEmpty() || m_skip(moduleName))
continue;
m_modules.push_back(
Import::createLibraryImport(moduleName, createVersion(parser.components())));
QString version = m_versionScanning == VersionScanning::Yes
? createVersion(parser.components())
: QString{};
m_modules.push_back(Import::createLibraryImport(moduleName, version));
}
}

13
tests/unit/unittest/modulescanner-test.cpp
View File

@@ -79,6 +79,19 @@ TEST_F(ModuleScanner, Version)
ASSERT_THAT(scanner.modules(), ElementsAre(AllOf(UrlProperty("Example"), VersionProperty("1.3"))));
}
TEST_F(ModuleScanner, NoVersion)
{
QmlDesigner::ModuleScanner scanner{[](QStringView moduleName) {
return moduleName.endsWith(u"impl");
},
QmlDesigner::VersionScanning::No};
scanner.scan(QStringList{TESTDATA_DIR "/modulescanner"});
ASSERT_THAT(scanner.modules(),
ElementsAre(AllOf(UrlProperty("Example"), VersionProperty(QString{}))));
}
TEST_F(ModuleScanner, Duplicates)
{
scanner.scan(QStringList{QT6_INSTALL_PREFIX});