Raise ValueError for unmatching chunks length in DataArray.chunk()

[lkstrp]

#9450 introduced strict handling of zip().
In DataArray.chunk(), this leads to a breaking change when passing a tuple/list as chunks where the length doesn't match self.dims. Previously these were just discarded.

xarray/xarray/core/dataarray.py

Line 1452 in 2c8f2e8

chunk_mapping = dict(zip(self.dims, chunks, strict=True))

Since passing a tuple/list as chunks is deprecated anyway, this PR simply adds a ValueError to make the problem clear to the user instead of this ambiguous strict zip error.

Current error:

Traceback (most recent call last):
  File "/Users/../.venv/lib/python3.12/site-packages/xarray/util/deprecation_helpers.py", line 118, in inner
    return func(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^
  File "/Users/../.venv/lib/python3.12/site-packages/xarray/core/dataarray.py", line 1447, in chunk
    chunk_mapping = dict(zip(self.dims, chunks, strict=True))
                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
ValueError: zip() argument 2 is longer than argument 1

[welcome]

Thank you for opening this pull request! It may take us a few days to respond here, so thank you for being patient.
If you have questions, some answers may be found in our contributing guidelines.

[max-sixty]

Adding a test would be great, but maybe this is one time where it's clear enough without one, so will plan to merge either way.

Thank you!

[welcome]

Congratulations on completing your first pull request! Welcome to Xarray! We are proud of you, and hope to see you again!

[max-sixty]

Thanks @lkstrp !