D6668: pass all test properties to CTestSuite

Rolf Eike Beer noreply at phabricator.kde.org
Thu Jul 13 04:58:08 UTC 2017


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

REVISION SUMMARY
  This way this and CTestRunJob can do any adaptions to the properties without
  needing more and more members.
  
  CTest: parse the test properties from CTestTestfile.cmake
  
  Otherwise e.g. the WILL_FAIL check in ctestutils.cpp will never match as the
  property hash is always empty.
  
  CTest: honor the WORKING_DIRECTORY property of tests
  
  BUG:349378
  FIXED-IN:5.2.0
  
  CTest: use the test name as name instead of the executable name
  
  Now one only needs to get rid of the first argument that is sometimes appended.

REPOSITORY
  R32 KDevelop

BRANCH
  master

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

AFFECTED FILES
  projectmanagers/cmake/cmakeutils.cpp
  projectmanagers/cmake/testing/ctestrunjob.cpp
  projectmanagers/cmake/testing/ctestrunjob.h
  projectmanagers/cmake/testing/ctestsuite.cpp
  projectmanagers/cmake/testing/ctestsuite.h
  projectmanagers/cmake/testing/ctestutils.cpp
  projectmanagers/cmake/tests/manual/unit_tests/CMakeLists.txt
  projectmanagers/cmake/tests/manual/unit_tests/five/CMakeLists.txt
  projectmanagers/cmake/tests/manual/unit_tests/math_test.cpp
  projectmanagers/cmake/tests/test_ctestfindsuites.cpp

To: dakon
Cc: kdevelop-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20170713/b141ff67/attachment.html>


More information about the KDevelop-devel mailing list