diff --git a/atst/forms/application_member.py b/atst/forms/application_member.py index 8dcb1059..a6af78e3 100644 --- a/atst/forms/application_member.py +++ b/atst/forms/application_member.py @@ -58,7 +58,6 @@ class NewForm(BaseForm): class UpdateMemberForm(PermissionsForm): - # permission_sets = FormField(PermissionsForm) environment_roles = FieldList(FormField(EnvironmentForm)) @property diff --git a/atst/models/application_role.py b/atst/models/application_role.py index 4e00c5b0..7e709f5f 100644 --- a/atst/models/application_role.py +++ b/atst/models/application_role.py @@ -71,20 +71,6 @@ class ApplicationRole( elif self.latest_invitation: return self.latest_invitation.user_name - @property - def user_first_name(self): - if self.user: - return self.user.first_name - elif self.latest_invitation: - return self.latest_invitation.first_name - - @property - def user_last_name(self): - if self.user: - return self.user.last_name - elif self.latest_invitation: - return self.latest_invitation.last_name - def __repr__(self): return "".format( self.application.name, self.user_id, self.id, self.permissions diff --git a/tests/forms/test_application_member.py b/tests/forms/test_application_member.py index e4d59f40..61a9e373 100644 --- a/tests/forms/test_application_member.py +++ b/tests/forms/test_application_member.py @@ -42,11 +42,6 @@ def test_update_member_form(): "perms_team_mgmt": True, "perms_env_mgmt": False, "perms_del_env": False, - "environment_roles": { - "environment_id": 123, - "environment_name": "testing", - "role": ENV_ROLES[0][0], - }, } form = UpdateMemberForm(data=form_data) assert form.validate()