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