From f2580f77fba4bae105a21af2ba5207a021f6e7bd Mon Sep 17 00:00:00 2001
From: Matt Shin <matthew.shin@metoffice.gov.uk>
Date: Mon, 22 Sep 2025 15:16:02 +0100
Subject: [PATCH] cmake configure test only when BUILD_TESTING=ON

Determine whether to build tests by following the CTest module
BUILD_TESTING option.
---
 CMakeLists.txt | 29 +++++++++++++++++++----------
 1 file changed, 19 insertions(+), 10 deletions(-)

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 89451cf2..8248a2bc 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -100,10 +100,26 @@ include( opsinputs_compiler_flags )
 add_subdirectory( deps )
 add_subdirectory( etc )
 add_subdirectory( src )
-add_subdirectory( test )
 
-if(ECBUILD_INSTALL_FORTRAN_MODULES)
-  install(DIRECTORY ${CMAKE_Fortran_MODULE_DIRECTORY}/${CMAKE_CFG_INTDIR} DESTINATION ${INSTALL_INCLUDE_DIR} )
+if( ECBUILD_INSTALL_FORTRAN_MODULES )
+  install(
+    DIRECTORY ${CMAKE_Fortran_MODULE_DIRECTORY}/${CMAKE_CFG_INTDIR}
+    DESTINATION ${INSTALL_INCLUDE_DIR}
+  )
+endif()
+
+if( BUILD_TESTING )
+  add_subdirectory( test )
+  ecbuild_add_test(
+    TARGET opsinputs_coding_norms
+    TYPE SCRIPT
+    COMMAND ${oops_BINDIR}/oops_cpplint.py
+    ARGS
+      --quiet
+      --recursive
+      ${CMAKE_CURRENT_SOURCE_DIR}/src
+      ${CMAKE_CURRENT_SOURCE_DIR}/test
+    WORKING_DIRECTORY ${CMAKE_BINARY_DIR}/bin )
 endif()
 
 ################################################################################
@@ -115,10 +131,3 @@ ecbuild_install_project( NAME opsinputs )
 
 # print the summary of the configuration
 ecbuild_print_summary()
-
-ecbuild_add_test( TARGET opsinputs_coding_norms
-                  TYPE SCRIPT
-                  COMMAND ${oops_BINDIR}/oops_cpplint.py
-                  ARGS --quiet --recursive ${CMAKE_CURRENT_SOURCE_DIR}/src ${CMAKE_CURRENT_SOURCE_DIR}/test
-                  WORKING_DIRECTORY ${CMAKE_BINARY_DIR}/bin )
-