test to guard against cache

Author: jathu

tools/cmake/common/__init__.py

@@ -74,7 +74,6 @@ class _CacheValue:
 
 
 # Get the key/value pair listed in a CMakeCache.txt file.
-@cache
 def _list_cmake_cache(cache_path: str) -> Dict[str, _CacheValue]:
     result = {}
     with open(cache_path, "r") as cache_file:

tools/cmake/common/preset.cmake

@@ -76,7 +76,7 @@ macro(define_overridable_option NAME DESCRIPTION VALUE_TYPE DEFAULT_VALUE)
     message(FATAL_ERROR "Invalid option (${NAME}) value type '${VALUE_TYPE}', must be either STRING or BOOL")
   endif()
 
-  if(DEFINED ${NAME})
+  if(DEFINED ${NAME} AND NOT DEFINED CACHE{${NAME}})
     set(${NAME} ${${NAME}} CACHE ${VALUE_TYPE} ${DESCRIPTION} FORCE)
   else()
     set(${NAME} ${DEFAULT_VALUE} CACHE ${VALUE_TYPE} ${DESCRIPTION})

tools/cmake/common/preset_test.py

@@ -201,12 +201,11 @@ def test_define_overridable_option_set_override_after_with_cache(self):
         # Setting the value after should not affect the cache.
         self.assert_cmake_cache("EXECUTORCH_TEST_MESSAGE", "default value", "STRING")
 
-    def test_define_overridable_option_cli_override_with_set_override(self):
+    def test_define_overridable_option_override_existing_cache(self):
         _cmake_lists_txt = """
             cmake_minimum_required(VERSION 3.24)
             project(test_preset)
             include(${PROJECT_SOURCE_DIR}/preset.cmake)
-            set(EXECUTORCH_TEST_MESSAGE "set value")
             add_subdirectory(example)
         """
         _example_cmake_lists_txt = """
@@ -220,9 +219,11 @@ def test_define_overridable_option_cli_override_with_set_override(self):
                 },
             }
         )
+        self.run_cmake()
+        self.assert_cmake_cache("EXECUTORCH_TEST_MESSAGE", "default value", "STRING")
+
         self.run_cmake(cmake_args=["-DEXECUTORCH_TEST_MESSAGE='cli value'"])
-        # If an option is set through cmake, it should NOT be overridable from the CLI.
-        self.assert_cmake_cache("EXECUTORCH_TEST_MESSAGE", "set value", "STRING")
+        self.assert_cmake_cache("EXECUTORCH_TEST_MESSAGE", "cli value", "STRING")
 
     def test_set_overridable_option_before(self):
         _cmake_lists_txt = """