Skip to content

Commit 1c922fa

Browse files
authored
App: Fix Endoscopy Tool Tracking C++ Test Config (nvidia-holoscan#477)
Fixes issue where Endoscopy Tool Tracking C++ tests were relying on the default YAML configuration rather than the expected test configuration. 153bdfd updated the Endoscopy Tool Tracking C++ app CLI: - Under previous behavior a configuration path could be passed as the 0th positional argument - Under new behavior a configuration path may be passed as a keyword `--config` argument. This brings the C++ app CLI into closer alignment with the existing `--data` argument and with the Python app CLI. Prior to this fix, C++ test failures were observed: - The test config file was not properly passed to the app under the new CLI. Instead the default test config was used, which does not define a stop condition and led to test timeout. - The C++ render test failed due to relying on frame output from the test above. Reproduced previous test failures locally and verified tests pass with this fix applied. Signed-off-by: Tom Birdsong <tbirdsong@nvidia.com> Signed-off-by: Rony Rado <rrado@nvidia.com>
1 parent cf06cc5 commit 1c922fa

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

applications/endoscopy_tool_tracking/cpp/CMakeLists.txt

+2-1
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,8 @@ if(BUILD_TESTING)
7373

7474
# Add test
7575
add_test(NAME endoscopy_tool_tracking_cpp_test
76-
COMMAND endoscopy_tool_tracking ${CMAKE_CURRENT_BINARY_DIR}/endoscopy_tool_tracking_testing.yaml
76+
COMMAND endoscopy_tool_tracking
77+
--config ${CMAKE_CURRENT_BINARY_DIR}/endoscopy_tool_tracking_testing.yaml
7778
--data "${HOLOHUB_DATA_DIR}/endoscopy"
7879
WORKING_DIRECTORY ${CMAKE_BINARY_DIR})
7980
set_tests_properties(endoscopy_tool_tracking_cpp_test PROPERTIES

0 commit comments

Comments
 (0)