Specify read-only mode

[szhan]

Fix #607

[szhan]

This seems a bit redundant to specify read-only mode again when calling open. I guess there's no harm in that.

[jeromekelleher]

It's a wart on Zarr apis all right, but it's necessary.

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.07%. Comparing base (1bd7cbb) to head (4cd6484).

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #608   +/-   ##
=======================================
  Coverage   88.07%   88.07%           
=======================================
  Files          12       12           
  Lines        4134     4134           
  Branches      585      585           
=======================================
  Hits         3641     3641           
  Misses        365      365           
  Partials      128      128           

Flag	Coverage Δ
python-tests	88.07% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[jeromekelleher]

Best add mode=r on DirectoryStore also

[szhan]

I checked but there's no mode argument

class DirectoryStore(
    path: Any,
    normalize_keys: bool = False,
    dimension_separator: DIMENSION_SEPARATOR | None = None
)