External-Mirrors/awx
2
0
Fork 0
mirror of https://github.com/ansible/awx.git synced 2026-03-20 10:27:34 -02:30
Code Issues Packages Projects Releases Wiki Activity

remove extra validation on project.organization, enforced in UI, but allowed in API

Browse Source
This commit is contained in:
Wayne Witzel III
2016-04-19 11:48:29 -04:00
parent 6c7382b7b9
commit 82464fb45b
2 changed files with 0 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
awx/api/serializers.py
View File

@@ -899,12 +899,6 @@ class ProjectSerializer(UnifiedJobTemplateSerializer, ProjectOptionsSerializer):
args=(obj.last_update.pk,)) args=(obj.last_update.pk,))
return res return res
def validate(self, attrs):
if 'organization' not in attrs or type(attrs['organization']) is not Organization:
raise serializers.ValidationError('Missing organization')
return super(ProjectSerializer, self).validate(attrs)
class ProjectPlaybooksSerializer(ProjectSerializer): class ProjectPlaybooksSerializer(ProjectSerializer):

6
awx/main/tests/functional/test_projects.py
View File

@@ -116,9 +116,3 @@ def test_create_project(post, organization, org_admin, org_member, admin, rando)
assert Project.objects.filter(name='Project %d' % i, organization=organization).exists() assert Project.objects.filter(name='Project %d' % i, organization=organization).exists()
else: else:
assert not Project.objects.filter(name='Project %d' % i, organization=organization).exists() assert not Project.objects.filter(name='Project %d' % i, organization=organization).exists()
@pytest.mark.django_db(transaction=True)
def test_cant_create_project_without_org(post, organization, org_admin, org_member, admin, rando):
assert post(reverse('api:project_list'), { 'name': 'Project foo', }, admin).status_code == 400
assert post(reverse('api:project_list'), { 'name': 'Project foo', 'organization': None}, admin).status_code == 400