taiga
/
taiga-back
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Fix tasks creation in bulk test

remotes/origin/issue/4795/notification_even_they_are_disabled
David Barragán Merino 2016-08-17 17:01:02 +02:00
parent f7c74fc63e
commit 275ce381f9
2 changed files with 19 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
taiga/projects/tasks/api.py
View File

@ -251,7 +251,9 @@ class TaskViewSet(OCCResourceMixin, VotedResourceMixin, HistoryResourceMixin,
@list_route(methods=["POST"]) @list_route(methods=["POST"])
def bulk_create(self, request, **kwargs): def bulk_create(self, request, **kwargs):
validator = validators.TasksBulkValidator(data=request.DATA) validator = validators.TasksBulkValidator(data=request.DATA)
if validator.is_valid(): if not validator.is_valid():
return response.BadRequest(validator.errors)
data = validator.data data = validator.data
project = Project.objects.get(id=data["project_id"]) project = Project.objects.get(id=data["project_id"])
self.check_permissions(request, 'bulk_create', project) self.check_permissions(request, 'bulk_create', project)
@ -268,7 +270,6 @@ class TaskViewSet(OCCResourceMixin, VotedResourceMixin, HistoryResourceMixin,
return response.Ok(tasks_serialized.data) return response.Ok(tasks_serialized.data)
return response.BadRequest(validator.errors)
def _bulk_update_order(self, order_field, request, **kwargs): def _bulk_update_order(self, order_field, request, **kwargs):
validator = validators.UpdateTasksOrderBulkValidator(data=request.DATA) validator = validators.UpdateTasksOrderBulkValidator(data=request.DATA)

8
tests/integration/resources_permissions/test_tasks_resources.py
View File

@ -656,7 +656,7 @@ def test_task_action_bulk_create(client, data):
"bulk_tasks": "test1\ntest2", "bulk_tasks": "test1\ntest2",
"us_id": data.public_task.user_story.pk, "us_id": data.public_task.user_story.pk,
"project_id": data.public_task.project.pk, "project_id": data.public_task.project.pk,
"sprint_id": data.public_task.milestone.pk, "milestone_id": data.public_task.milestone.pk,
}) })
results = helper_test_http_method(client, 'post', url, bulk_data, users) results = helper_test_http_method(client, 'post', url, bulk_data, users)
assert results == [401, 403, 403, 200, 200] assert results == [401, 403, 403, 200, 200]
@ -665,7 +665,7 @@ def test_task_action_bulk_create(client, data):
"bulk_tasks": "test1\ntest2", "bulk_tasks": "test1\ntest2",
"us_id": data.private_task1.user_story.pk, "us_id": data.private_task1.user_story.pk,
"project_id": data.private_task1.project.pk, "project_id": data.private_task1.project.pk,
"sprint_id": data.private_task1.milestone.pk, "milestone_id": data.private_task1.milestone.pk,
}) })
results = helper_test_http_method(client, 'post', url, bulk_data, users) results = helper_test_http_method(client, 'post', url, bulk_data, users)
assert results == [401, 403, 403, 200, 200] assert results == [401, 403, 403, 200, 200]
@ -674,7 +674,7 @@ def test_task_action_bulk_create(client, data):
"bulk_tasks": "test1\ntest2", "bulk_tasks": "test1\ntest2",
"us_id": data.private_task2.user_story.pk, "us_id": data.private_task2.user_story.pk,
"project_id": data.private_task2.project.pk, "project_id": data.private_task2.project.pk,
"sprint_id": data.private_task2.milestone.pk, "milestone_id": data.private_task2.milestone.pk,
}) })
results = helper_test_http_method(client, 'post', url, bulk_data, users) results = helper_test_http_method(client, 'post', url, bulk_data, users)
assert results == [401, 403, 403, 200, 200] assert results == [401, 403, 403, 200, 200]
@ -683,7 +683,7 @@ def test_task_action_bulk_create(client, data):
"bulk_tasks": "test1\ntest2", "bulk_tasks": "test1\ntest2",
"us_id": data.blocked_task.user_story.pk, "us_id": data.blocked_task.user_story.pk,
"project_id": data.blocked_task.project.pk, "project_id": data.blocked_task.project.pk,
"sprint_id": data.blocked_task.milestone.pk, "milestone_id": data.blocked_task.milestone.pk,
}) })
results = helper_test_http_method(client, 'post', url, bulk_data, users) results = helper_test_http_method(client, 'post', url, bulk_data, users)
assert results == [401, 403, 403, 451, 451] assert results == [401, 403, 403, 451, 451]