summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorCecile Tonglet <cecile.tonglet@gmail.com>2017-08-01 12:16:56 +0200
committerJoffrey F <f.joffrey@gmail.com>2017-08-17 13:38:40 -0700
commit9e793806ff79559c3bc591d8c52a3bbe3cdb7350 (patch)
tree339c399b1db4813bf0775d3f28c279773a63fb88
parentf3374959b7145d0c25042d51065b5d387832d19c (diff)
downloaddocker-py-9e793806ff79559c3bc591d8c52a3bbe3cdb7350.tar.gz
Return the result of the API when using remove_image and load_image
Those calls return result that can be used by the developers. Signed-off-by: Cecile Tonglet <cecile.tonglet@gmail.com>
-rw-r--r--docker/api/image.py4
-rw-r--r--tests/integration/api_image_test.py3
2 files changed, 4 insertions, 3 deletions
diff --git a/docker/api/image.py b/docker/api/image.py
index 181c4a1..85ff435 100644
--- a/docker/api/image.py
+++ b/docker/api/image.py
@@ -272,7 +272,7 @@ class ImageApiMixin(object):
data (binary): Image data to be loaded.
"""
res = self._post(self._url("/images/load"), data=data)
- self._raise_for_status(res)
+ return self._result(res, True)
@utils.minimum_version('1.25')
def prune_images(self, filters=None):
@@ -455,7 +455,7 @@ class ImageApiMixin(object):
"""
params = {'force': force, 'noprune': noprune}
res = self._delete(self._url("/images/{0}", image), params=params)
- self._raise_for_status(res)
+ return self._result(res, True)
def search(self, term):
"""
diff --git a/tests/integration/api_image_test.py b/tests/integration/api_image_test.py
index 917bc50..192e6f8 100644
--- a/tests/integration/api_image_test.py
+++ b/tests/integration/api_image_test.py
@@ -113,7 +113,8 @@ class RemoveImageTest(BaseAPIIntegrationTest):
self.assertIn('Id', res)
img_id = res['Id']
self.tmp_imgs.append(img_id)
- self.client.remove_image(img_id, force=True)
+ logs = self.client.remove_image(img_id, force=True)
+ self.assertIn({"Deleted": img_id}, logs)
images = self.client.images(all=True)
res = [x for x in images if x['Id'].startswith(img_id)]
self.assertEqual(len(res), 0)