issue_comments: 349198982

Right, though xarray used to call .load, which I think must have called through to obj.compute? If that's the case, potentially the tests need changing

``` xarray/tests/test_backends.py:252: AssertionError ___ test_dask_kwargs_variable[compute] ___ method = 'compute' @pytest.mark.parametrize("method", ['load', 'compute']) def test_dask_kwargs_variable(method): x = Variable('y', da.from_array(np.arange(3), chunks=(2,))) # args should be passed on to da.Array.compute() with mock.patch.object(da.Array, 'compute', return_value=np.arange(3)) as mock_compute: getattr(x, method)(foo='bar')

  mock_compute.assert_called_with(foo='bar')

xarray/tests/test_dask.py:724:

_mock_self = <MagicMock name='compute' id='139838059011600'>, args = () kwargs = {'foo': 'bar'}, expected = "compute(foo='bar')" def assert_called_with(_mock_self, args, *kwargs): """assert that the mock was called with the specified arguments.

        Raises an AssertionError if the args and keyword args passed in are
        different to the last call to the mock."""
    self = _mock_self
    if self.call_args is None:
        expected = self._format_mock_call_signature(args, kwargs)

      raise AssertionError('Expected call: %s\nNot called' % (expected,))

E AssertionError: Expected call: compute(foo='bar') E Not called ```