From b417154cac801820d371b9581cab54ca528af9d6 Mon Sep 17 00:00:00 2001 From: Christian Stenger Date: Thu, 1 Sep 2022 09:27:02 +0200 Subject: [PATCH] Squish: Rename function Better reflects what it does and avoids being used for creation of other test tree items. Change-Id: I48e26c2f76cd1d80b22fb435a34a3fab6f036db2 Reviewed-by: David Schulz --- src/plugins/squish/squishfilehandler.cpp | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/src/plugins/squish/squishfilehandler.cpp b/src/plugins/squish/squishfilehandler.cpp index 6c34a8307d8..1d57ced4896 100644 --- a/src/plugins/squish/squishfilehandler.cpp +++ b/src/plugins/squish/squishfilehandler.cpp @@ -38,9 +38,9 @@ SquishFileHandler *SquishFileHandler::instance() return m_instance; } -SquishTestTreeItem *createTestTreeItem(const QString &name, - const QString &filePath, - const QStringList &cases) +SquishTestTreeItem *createSuiteTreeItem(const QString &name, + const QString &filePath, + const QStringList &cases) { SquishTestTreeItem *item = new SquishTestTreeItem(name, SquishTestTreeItem::SquishSuite); item->setFilePath(filePath); @@ -57,7 +57,7 @@ void SquishFileHandler::modifySuiteItem(const QString &suiteName, const QString &filePath, const QStringList &cases) { - SquishTestTreeItem *item = createTestTreeItem(suiteName, filePath, cases); + SquishTestTreeItem *item = createSuiteTreeItem(suiteName, filePath, cases); // TODO update file watcher m_suites.insert(suiteName, filePath); emit suiteTreeItemModified(item, suiteName); @@ -92,9 +92,9 @@ void SquishFileHandler::openTestSuites() modifySuiteItem(suiteName, suiteConf.absoluteFilePath(), cases); } } else { - SquishTestTreeItem *item = createTestTreeItem(suiteName, - suiteConf.absoluteFilePath(), - cases); + SquishTestTreeItem *item = createSuiteTreeItem(suiteName, + suiteConf.absoluteFilePath(), + cases); // TODO add file watcher m_suites.insert(suiteName, suiteConf.absoluteFilePath()); emit testTreeItemCreated(item);