+ Permissions+ | + + {{ state.name | translate }} + + + {{ state.name | translate | limitTo: 1 }}... + + + | +
|---|---|
| + Action word + | +
+
+
+
+
+ {{ state.action_word | translate }}
+
+
+ —
+
+
+
+ |
+
| + Recommendation label + | +
+
+
+
+
+ {{ state.recommendation_label | translate }}
+
+
+ —
+
+
+
+ |
+
| + {{ member.displayName | translate }} + | ++ + + | +
| + Label color + | ++ + + + {{ cssClasses[state.css_class] }} + + + + + + | +
| + Required permission to see + | +
+
+
+
+
+ {{ getPermissionDisplayName(state.required_permission_to_see) | translate }}
+
+
+ —
+
+
+
+
+
+
+ |
+
| + Next states + | +
+
+ —
+
+
+
+ {{ nextState.name | translate }},
+
+
+
+
+
+
+
+
+ |
+
|
+ |
+
|---|
| + {{ workflow.name | translate }} + + | +
All your changes are saved immediately. Changes you make are only effective once you (or the users concerned) reload the page.
-| + |
Permissions |
@@ -49,7 +49,7 @@
- |
{{ app.app_name | translate}}
diff --git a/openslides/utils/rest_api.py b/openslides/utils/rest_api.py
index a9d509428..80c1bbdbe 100644
--- a/openslides/utils/rest_api.py
+++ b/openslides/utils/rest_api.py
@@ -7,7 +7,11 @@ from rest_framework.decorators import detail_route, list_route # noqa
from rest_framework.metadata import SimpleMetadata # noqa
from rest_framework.mixins import ListModelMixin as _ListModelMixin
from rest_framework.mixins import RetrieveModelMixin as _RetrieveModelMixin
-from rest_framework.mixins import DestroyModelMixin, UpdateModelMixin # noqa
+from rest_framework.mixins import ( # noqa
+ CreateModelMixin,
+ DestroyModelMixin,
+ UpdateModelMixin,
+)
from rest_framework.response import Response
from rest_framework.routers import DefaultRouter
from rest_framework.serializers import ModelSerializer as _ModelSerializer
diff --git a/tests/integration/motions/test_viewset.py b/tests/integration/motions/test_viewset.py
index 9b1558627..8f04e3223 100644
--- a/tests/integration/motions/test_viewset.py
+++ b/tests/integration/motions/test_viewset.py
@@ -1237,3 +1237,117 @@ class FollowRecommendationsForMotionBlock(TestCase):
self.assertEqual(response.status_code, status.HTTP_200_OK)
self.assertEqual(Motion.objects.get(pk=self.motion.pk).state.id, self.state_id_accepted)
self.assertEqual(Motion.objects.get(pk=self.motion_2.pk).state.id, self.state_id_rejected)
+
+
+class CreateWorkflow(TestCase):
+ """
+ Tests the creating of workflows.
+ """
+ def setUp(self):
+ self.client = APIClient()
+ self.client.login(username='admin', password='admin')
+
+ def test_creation(self):
+ Workflow.objects.all().delete()
+ response = self.client.post(
+ reverse('workflow-list'),
+ {'name': 'test_name_OoCoo3MeiT9li5Iengu9'})
+ self.assertEqual(response.status_code, status.HTTP_201_CREATED)
+ workflow = Workflow.objects.get()
+ self.assertEqual(workflow.name, 'test_name_OoCoo3MeiT9li5Iengu9')
+ first_state = workflow.first_state
+ self.assertEqual(type(first_state), State)
+
+ def test_creation_with_wrong_first_state(self):
+ response = self.client.post(
+ reverse('workflow-list'),
+ {'name': 'test_name_OoCoo3MeiT9li5Iengu9',
+ 'first_state': 1})
+ self.assertEqual(response.status_code, status.HTTP_400_BAD_REQUEST)
+
+ def test_creation_with_not_existing_first_state(self):
+ Workflow.objects.all().delete()
+ response = self.client.post(
+ reverse('workflow-list'),
+ {'name': 'test_name_OoCoo3MeiT9li5Iengu9',
+ 'first_state': 49})
+ self.assertEqual(response.status_code, status.HTTP_400_BAD_REQUEST)
+
+
+class UpdateWorkflow(TestCase):
+ """
+ Tests the updating of workflows.
+ """
+ def setUp(self):
+ self.client = APIClient()
+ self.client.login(username='admin', password='admin')
+ self.workflow = Workflow.objects.first()
+
+ def test_rename_workflow(self):
+ response = self.client.patch(
+ reverse('workflow-detail', args=[self.workflow.pk]),
+ {'name': 'test_name_wofi38DiWLT"8d3lwfo3'})
+
+ workflow = Workflow.objects.get(pk=self.workflow.id)
+ self.assertEqual(response.status_code, status.HTTP_200_OK)
+ self.assertEqual(workflow.name, 'test_name_wofi38DiWLT"8d3lwfo3')
+
+ def test_change_first_state_correct(self):
+ first_state = self.workflow.first_state
+ other_workflow_state = self.workflow.states.exclude(pk=first_state.pk).first()
+ response = self.client.patch(
+ reverse('workflow-detail', args=[self.workflow.pk]),
+ {'first_state': other_workflow_state.pk})
+
+ workflow = Workflow.objects.get(pk=self.workflow.id)
+ self.assertEqual(response.status_code, status.HTTP_200_OK)
+ self.assertEqual(workflow.first_state, other_workflow_state)
+
+ def test_change_first_state_not_existing(self):
+ first_state = self.workflow.first_state
+ response = self.client.patch(
+ reverse('workflow-detail', args=[self.workflow.pk]),
+ {'first_state': 42})
+
+ workflow = Workflow.objects.get(pk=self.workflow.id)
+ self.assertEqual(response.status_code, status.HTTP_400_BAD_REQUEST)
+ self.assertEqual(workflow.first_state, first_state)
+
+ def test_change_first_state_wrong_workflow(self):
+ first_state = self.workflow.first_state
+ other_workflow = Workflow.objects.exclude(pk=self.workflow.pk).first()
+ response = self.client.patch(
+ reverse('workflow-detail', args=[self.workflow.pk]),
+ {'first_state': other_workflow.first_state.pk})
+
+ workflow = Workflow.objects.get(pk=self.workflow.id)
+ self.assertEqual(response.status_code, status.HTTP_400_BAD_REQUEST)
+ self.assertEqual(workflow.first_state, first_state)
+
+
+class DeleteWorkflow(TestCase):
+ """
+ Tests the deletion of workflows.
+ """
+ def setUp(self):
+ self.client = APIClient()
+ self.client.login(username='admin', password='admin')
+ self.workflow = Workflow.objects.first()
+
+ def test_simple_delete(self):
+ response = self.client.delete(
+ reverse('workflow-detail', args=[self.workflow.pk]))
+ self.assertEqual(response.status_code, status.HTTP_204_NO_CONTENT)
+ self.assertEqual(Workflow.objects.count(), 1) # Just the other default one
+
+ def test_delete_with_assigned_motions(self):
+ self.motion = Motion(
+ title='test_title_chee7ahCha6bingaew4e',
+ text='test_text_birah1theL9ooseeFaip')
+ self.motion.reset_state(self.workflow)
+ self.motion.save()
+
+ response = self.client.delete(
+ reverse('workflow-detail', args=[self.workflow.pk]))
+ self.assertEqual(response.status_code, status.HTTP_400_BAD_REQUEST)
+ self.assertEqual(Workflow.objects.count(), 2)
| |
|---|