diff --git a/docker/api/volume.py b/docker/api/volume.py index 83cccb6c..bb8b39b3 100644 --- a/docker/api/volume.py +++ b/docker/api/volume.py @@ -33,4 +33,3 @@ class VolumeApiMixin(object): url = self._url('/volumes/{0}', name) resp = self._delete(url) self._raise_for_status(resp) - return True diff --git a/docs/api.md b/docs/api.md index 31ec86a8..37b379b4 100644 --- a/docs/api.md +++ b/docs/api.md @@ -780,8 +780,7 @@ Remove a volume. Similar to the `docker volume rm` command. * name (str): The volume's name -**Returns** (bool): True on successful removal. Failure will raise a -`docker.errors.APIError` exception. +Failure to remove will raise a `docker.errors.APIError` exception. ## rename diff --git a/tests/integration/volume_test.py b/tests/integration/volume_test.py index b532e857..8fa2dab5 100644 --- a/tests/integration/volume_test.py +++ b/tests/integration/volume_test.py @@ -47,8 +47,7 @@ class TestVolumes(helpers.BaseTestCase): name = 'shootthebullet' self.tmp_volumes.append(name) self.client.create_volume(name) - result = self.client.remove_volume(name) - self.assertTrue(result) + self.client.remove_volume(name) def test_remove_nonexistent_volume(self): name = 'shootthebullet' diff --git a/tests/unit/volume_test.py b/tests/unit/volume_test.py index 876d0e92..5b1823a4 100644 --- a/tests/unit/volume_test.py +++ b/tests/unit/volume_test.py @@ -89,8 +89,7 @@ class VolumeTest(DockerClientTest): @base.requires_api_version('1.21') def test_remove_volume(self): name = 'perfectcherryblossom' - result = self.client.remove_volume(name) - self.assertTrue(result) + self.client.remove_volume(name) args = fake_request.call_args self.assertEqual(args[0][0], 'DELETE')