Skip to content

Commit

Permalink
AK update must contain org id, it is a required field (SatelliteQE#12509
Browse files Browse the repository at this point in the history
)
  • Loading branch information
lhellebr authored and damoore044 committed Oct 10, 2023
1 parent b7db7e1 commit 5cda9ef
Showing 1 changed file with 12 additions and 2 deletions.
14 changes: 12 additions & 2 deletions tests/foreman/api/test_permission.py
Original file line number Diff line number Diff line change
Expand Up @@ -367,8 +367,18 @@ def test_positive_check_update(self, entity_cls, class_org, class_location):
new_entity = new_entity.create()
name = new_entity.get_fields()['name'].gen_value()
with pytest.raises(HTTPError):
entity_cls(self.cfg, id=new_entity.id, name=name).update(['name'])
if entity_cls is entities.ActivationKey:
entity_cls(self.cfg, id=new_entity.id, name=name, organization=class_org).update(
['name']
)
else:
entity_cls(self.cfg, id=new_entity.id, name=name).update(['name'])
self.give_user_permission(_permission_name(entity_cls, 'update'))
# update() calls read() under the hood, which triggers
# permission error
entity_cls(self.cfg, id=new_entity.id, name=name).update_json(['name'])
if entity_cls is entities.ActivationKey:
entity_cls(self.cfg, id=new_entity.id, name=name, organization=class_org).update_json(
['name']
)
else:
entity_cls(self.cfg, id=new_entity.id, name=name).update_json(['name'])

0 comments on commit 5cda9ef

Please sign in to comment.