Enable CI/CD tests

TileDB-Inc · May 20, 2024 · cff8080 · cff8080
1 parent 51723fd
commit cff8080
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 7 deletions.
diff --git a/.github/workflows/build-wheels.yml b/.github/workflows/build-wheels.yml
@@ -49,6 +49,13 @@ jobs:
           CIBW_ARCHS: all
           CIBW_PRERELEASE_PYTHONS: True
           CIBW_BUILD: ${{ matrix.python }}-${{ matrix.buildplat[1] }}
+          CIBW_TEST_REQUIRES: pytest
+          # This command should be just `pytest`, however, our tests require test data and those are only in the
+          # original {project} folder. The cibuildwheel logic unfortunately creates an empty folder for tests, changes
+          # directory there and so our tests will search for data folder in a completely different working directory.
+          CIBW_TEST_COMMAND: "pytest"
+          CIBW_ENVIRONMENT: TILEDB_REST_TOKEN=${{ secrets.TILEDB_CLOUD_HELPER_VAR }}
+          SYSTEM_VERSION_COMPAT: ${{ startsWith(matrix.os, 'macos-') }}
         with:
           output-dir: wheelhouse
 

diff --git a/tiledb/CMakeLists.txt b/tiledb/CMakeLists.txt
@@ -61,14 +61,14 @@ install(TARGETS main libtiledb DESTINATION tiledb)
 if(TILEDB_DOWNLOADED)
     message(STATUS "Adding libtiledb.so into install group")
 
-    install(IMPORTED_RUNTIME_ARTIFACTS TileDB::tiledb_shared DESTINATION tiledb/lib)
+    install(IMPORTED_RUNTIME_ARTIFACTS TileDB::tiledb_shared DESTINATION tiledb)
 
     if (APPLE)
-        set_target_properties(main PROPERTIES INSTALL_RPATH "@loader_path/lib")
-        set_target_properties(libtiledb PROPERTIES INSTALL_RPATH "@loader_path/lib")
+        set_target_properties(main PROPERTIES INSTALL_RPATH "@loader_path")
+        set_target_properties(libtiledb PROPERTIES INSTALL_RPATH "@loader_path")
     elseif(UNIX)
-        set_target_properties(main PROPERTIES INSTALL_RPATH "\$ORIGIN/lib")
-        set_target_properties(libtiledb PROPERTIES INSTALL_RPATH "\$ORIGIN/lib")
+        set_target_properties(main PROPERTIES INSTALL_RPATH "\$ORIGIN")
+        set_target_properties(libtiledb PROPERTIES INSTALL_RPATH "\$ORIGIN")
     endif()
 endif()
 

diff --git a/tiledb/cc/CMakeLists.txt b/tiledb/cc/CMakeLists.txt
@@ -37,8 +37,8 @@ install(TARGETS cc DESTINATION tiledb)
 
 if(TILEDB_DOWNLOADED)
     if (APPLE)
-    set_target_properties(cc PROPERTIES INSTALL_RPATH "@loader_path/lib")
+    set_target_properties(cc PROPERTIES INSTALL_RPATH "@loader_path")
     elseif(UNIX)
-    set_target_properties(cc PROPERTIES INSTALL_RPATH "\$ORIGIN/lib")
+    set_target_properties(cc PROPERTIES INSTALL_RPATH "\$ORIGIN")
     endif()
 endif()