Fix legacy tests (AIDASoft#510)

Ananya2003Gupta · Nov 6, 2023 · 9ad574a · 9ad574a
1 parent 01e1d05
commit 9ad574a
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/tests/root_io/CMakeLists.txt b/tests/root_io/CMakeLists.txt
@@ -59,8 +59,7 @@ target_link_libraries(read-legacy-files-root PRIVATE TestDataModel TestDataModel
 macro(ADD_PODIO_LEGACY_TEST version base_test input_file)
   ExternalData_Add_Test(legacy_test_cases
     NAME ${base_test}_${version}
-    COMMAND ${base_test} ${CMAKE_CURRENT_BINARY_DIR}/../input_files/${input_file}
-    DATA{${PROJECT_SOURCE_DIR}/tests/input_files/${input_file}}
+    COMMAND ${base_test} DATA{${PROJECT_SOURCE_DIR}/tests/input_files/${input_file}}
   )
   set_property(TEST ${base_test}_${version} PROPERTY ENVIRONMENT
     LD_LIBRARY_PATH=${PROJECT_BINARY_DIR}/tests:${PROJECT_BINARY_DIR}/src:$ENV{LD_LIBRARY_PATH}

diff --git a/tests/root_io/read_frame_root.cpp b/tests/root_io/read_frame_root.cpp
@@ -12,6 +12,10 @@ int main(int argc, char* argv[]) {
   if (argc == 2) {
     inputFile = argv[1];
     assertBuildVersion = false;
+  } else if (argc > 2) {
+    std::cout << "Wrong number of arguments" << std::endl;
+    std::cout << "Usage: " << argv[0] << " FILE" << std::endl;
+    return 1;
   }
 
   return read_frames<podio::ROOTFrameReader>(inputFile, assertBuildVersion) +