Merge pull request #128 from TGSAI/105_integration_UX

Improve user experience for integration test failures

README.md

@@ -35,6 +35,7 @@ Welcome to **MDIO** - a descriptive format for energy data that is intended to r
 
 ## Optional tools (Integration)
 - Python module xarray version 2024.6.0 *or better*
+- Python module zarr version 2.18.2 *or better*
 
 # Getting Started
 

mdio/acceptance_test.cc

@@ -566,7 +566,7 @@ TEST(Variable, sliceByDimIdx) {
       << voidedSlice.value().dimensions();
 }
 
-TEST(Variable, xarrayCompatibility) {
+TEST(Variable, zarrCompatibility) {
   const char* basePath = std::getenv("PROJECT_BASE_PATH");
   if (!basePath) {
     std::cout << "PROJECT_BASE_PATH environment variable not set. Expecting to "
@@ -1663,6 +1663,10 @@ TEST(Dataset, xarrayCompatible) {
   std::string datasetPath = "./zarrs/acceptance";
   std::string command = "python3 " + srcPath + " " + datasetPath + " False";
   int status = system(command.c_str());
+  if (status == 0xfd00) {  // 0xfd from Python is 0xfd00 in C++
+    GTEST_SKIP()
+        << "Xarray compatibility skipped due to import error for xarray";
+  }
   ASSERT_TRUE(status == 0)
       << "xarray compatibility test failed without consolidated "
          "metadata\n\tThere was some expected output above...";

mdio/regression_tests/xarray_compatibility_test.py

@@ -1,6 +1,10 @@
 import argparse
 import sys
-import xarray as xr
+try:
+    import xarray as xr
+except ImportError:
+    print("Failed to import xarray.")
+    sys.exit(0xfd) # 64768
 
 
 def test_xarray_dataset(file_path, consolidated_metadata):

mdio/regression_tests/zarr_compatibility.py

@@ -14,7 +14,11 @@
 
 import argparse
 import sys
-import zarr
+try:
+    import zarr
+except ImportError:
+    print("Failed to import zarr.")
+    sys.exit(0xfd)
 
 
 def test_zarr_dataset(file_path):