Remove special mapping of auto to {} in open_zarr

[jsignell]

• ClosesInconsistent chunking between xr.open_zarr and xr.open_dataset(..., engine='zarr') with chunks="auto" #11002
• Tests added
• User visible changes (including notable bug fixes) are documented in whats-new.rst

This PR makes the handling of chunks="auto" consistent between open_zarr and open_dataset(..., engine="zarr").

The handling of chunks still differs in open_zarr vs open_dataset(..., engine="zarr") in that the default in open_zarr is to use chunks={} and a chunk manager (aka dask) when available in your env. And in open_dataset the default is to use chunks=None (aka no chunks).

[dcherian]

let's issue a DeprecationWarning saying the default will switch to chunks=None to match open_dataset. If they want the current behaviour with dask et al, users should pass in chunks={}

[jsignell]

Oh I was actually thinking that we not do that part. The issue that we are trying to fix with this PR is really that chunks="auto" means different things in open_zarr and open_dataset(,,, engine="zarr"). That was the part that felt deeply surprising to me and @norlandrhagen. As long as we fix that I don't think we need to change the default value.

[dcherian]

👍 to merging this. It'd be nice to have the default be chunks=None always just like open_dataset. Otherwise we get behaviour that's dependent on the installed packages which is just frustrating

[norlandrhagen]

Thanks for this PR @jsignell!