diff --git a/glanceclient/v1/image_members.py b/glanceclient/v1/image_members.py index f1863f9d..b42fc74f 100644 --- a/glanceclient/v1/image_members.py +++ b/glanceclient/v1/image_members.py @@ -20,6 +20,10 @@ class ImageMember(base.Resource): def __repr__(self): return "" % self._info + @property + def id(self): + return self.member_id + def delete(self): self.manager.delete(self) diff --git a/tests/v1/test_image_members.py b/tests/v1/test_image_members.py index 4613a505..2a0ed38b 100644 --- a/tests/v1/test_image_members.py +++ b/tests/v1/test_image_members.py @@ -24,13 +24,11 @@ class ImageMemberManagerTest(unittest.TestCase): self.assertEqual(members[0].can_share, False) def test_list_by_member(self): - members = self.mgr.list(member='1') + resource_class = glanceclient.v1.image_members.ImageMember + member = resource_class(self.api, {'member_id': '1'}, True) + self.mgr.list(member=member) expect = [('GET', '/v1/shared-images/1', {}, None)] self.assertEqual(self.api.calls, expect) - self.assertEqual(len(members), 1) - self.assertEqual(members[0].member_id, '1') - self.assertEqual(members[0].image_id, '1') - self.assertEqual(members[0].can_share, False) def test_get(self): member = self.mgr.get(self.image, '1') @@ -63,9 +61,9 @@ class ImageMemberManagerTest(unittest.TestCase): def test_replace_objects(self): body = [ glanceclient.v1.image_members.ImageMember( - self.mgr, {'member_id': '2', 'can_share': False}), + self.mgr, {'member_id': '2', 'can_share': False}, True), glanceclient.v1.image_members.ImageMember( - self.mgr, {'member_id': '3', 'can_share': True}), + self.mgr, {'member_id': '3', 'can_share': True}, True), ] self.mgr.replace(self.image, body) expect_body = {