AutoTest: Do not create marks twice

Reporting finished on the future will call onFinished()
due to respective connections.
Removing the additional explicit call to onFinished() to
avoid creating test marks twice.

Change-Id: I3d1a6b0b37bb5152c867a1c8fa488258c9b44268
Reviewed-by: David Schulz <david.schulz@qt.io>

src/plugins/autotest/testrunner.cpp

@@ -249,12 +249,10 @@ void TestRunner::onProcessFinished()
 
     resetInternalPointers();
 
-    if (!m_selectedTests.isEmpty() && !m_fakeFutureInterface->isCanceled()) {
+    if (!m_selectedTests.isEmpty() && !m_fakeFutureInterface->isCanceled())
         scheduleNext();
-    } else {
+    else
         m_fakeFutureInterface->reportFinished();
-        onFinished();
-    }
 }
 
 void TestRunner::resetInternalPointers()