diff options
author | Christian Stenger <christian.stenger@qt.io> | 2017-06-15 10:43:44 +0200 |
---|---|---|
committer | Christian Stenger <christian.stenger@qt.io> | 2017-06-19 06:59:08 +0000 |
commit | ea86ff5eb0aa63d61729366184c00d9612dcfa97 (patch) | |
tree | e208d2ba60bc4baacd78368a97cbc21c0a9d7cc2 /src/plugins/autotest/autotestplugin.cpp | |
parent | 43a7fa28b1b0256d2d03344c31f4a1ca946cde06 (diff) |
AutoTest: Avoid triggering run tests while building
Disable respective actions while a build is running to avoid strange
side-effects or race conditions.
Change-Id: Ifc8d1c9fc9fd4d705cabd148b145708e23029dba
Reviewed-by: David Schulz <david.schulz@qt.io>
Diffstat (limited to 'src/plugins/autotest/autotestplugin.cpp')
-rw-r--r-- | src/plugins/autotest/autotestplugin.cpp | 14 |
1 files changed, 11 insertions, 3 deletions
diff --git a/src/plugins/autotest/autotestplugin.cpp b/src/plugins/autotest/autotestplugin.cpp index 571af4ba08..3d00400ccd 100644 --- a/src/plugins/autotest/autotestplugin.cpp +++ b/src/plugins/autotest/autotestplugin.cpp @@ -46,8 +46,8 @@ #include <coreplugin/actionmanager/command.h> #include <coreplugin/actionmanager/actioncontainer.h> #include <coreplugin/coreconstants.h> - #include <extensionsystem/pluginmanager.h> +#include <projectexplorer/buildmanager.h> #include <QAction> #include <QMessageBox> @@ -102,6 +102,7 @@ void AutotestPlugin::initializeMenuEntries() command->setDefaultKeySequence(QKeySequence(tr("Alt+Shift+T,Alt+A"))); connect(action, &QAction::triggered, this, &AutotestPlugin::onRunAllTriggered); + action->setEnabled(false); menu->addAction(command); action = new QAction(tr("&Run Selected Tests"), this); @@ -109,6 +110,7 @@ void AutotestPlugin::initializeMenuEntries() command->setDefaultKeySequence(QKeySequence(tr("Alt+Shift+T,Alt+R"))); connect(action, &QAction::triggered, this, &AutotestPlugin::onRunSelectedTriggered); + action->setEnabled(false); menu->addAction(command); action = new QAction(tr("Re&scan Tests"), this); @@ -121,7 +123,12 @@ void AutotestPlugin::initializeMenuEntries() ActionContainer *toolsMenu = ActionManager::actionContainer(Core::Constants::M_TOOLS); toolsMenu->addMenu(menu); - connect(toolsMenu->menu(), &QMenu::aboutToShow, + using namespace ProjectExplorer; + connect(BuildManager::instance(), &BuildManager::buildStateChanged, + this, &AutotestPlugin::updateMenuItemsEnabledState); + connect(BuildManager::instance(), &BuildManager::buildQueueFinished, + this, &AutotestPlugin::updateMenuItemsEnabledState); + connect(TestTreeModel::instance(), &TestTreeModel::testTreeModelChanged, this, &AutotestPlugin::updateMenuItemsEnabledState); } @@ -176,7 +183,8 @@ void AutotestPlugin::onRunSelectedTriggered() void AutotestPlugin::updateMenuItemsEnabledState() { - const bool enabled = !TestRunner::instance()->isTestRunning() + const bool enabled = !ProjectExplorer::BuildManager::isBuilding() + && !TestRunner::instance()->isTestRunning() && TestTreeModel::instance()->parser()->state() == TestCodeParser::Idle; const bool hasTests = TestTreeModel::instance()->hasTests(); |