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

AutoTest: Fix placing of summary messages

Browse Source 
...for running CTest based tests. These where placed below the
last project although they are meant globally.
Fixes filtering issue of the results where these information
interfered with the test execution items and broke the logic
to determine whether an item is to be shown or not.

Fixes: QTCREATORBUG-32452
Change-Id: Ie8aa3e711398ca92c4ade0a65b082f97605e17c1
Reviewed-by: David Schulz <david.schulz@qt.io>
This commit is contained in:
Christian Stenger
2025-02-05 12:22:36 +01:00
parent c7b38a86fc
commit ed0a77571d
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/plugins/autotest/ctest/ctestoutputreader.cpp
View File

@@ -124,7 +124,7 @@ void CTestOutputReader::processOutputLine(const QByteArray &outputLine)
} else if (ExactMatch match = summary.match(line)) {
if (!m_testName.isEmpty())
sendCompleteInformation();
TestResult testResult = createDefaultResult();
CTestResult testResult(id(), {}, {});
testResult.setResult(ResultType::MessageInfo);
testResult.setDescription(match.captured());
reportResult(testResult);
@@ -135,8 +135,8 @@ void CTestOutputReader::processOutputLine(const QByteArray &outputLine)
} else if (ExactMatch match = summaryTime.match(line)) {
if (!m_testName.isEmpty()) // possible?
sendCompleteInformation();
TestResult testResult = createDefaultResult();
testResult.setResult(ResultType::TestEnd);
CTestResult testResult(id(), {}, {});
testResult.setResult(ResultType::MessageInfo);
testResult.setDescription(match.captured());
m_executionDuration = qRound(match.captured(1).toDouble() * 1000.);
reportResult(testResult);