add macos to build matrix (skip tests using docker)

Author: zfarrell

.github/workflows/ci.yml

@@ -32,7 +32,7 @@ jobs:
       matrix:
         os:
           - ubuntu-latest
-#          - macos-latest
+          - macos-latest
         rust: [stable]
     steps:
       - uses: actions/checkout@v4
@@ -71,7 +71,12 @@ jobs:
         run: cargo clippy --all-targets --all-features --no-deps -- -D warnings
 
       - name: Run tests
-        run: cargo test --all-features
+        run: |
+          if [[ "$RUNNER_OS" == "macOS" ]]; then
+            cargo test --features skip-tests-with-docker
+          else
+            cargo test
+          fi
 
       - name: Check documentation
         run: cargo doc --no-deps --all-features

Cargo.toml

@@ -23,3 +23,7 @@ tempfile = "3.14"
 anyhow = "1.0"
 aws-sdk-s3 = "1.75"
 aws-credential-types = "1.2"
+
+[features]
+# Allow skipping tests that use docker (in CI, macos doesn't support docker).
+skip-tests-with-docker = []

tests/object_store_integration_test.rs

@@ -200,6 +200,7 @@ async fn create_s3_test_catalog(
 }
 
 #[tokio::test]
+#[cfg_attr(feature = "skip-docker-tests", ignore)]
 async fn test_minio_object_store_integration() -> anyhow::Result<()> {
     // Tests DataFusion-DuckLake reading from S3/MinIO with DuckDB-created data