diff options
author | Christian Stenger <christian.stenger@qt.io> | 2019-06-03 13:17:55 +0200 |
---|---|---|
committer | Christian Stenger <christian.stenger@qt.io> | 2019-06-07 06:40:52 +0000 |
commit | c23fe4f403734b5285a7bbcdc85e41b4ffae7f81 (patch) | |
tree | bf4f202655dcfbf7e4cd12f77a26d9ee55dec576 /src/plugins/autotest | |
parent | 6b124b580e857f7767f817700a42ad41f364a92f (diff) |
AutoTest: Remove wrong todos
These todos have been resolved already or turned out
to be wrong / not necessary.
Change-Id: Iefa46bd8144834a60fbb9d523ac73d3372d4a6b6
Reviewed-by: David Schulz <david.schulz@qt.io>
Diffstat (limited to 'src/plugins/autotest')
-rw-r--r-- | src/plugins/autotest/boost/boosttestoutputreader.cpp | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/src/plugins/autotest/boost/boosttestoutputreader.cpp b/src/plugins/autotest/boost/boosttestoutputreader.cpp index 58e54160e57..3ae5756d3a5 100644 --- a/src/plugins/autotest/boost/boosttestoutputreader.cpp +++ b/src/plugins/autotest/boost/boosttestoutputreader.cpp @@ -107,7 +107,7 @@ void BoostTestOutputReader::sendCompleteInformation() if (m_lineNumber) { result->setLine(m_lineNumber); result->setFileName(m_fileName); - } // else TODO + } result->setDescription(m_description); result->setResult(m_result); @@ -406,7 +406,6 @@ TestResultPtr BoostTestOutputReader::createDefaultResult() const result->setTestSuite(m_currentSuite); result->setTestCase(m_currentTest); - // TODO find corresponding TestTreeItem and set filename/line return TestResultPtr(result); } @@ -418,7 +417,6 @@ void BoostTestOutputReader::onFinished(int exitCode, QProcess::ExitStatus /*exit // boost::exit_success (0), boost::exit_test_failure (201) // or boost::exit_exception_failure (200) // be graceful and do not add a fatal for exit_test_failure - // but exit code 0 can be forced with an option - what todo in that case? if (m_logLevel == LogLevel::Nothing && m_reportLevel == ReportLevel::No) { switch (exitCode) { case 0: |