review comments

Author: dcherian

changes/2804.feature.rst

@@ -1 +1 @@
-:py:class:`LocalStore` learned to ``delete_dir``.
+:py:class:`LocalStore` learned to ``delete_dir``. This makes array and group deletes more efficient.

tests/test_store/test_stateful.py

@@ -29,5 +29,8 @@ def mk_test_instance_sync() -> None:
         pytest.skip(reason="ZipStore does not support delete")
 
     if isinstance(sync_store, LocalStore):
+        # This test uses arbitrary keys, which are passed to `set` and `delete`.
+        # It assumes that `set` and `delete` are the only two operations that modify state.
+        # But LocalStore, directories can hang around even after a key is delete-d.
         pytest.skip(reason="Test isn't suitable for LocalStore.")
     run_state_machine_as_test(mk_test_instance_sync)