diff --git a/src/plugins/autotest/testframeworkmanager.cpp b/src/plugins/autotest/testframeworkmanager.cpp index b9b61b79d11..0c9ffeef616 100644 --- a/src/plugins/autotest/testframeworkmanager.cpp +++ b/src/plugins/autotest/testframeworkmanager.cpp @@ -25,22 +25,20 @@ TestFrameworkManager::~TestFrameworkManager() s_instance = nullptr; } -bool TestFrameworkManager::registerTestFramework(ITestFramework *framework) +void TestFrameworkManager::registerTestFramework(ITestFramework *framework) { - QTC_ASSERT(framework, return false); - QTC_ASSERT(!m_registeredFrameworks.contains(framework), return false); + QTC_ASSERT(framework, return); + QTC_ASSERT(!m_registeredFrameworks.contains(framework), return); // TODO check for unique priority before registering m_registeredFrameworks.append(framework); Utils::sort(m_registeredFrameworks, &ITestFramework::priority); - return true; } -bool TestFrameworkManager::registerTestTool(ITestTool *testTool) +void TestFrameworkManager::registerTestTool(ITestTool *testTool) { - QTC_ASSERT(testTool, return false); - QTC_ASSERT(!m_registeredTestTools.contains(testTool), return false); + QTC_ASSERT(testTool, return); + QTC_ASSERT(!m_registeredTestTools.contains(testTool), return); m_registeredTestTools.append(testTool); - return true; } void TestFrameworkManager::activateFrameworksAndToolsFromSettings() diff --git a/src/plugins/autotest/testframeworkmanager.h b/src/plugins/autotest/testframeworkmanager.h index e42001dc2d4..444ea3fc2fe 100644 --- a/src/plugins/autotest/testframeworkmanager.h +++ b/src/plugins/autotest/testframeworkmanager.h @@ -14,8 +14,8 @@ public: TestFrameworkManager(); ~TestFrameworkManager(); - bool registerTestFramework(ITestFramework *framework); - bool registerTestTool(ITestTool *testTool); + void registerTestFramework(ITestFramework *framework); + void registerTestTool(ITestTool *testTool); void synchronizeSettings(); static ITestFramework *frameworkForId(Utils::Id frameworkId);