Fix serialization test and add command line option (#2052)

Author: kounelisagis

pyproject.toml

@@ -83,6 +83,7 @@ TILEDB_PATH = {env="TILEDB_PATH"}
 TILEDB_VERSION = {env="TILEDB_VERSION"}
 TILEDB_HASH = {env="TILEDB_HASH"}
 TILEDB_REMOVE_DEPRECATIONS = "ON"
+TILEDB_SERIALIZATION = "OFF"
 
 [tool.pytest.ini_options]
 python_classes = "*Test*"

setup.py

@@ -9,6 +9,7 @@ def main():
     parser.add_argument("--tiledb", type=str, required=False)
     parser.add_argument("--debug", action="store_true")
     parser.add_argument("--enable-deprecations", action="store_true", required=False)
+    parser.add_argument("--enable-serialization", action="store_true", required=False)
     parser.add_argument("-v", action="store_true")
     args = parser.parse_args()
 
@@ -33,6 +34,9 @@ def main():
     if args.enable_deprecations:
         cmd.append(f"-Cskbuild.cmake.define.TILEDB_REMOVE_DEPRECATIONS=OFF")
 
+    if args.enable_serialization:
+        cmd.append(f"-Cskbuild.cmake.define.TILEDB_SERIALIZATION=ON")
+
     if args.v:
         cmd.append("-v")
 

tiledb/CMakeLists.txt

@@ -56,6 +56,14 @@ target_compile_features(
     cxx_std_20
 )
 
+if (TILEDB_SERIALIZATION)
+    target_compile_definitions(
+        main
+        PRIVATE
+        TILEDB_SERIALIZATION
+    )
+endif()
+
 install(TARGETS main libtiledb DESTINATION tiledb)
 
 if(TILEDB_DOWNLOADED)

tiledb/cc/CMakeLists.txt

@@ -42,6 +42,14 @@ if (TILEDB_REMOVE_DEPRECATIONS)
     )
 endif()
 
+if (TILEDB_SERIALIZATION)
+    target_compile_definitions(
+        cc
+        PRIVATE
+        TILEDB_SERIALIZATION
+    )
+endif()
+
 install(TARGETS cc DESTINATION tiledb)
 
 if(TILEDB_DOWNLOADED)

tiledb/tests/test_serialization.py

@@ -5,9 +5,10 @@
 
 from .common import DiskTestCase
 
-ser_test = pytest.importorskip(
-    "tiledb.main.test_serialization", reason="Serialization not enabled."
-)
+try:
+    from tiledb.main import test_serialization as ser_test
+except ImportError:
+    pytest.skip("Serialization not enabled.", allow_module_level=True)
 
 
 class SerializationTest(DiskTestCase):