Skip to content
Snippets Groups Projects
Commit dca4896c authored by Christoph Grüninger's avatar Christoph Grüninger
Browse files

Merge branch 'bugfix/cmake-test-macro' into 'master'

[bugfix] Make test require executable

The test falsely required an executable with the name of the test
instead of the name of the executable. As the comment correctly states
the test needs the executable to run not a test target..

See merge request !45
parents e8189455 b0c49f92
No related branches found
No related tags found
No related merge requests found
...@@ -57,6 +57,6 @@ macro(add_dumux_test dumux_test dumux_test_executable dumux_test_executable_sour ...@@ -57,6 +57,6 @@ macro(add_dumux_test dumux_test dumux_test_executable dumux_test_executable_sour
COMMAND ${dumux_test_args} COMMAND ${dumux_test_args}
SKIP_ON_77) SKIP_ON_77)
# tests always require the executable to run # tests always require the executable to run
set_tests_properties(${dumux_test} PROPERTIES REQUIRED_FILES ${dumux_test}) set_tests_properties(${dumux_test} PROPERTIES REQUIRED_FILES ${dumux_test_executable})
endif() endif()
endmacro() endmacro()
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment