Details
Diff Detail
- Repository
- R32 KDevelop
- Branch
- master
- Lint
No Linters Available - Unit
No Unit Test Coverage
This is the first step on the way to make the tests aware of their working directory (other patches are finished and will be uploaded soon). It is easier to just pass all things down instead of having expectedFail, workindDir, and who knows what comes in the future all in separate variables down the stack.
- pass all test properties to CTestSuite
- CTest: parse the test properties from CTestTestfile.cmake
- CTest: honor the WORKING_DIRECTORY property of tests
- CTest: use the test name as name instead of the executable name
In other words, he's (also) trying to fix https://bugs.kde.org/show_bug.cgi?id=349378
Great work!
| projectmanagers/cmake/tests/test_ctestfindsuites.cpp | ||
|---|---|---|
| 75 ↗ | (On Diff #16604) | Note: This should probably be turned into a data-driven test in future. Too many branches checking for individual test rows in the body now. |
Note: This should probably be turned into a data-driven test in future. Too many branches checking for individual test rows in the body now.
Thought so too. +1
I second the comment about the data driven test, I was thinking the same. Maybe if I find time...