From f16dcaf36154b329cc57e57267467bb5ab022f7f Mon Sep 17 00:00:00 2001 From: Deepak Cherian Date: Fri, 25 Oct 2024 11:48:46 -0600 Subject: [PATCH 1/2] Unxfail tests xref #348 --- .../tests/test_zarr/test_store/test_icechunk_store.py | 8 -------- 1 file changed, 8 deletions(-) diff --git a/icechunk-python/tests/test_zarr/test_store/test_icechunk_store.py b/icechunk-python/tests/test_zarr/test_store/test_icechunk_store.py index 1540a1be..74c5aea1 100644 --- a/icechunk-python/tests/test_zarr/test_store/test_icechunk_store.py +++ b/icechunk-python/tests/test_zarr/test_store/test_icechunk_store.py @@ -27,10 +27,6 @@ class TestIcechunkStore(StoreTests[IcechunkStore, cpu.Buffer]): def test_store_eq(self, store: IcechunkStore, store_kwargs: dict[str, Any]) -> None: pass - @pytest.mark.xfail(reason="not implemented", strict=False) - async def test_serizalizable_store(self, store) -> None: - pass - async def set(self, store: IcechunkStore, key: str, value: Buffer) -> None: await store._store.set(key, value.to_bytes()) @@ -62,10 +58,6 @@ async def store(self, store_kwargs: dict[str, Any]) -> IcechunkStore: def test_store_repr(self, store: IcechunkStore) -> None: super().test_store_repr(store) - @pytest.mark.xfail(reason="Not implemented") - def test_serializable_store(self, store: IcechunkStore) -> None: - super().test_serializable_store(store) - def test_store_mode(self, store, store_kwargs: dict[str, Any]) -> None: assert store.mode == AccessMode.from_literal("w") assert not store.mode.readonly From d35a0e5bf5e0458bc618479a35427c47737e74d1 Mon Sep 17 00:00:00 2001 From: Matthew Iannucci Date: Tue, 29 Oct 2024 08:59:52 -0700 Subject: [PATCH 2/2] Switch the store test to use local filesystem --- .../tests/test_zarr/test_store/test_icechunk_store.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/icechunk-python/tests/test_zarr/test_store/test_icechunk_store.py b/icechunk-python/tests/test_zarr/test_store/test_icechunk_store.py index 74c5aea1..fcad0d19 100644 --- a/icechunk-python/tests/test_zarr/test_store/test_icechunk_store.py +++ b/icechunk-python/tests/test_zarr/test_store/test_icechunk_store.py @@ -42,10 +42,10 @@ async def get(self, store: IcechunkStore, key: str) -> Buffer: return self.buffer_cls.from_bytes(result) - @pytest.fixture(scope="function", params=[None, True]) - def store_kwargs(self) -> dict[str, Any]: + @pytest.fixture + def store_kwargs(self, tmpdir) -> dict[str, Any]: kwargs = { - "storage": StorageConfig.memory("store_test"), + "storage": StorageConfig.filesystem(f"{tmpdir}/store_test"), "mode": "w", } return kwargs