cmake: minor fixes to test targets after prev

- run tests-clang-tidy when building testbins.
- drop redundant build-certs dependency for test targets.
  Already present via testdeps.

Follow-up to aae361242f30eafacf46085ed731b1524bfd3be9 #20708

Closes #20727

diff --git a/tests/CMakeLists.txt b/tests/CMakeLists.txt
index d806e8c4bb625750c2c9e6f2241480e74ae2ae01..7edfaa7ed85a8eb516e734b044df78576d0308dc 100644 (file)
--- a/tests/CMakeLists.txt
+++ b/tests/CMakeLists.txt
@@ -37,14 +37,14 @@ if(CURL_BUILD_EVERYTHING)
   set_target_properties(testbins PROPERTIES EXCLUDE_FROM_ALL FALSE)
 endif()
 
-add_custom_target(testdeps)
-add_dependencies(testdeps "testbins")
-
 if(CURL_CLANG_TIDY)
   add_custom_target(tests-clang-tidy)
-  add_dependencies(testdeps tests-clang-tidy)
+  add_dependencies(testbins tests-clang-tidy)
 endif()
 
+add_custom_target(testdeps)
+add_dependencies(testdeps "testbins")
+
 add_subdirectory(http)
 add_subdirectory(server)
 add_subdirectory(libtest)
@@ -66,7 +66,6 @@ function(curl_add_runtests _targetname _test_flags)
       list(APPEND _depends "${EXE_NAME}")
     endif()
     list(APPEND _depends "testdeps")
-    list(APPEND _depends "build-certs")
   endif()
   set(_setenvs "")
   if(CMAKE_CONFIGURATION_TYPES)