Skip to content

Commit fc3a446

Browse files
committed
Updated the test to reflect the break between class name and qualname
The config uses the public `zarr.buffer.cpu.Buffer`, which differs from the implementation path `zarr.core.buffer.cpu.Buffer`. This is OK because the public API for getting the buffer doesn't depend on where it's implemented at.
1 parent 83c8c32 commit fc3a446

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

src/zarr/core/config.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ def enable_gpu(self) -> ConfigSet:
6464
Configure Zarr to use GPUs where possible.
6565
"""
6666
return self.set(
67-
{"buffer": "zarr.core.buffer.gpu.Buffer", "ndbuffer": "zarr.core.buffer.gpu.NDBuffer"}
67+
{"buffer": "zarr.buffer.gpu.Buffer", "ndbuffer": "zarr.buffer.gpu.NDBuffer"}
6868
)
6969

7070

tests/test_config.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -240,7 +240,7 @@ def test_config_ndbuffer_implementation(store: Store) -> None:
240240

241241
def test_config_buffer_implementation() -> None:
242242
# has default value
243-
assert fully_qualified_name(get_buffer_class()) == config.defaults[0]["buffer"]
243+
assert config.defaults[0]["buffer"] == "zarr.buffer.cpu.Buffer"
244244

245245
arr = zeros(shape=(100,), store=StoreExpectingTestBuffer())
246246

0 commit comments

Comments
 (0)