Raise KeyError instead of ValueError.

dcherian · dcherian · commit 0b47042779f3 · 2020-07-30T15:06:09.000-06:00
I should be using Pull Requests!
diff --git a/cf_xarray/accessor.py b/cf_xarray/accessor.py
@@ -921,7 +921,7 @@ def __getitem__(self, key: Union[str, List[str]]):
                 try:
                     names = axis_coord_mapper(self._obj, k)
                 except KeyError as e:
-                    raise ValueError(
+                    raise KeyError(
                         f"Receive multiple variables for key {k!r}. Expected only one. Please pass a list [{k!r}] instead to get all variables matching {k!r}."
                     )
                     raise e

Original file line number	Diff line number	Diff line change
`@@ -921,7 +921,7 @@ def __getitem__(self, key: Union[str, List[str]]):`
`921`	`921`	`try:`
`922`	`922`	`names = axis_coord_mapper(self._obj, k)`
`923`	`923`	`except KeyError as e:`
`924`		`- raise ValueError(`
	`924`	`+ raise KeyError(`
`925`	`925`	`f"Receive multiple variables for key {k!r}. Expected only one. Please pass a list [{k!r}] instead to get all variables matching {k!r}."`
`926`	`926`	`)`
`927`	`927`	`raise e`