decisions: added to endpoint to resolve

backoffice/backoffice/workflows/api/views.py

@@ -43,6 +43,16 @@
 logger = logging.getLogger(__name__)
 
 
+def add_decision(workflow_id, user, action):
+    serializer_class = DecisionSerializer
+    data = {"workflow": workflow_id, "user": user, "action": action}
+
+    serializer = serializer_class(data=data)
+    if serializer.is_valid(raise_exception=True):
+        serializer.save()
+        return Response(serializer.data, status=status.HTTP_201_CREATED)
+
+
 class WorkflowViewSet(viewsets.ModelViewSet):
     queryset = Workflow.objects.all()
     serializer_class = WorkflowSerializer
@@ -103,25 +113,11 @@ def create(self, request, *args, **kwargs):
 
 class DecisionViewSet(viewsets.ModelViewSet):
     queryset = Decision.objects.all()
-    serializer_class = DecisionSerializer
-
-    def get_queryset(self):
-        status = self.request.query_params.get("status")
-        if status:
-            return self.queryset.filter(status__status=status)
-        return self.queryset
 
     def create(self, request, *args, **kwargs):
-        data = {
-            "workflow": request.data["workflow_id"],
-            "user": request.user,
-            "action": request.data["action"],
-        }
-
-        serializer = self.serializer_class(data=data)
-        if serializer.is_valid(raise_exception=True):
-            serializer.save()
-            return Response(serializer.data, status=status.HTTP_201_CREATED)
+        return add_decision(
+            request.data["workflow_id"], request.user, request.data["action"]
+        )
 
 
 class AuthorWorkflowViewSet(viewsets.ViewSet):
@@ -190,6 +186,7 @@ def resolve(self, request, pk=None):
                 ResolutionDags[serializer.validated_data["value"]],
                 pk,
             )
+            add_decision(pk, request.user, serializer.validated_data["value"])
 
             return airflow_utils.trigger_airflow_dag(
                 ResolutionDags[serializer.validated_data["value"]].label, pk, extra_data

backoffice/backoffice/workflows/tests/test_views.py

@@ -324,7 +324,8 @@ def test_create_author(self):
     @pytest.mark.vcr()
     def test_accept_author(self):
         self.api_client.force_authenticate(user=self.curator)
-        data = {"create_ticket": True, "value": "accept"}
+        action = "accept"
+        data = {"create_ticket": True, "value": action}
 
         response = self.api_client.post(
             reverse("api:workflows-authors-resolve", kwargs={"pk": self.workflow.id}),
@@ -333,6 +334,9 @@ def test_accept_author(self):
         )
 
         self.assertEqual(response.status_code, 200)
+        self.assertEqual(
+            Decision.objects.filter(workflow=self.workflow.id)[0].action, action
+        )
 
         airflow_utils.delete_workflow_dag(
             WORKFLOW_DAGS[WorkflowType.AUTHOR_CREATE].approve, self.workflow.id
@@ -341,7 +345,8 @@ def test_accept_author(self):
     @pytest.mark.vcr()
     def test_reject_author(self):
         self.api_client.force_authenticate(user=self.curator)
-        data = {"create_ticket": True, "value": "reject"}
+        action = "reject"
+        data = {"create_ticket": True, "value": action}
 
         response = self.api_client.post(
             reverse("api:workflows-authors-resolve", kwargs={"pk": self.workflow.id}),
@@ -350,6 +355,9 @@ def test_reject_author(self):
         )
 
         self.assertEqual(response.status_code, 200)
+        self.assertEqual(
+            Decision.objects.filter(workflow=self.workflow.id)[0].action, action
+        )
 
         airflow_utils.delete_workflow_dag(
             WORKFLOW_DAGS[WorkflowType.AUTHOR_CREATE].reject, self.workflow.id