Author: Jonas Devlieghere Date: 2020-06-12T14:18:27-07:00 New Revision: 8f9eb7035344715194d73272b2f15205ea362f58
URL: https://github.com/llvm/llvm-project/commit/8f9eb7035344715194d73272b2f15205ea362f58 DIFF: https://github.com/llvm/llvm-project/commit/8f9eb7035344715194d73272b2f15205ea362f58.diff LOG: [lldb/Test] Fix unit test suffixes and add a CMake error. Lit will only look for unittest targets ending in `Tests`. Add an error when the target doesn't have the `Tests` suffix. Added: Modified: lldb/unittests/CMakeLists.txt lldb/unittests/Platform/Android/CMakeLists.txt lldb/unittests/Process/Linux/CMakeLists.txt lldb/unittests/Process/POSIX/CMakeLists.txt Removed: ################################################################################ diff --git a/lldb/unittests/CMakeLists.txt b/lldb/unittests/CMakeLists.txt index 52f5c964fcee..6422b726ca69 100644 --- a/lldb/unittests/CMakeLists.txt +++ b/lldb/unittests/CMakeLists.txt @@ -33,6 +33,10 @@ function(add_lldb_unittest test_name) "LINK_LIBS;LINK_COMPONENTS" ${ARGN}) + if (NOT ${test_name} MATCHES "Tests$") + message(FATAL_ERROR "Unit test name must end with 'Tests' for lit to find it.") + endif() + list(APPEND LLVM_LINK_COMPONENTS ${ARG_LINK_COMPONENTS}) add_unittest(LLDBUnitTests diff --git a/lldb/unittests/Platform/Android/CMakeLists.txt b/lldb/unittests/Platform/Android/CMakeLists.txt index 3de2a2d12016..489fdab7036c 100644 --- a/lldb/unittests/Platform/Android/CMakeLists.txt +++ b/lldb/unittests/Platform/Android/CMakeLists.txt @@ -1,6 +1,6 @@ include_directories(${LLDB_SOURCE_DIR}/source/Plugins/Platform/Android) -add_lldb_unittest(AdbClientTest +add_lldb_unittest(AdbClientTests AdbClientTest.cpp LINK_LIBS diff --git a/lldb/unittests/Process/Linux/CMakeLists.txt b/lldb/unittests/Process/Linux/CMakeLists.txt index 3b55b5c84305..31e9a57a4e46 100644 --- a/lldb/unittests/Process/Linux/CMakeLists.txt +++ b/lldb/unittests/Process/Linux/CMakeLists.txt @@ -1,6 +1,6 @@ include_directories(${LLDB_SOURCE_DIR}/source/Plugins/Process/Linux) -add_lldb_unittest(ProcessorTraceTest +add_lldb_unittest(ProcessorTraceTests ProcessorTraceTest.cpp LINK_LIBS diff --git a/lldb/unittests/Process/POSIX/CMakeLists.txt b/lldb/unittests/Process/POSIX/CMakeLists.txt index 53318d908da7..eabfb58d46fc 100644 --- a/lldb/unittests/Process/POSIX/CMakeLists.txt +++ b/lldb/unittests/Process/POSIX/CMakeLists.txt @@ -1,6 +1,6 @@ include_directories(${LLDB_SOURCE_DIR}/source/Plugins/Process/POSIX) -add_lldb_unittest(ProcessPOSIXTest +add_lldb_unittest(ProcessPOSIXTests NativeProcessELFTest.cpp LINK_LIBS _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits