Don't use absolute path when loading test data

snowflakedb · Oct 30, 2024 · 176db13 · 176db13
1 parent ab5d1aa
commit 176db13
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 19 deletions.
diff --git a/tests/CMakeLists.txt b/tests/CMakeLists.txt
@@ -121,6 +121,28 @@ set(AWS_ALL_LIBS
         ${AWS_C_SDKUTILS_LIB}
     )
 
+SET(TEST_DATA
+        TestOrcFile.test1.orc
+        brotli_sample.txt.br
+        decode
+        decode2
+        encode
+        encode2
+        large_file.csv.gz
+        nation.impala.parquet
+        one_byte.csv
+        public_domain_books.csv
+        small_file.csv
+        small_file.csv.bz2
+        small_file.csv.gz
+        small_file.csv.zst
+        zero_byte.csv
+)
+
+FOREACH (TD ${TEST_DATA})
+    configure_file("data/${TD}" "data/${TD}" COPYONLY)
+ENDFOREACH ()
+
 message("CLOUD_PROVIDER is set to " $ENV{CLOUD_PROVIDER})
 #Due to docker issue the tests are failing on linux.
 if (APPLE OR WIN32 OR CLIENT_CODE_COVERAGE)

diff --git a/tests/utils/TestSetup.cpp b/tests/utils/TestSetup.cpp
@@ -7,23 +7,5 @@
 
 std::string TestSetup::getDataDir()
 {
-#ifdef _WIN32
-  char appveyorBuildDir[500];
-  DWORD dwRet = 0;
-  dwRet= GetEnvironmentVariable("APPVEYOR_BUILD_FOLDER", appveyorBuildDir, sizeof(appveyorBuildDir));
-  if (dwRet != 0)
-  {
-    return std::string(appveyorBuildDir) + PATH_SEP + "tests" + PATH_SEP + "data" + PATH_SEP;
-  }
-#else
-  const char * travisBuildDir = getenv("TRAVIS_BUILD_DIR");
-  if (travisBuildDir)
-  {
-    return std::string(travisBuildDir) + PATH_SEP + "tests" + PATH_SEP + "data" + PATH_SEP;
-  }
-#endif
-
-  const std::string current_file = __FILE__;
-  std::string utilDir = current_file.substr(0, current_file.find_last_of(PATH_SEP));
-  return utilDir + PATH_SEP + ".." + PATH_SEP + "data" + PATH_SEP;
+  return std::string("data") + PATH_SEP;
 }