D6593: Remove messages of already running CTests for UnitTests

Christoph Roick noreply at phabricator.kde.org
Sun Jul 9 23:12:47 UTC 2017


croick created this revision.
Restricted Application added a subscriber: kdevelop-devel.

REVISION SUMMARY
  - starting unit tests for a project or simply "Run All Tests" could start showing up messages of already running instances of CTest
  - after killing or not killing those, KDevelop could finally crash
  
  BUG: 377639
  
  Fixes:
  
  - use job name in NativeAppJob depending on executable to be able to distinguish between different tests
  - after the message box, re-read the list of running jobs
  - actual bugfix: only emit the result once for failing tests

TEST PLAN
  - run kdevelop/kdevelop unit tests at once
  - see what happens after tests are failing

REPOSITORY
  R33 KDevPlatform

BRANCH
  ctest

REVISION DETAIL
  https://phabricator.kde.org/D6593

AFFECTED FILES
  plugins/execute/nativeappjob.cpp
  plugins/execute/nativeappjob.h
  util/executecompositejob.cpp

To: croick, #kdevelop
Cc: apol, kfunk, kdevelop-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20170709/6361aa54/attachment.html>


More information about the KDevelop-devel mailing list