openstack/cyborg
Code Issues Proposed changes

Fix delete attribute api bug

Browse Source 
Closes-Bug: #2020179
Change-Id: I7a97cecff3bdfa1ee80bb1d2ee5f8a9e9d1448cf
This commit is contained in:
songwenping 2023-05-19 18:06:42 +08:00
parent b4ca84cd90
commit df85db5677
2 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
cyborg/api/controllers/v2/attributes.py
View File

@ -163,4 +163,5 @@ class AttributesController(base.CyborgController,
"""
LOG.info('[attributes] delete by uuid: %s.', uuid)
context = pecan.request.context
objects.Attribute.destroy(context, uuid)
api_obj_attribute = objects.Attribute.get(context, uuid)
api_obj_attribute.destroy(context)

3
cyborg/tests/unit/api/controllers/v2/test_attributes.py
View File

@ -114,8 +114,9 @@ class TestAttributes(v2_test.APITestV2):
for in_attribute, out_attribute in zip(attributes, out_attributes):
self._validate_attributes(in_attribute, out_attribute)
@mock.patch('cyborg.objects.Attribute.get')
@mock.patch('cyborg.objects.Attribute.destroy')
def test_delete(self, mock_attribute_uuid):
def test_delete(self, mock_attribute_delete, mock_attribute):
uuid = self.fake_attribute_objs[0]['uuid']
# Delete by UUID
url = self.ATTRIBUTE_URL + '/%s' % uuid