issues: 1093467156

This data as json

id node_id number title user state locked assignee milestone comments created_at updated_at closed_at author_association active_lock_reason draft pull_request body reactions performed_via_github_app state_reason repo type
1093467156 I_kwDOAMm_X85BLPwU 6136 DataArray.from_dict: convert docstring code to doctest 72647856 closed 0     6 2022-01-04T15:28:56Z 2022-02-28T09:11:02Z 2022-02-28T09:11:02Z CONTRIBUTOR      

What happened?

This test is passing in a str for dims: https://github.com/pydata/xarray/blob/60754fdbc4ecd9eb3c0978e82635c6d43e8d485b/xarray/tests/test_dataarray.py#L3074-L3078

dims should be a tuple. I suspect the intent was to specify d = {"dims": ("t",)} when the test was written.

This is small issue, but tests that call functions using the kind of data that would be used in "live" will improve quality of tests. Would be a shame for bug to affect users because tests use a str while in userspace a tuple is used.

What did you expect to happen?

code to be d = {"dims": ("t",)}

Minimal Complete Verifiable Example

No response

Relevant log output

No response

Anything else we need to know?

I'm happy to make a PR to fix this

Environment

NA

 
{
    "url": "https://api.github.com/repos/pydata/xarray/issues/6136/reactions",
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   completed 13221727 issue

Links from other tables