From 2f1018ab24213f6aac7a15eb474082df298a0f80 Mon Sep 17 00:00:00 2001 From: Christian Stenger Date: Mon, 14 Dec 2015 07:23:18 +0100 Subject: [PATCH] Fix missing MessageTestCaseEnd for Qt4 based tests Qt4 based tests have no duration, so this message had been omitted accidently. Change-Id: I11b03234c8f7fd022e951728b70ae186091c15d9 Reviewed-by: Niels Weber --- plugins/autotest/testoutputreader.cpp | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/plugins/autotest/testoutputreader.cpp b/plugins/autotest/testoutputreader.cpp index a6aa10d3b34..facf7353b62 100644 --- a/plugins/autotest/testoutputreader.cpp +++ b/plugins/autotest/testoutputreader.cpp @@ -282,10 +282,12 @@ void TestOutputReader::processOutput() testResultCreated(testResult); } emit increaseProgress(); - } else if (currentTag == QStringLiteral("TestCase") && !duration.isEmpty()) { + } else if (currentTag == QStringLiteral("TestCase")) { auto testResult = new TestResult(className); testResult->setResult(Result::MessageTestCaseEnd); - testResult->setDescription(tr("Test execution took %1 ms.").arg(duration)); + testResult->setDescription( + duration.isEmpty() ? tr("Test finished.") + : tr("Test execution took %1 ms.").arg(duration)); testResultCreated(testResult); } else if (validEndTags.contains(currentTag.toString())) { auto testResult = new TestResult(className);