diff options
author | Chuck Atkins <chuck.atkins@kitware.com> | 2019-09-16 16:46:33 (GMT) |
---|---|---|
committer | Chuck Atkins <chuck.atkins@kitware.com> | 2019-09-16 17:10:40 (GMT) |
commit | 0301a76bb464fc3231193c4f1c957327e8258d44 (patch) | |
tree | 38f591b33001d8823cbeaed50a4f154534822772 /Tests/FindEnvModules/EnvModules.cmake | |
parent | 1ac4e0ef1b29affc9e4f2cd86c4fc8c2252f2ab2 (diff) | |
download | CMake-0301a76bb464fc3231193c4f1c957327e8258d44.zip CMake-0301a76bb464fc3231193c4f1c957327e8258d44.tar.gz CMake-0301a76bb464fc3231193c4f1c957327e8258d44.tar.bz2 |
FindEnvMod: Remove unreliable test case
There's no way to know that loading an arbitrary module doesn't pull in
extra dependent modules so the test case asserting that exactly one module
was loaded isn't valid.
Diffstat (limited to 'Tests/FindEnvModules/EnvModules.cmake')
-rw-r--r-- | Tests/FindEnvModules/EnvModules.cmake | 16 |
1 files changed, 7 insertions, 9 deletions
diff --git a/Tests/FindEnvModules/EnvModules.cmake b/Tests/FindEnvModules/EnvModules.cmake index 0c81bf2..21b0042 100644 --- a/Tests/FindEnvModules/EnvModules.cmake +++ b/Tests/FindEnvModules/EnvModules.cmake @@ -18,18 +18,16 @@ if(avail_mods) message("module list") env_module_list(loaded_mods) + set(mod0_found FALSE) foreach(mod IN LISTS loaded_mods) message(" ${mod}") + if(NOT mod0_found AND mod MATCHES "^${mod0}") + set(mod0_found ${mod}) + endif() endforeach() - list(LENGTH loaded_mods num_loaded_mods) - message("Number of modules loaded: ${num_loaded_mods}") - if(NOT num_loaded_mods EQUAL 1) - message(FATAL_ERROR "Exactly 1 module should be loaded. Found ${num_loaded_mods}") - endif() - - list(GET loaded_mods 0 mod0_actual) - if(NOT (mod0_actual MATCHES "^${mod0}")) - message(FATAL_ERROR "Loaded module does not match ${mod0}. Actual: ${mod0_actual}") + if(NOT mod0_found) + message(FATAL_ERROR "Requested module ${mod0} not found in loaded modules") endif() + message("module ${mod0} found loaded as ${mod0_found}") endif() |