From 741dcb5277cf8c60d0654b9589b568e2f7db03c4 Mon Sep 17 00:00:00 2001 From: wangqun Date: Sat, 26 Mar 2016 02:27:57 +0000 Subject: [PATCH] Remove the update function of container The container update function isn't called by any function. So we should delete it. Change-Id: I832b2082f8da441a7924623c29df2255fbd7ad28 Closes-Bug: #1562236 --- magnumclient/tests/v1/test_containers.py | 11 ----------- magnumclient/v1/containers.py | 3 --- 2 files changed, 14 deletions(-) diff --git a/magnumclient/tests/v1/test_containers.py b/magnumclient/tests/v1/test_containers.py index fd85d8c..17a8a61 100644 --- a/magnumclient/tests/v1/test_containers.py +++ b/magnumclient/tests/v1/test_containers.py @@ -368,17 +368,6 @@ class ContainerManagerTest(testtools.TestCase): self.assertEqual(expect, self.api.calls) self.assertIsNone(container) - def test_container_update(self): - patch = {'op': 'replace', - 'value': NEW_NAME, - 'path': '/name'} - container = self.mgr.update(id=CONTAINER1['id'], patch=patch) - expect = [ - ('PATCH', '/v1/containers/%s' % CONTAINER1['id'], {}, patch), - ] - self.assertEqual(expect, self.api.calls) - self.assertEqual(NEW_NAME, container.name) - def test_container_start(self): container = self.mgr.start(CONTAINER1['id']) expect = [ diff --git a/magnumclient/v1/containers.py b/magnumclient/v1/containers.py index bfd39cf..a377b00 100644 --- a/magnumclient/v1/containers.py +++ b/magnumclient/v1/containers.py @@ -110,9 +110,6 @@ class ContainerManager(base.Manager): def delete(self, id): return self._delete(self._path(id)) - def update(self, id, patch): - return self._update(self._path(id), patch) - def _action(self, id, action, method='PUT', qparams=None, **kwargs): if qparams: action = "%s?%s" % (action,