Skip to content

Commit dcf6b16

Browse files
author
NIK-TIGER-BILL
committed
fix: address review comment — remove standalone regression test per maintainer feedback
Signed-off-by: NIK-TIGER-BILL <nik.tiger.bill@github.com>
1 parent d492d74 commit dcf6b16

1 file changed

Lines changed: 0 additions & 20 deletions

File tree

tests/test_store/test_stateful.py

Lines changed: 0 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -51,23 +51,3 @@ def mk_test_instance_sync() -> ZarrStoreStateMachine:
5151
run_state_machine_as_test(mk_test_instance_sync) # type: ignore[no-untyped-call]
5252

5353

54-
def test_delete_dir_prefix_matching() -> None:
55-
"""Regression test for delete_dir prefix matching bug (GH#3977).
56-
57-
Verifies that delete_dir bookkeeping only removes exact path matches
58-
and true descendants, not unrelated nodes that merely share a string
59-
prefix (e.g. ``6/faNT…`` must NOT be deleted when removing ``6/f``).
60-
"""
61-
all_groups = {"6/f", "6/faNT7p7jvJsO3_C._HYi", "other"}
62-
all_arrays = {"6/f/child", "6/other"}
63-
path = "6/f"
64-
65-
matches = set()
66-
for node in all_groups | all_arrays:
67-
if node == path or node.startswith(path + "/"):
68-
matches.add(node)
69-
70-
assert matches == {"6/f", "6/f/child"}
71-
assert "6/faNT7p7jvJsO3_C._HYi" not in matches
72-
assert "other" not in matches
73-
assert "6/other" not in matches

0 commit comments

Comments
 (0)