diff --git a/cvat/apps/engine/serializers.py b/cvat/apps/engine/serializers.py index 49badfee..da90c66f 100644 --- a/cvat/apps/engine/serializers.py +++ b/cvat/apps/engine/serializers.py @@ -273,6 +273,7 @@ class TaskSerializer(WriteOnceMixin, serializers.ModelSerializer): db_attr.values = attr.get('values', db_attr.values) db_attr.save() + instance.save() return instance class UserSerializer(serializers.ModelSerializer): diff --git a/cvat/apps/engine/tests/test_rest_api.py b/cvat/apps/engine/tests/test_rest_api.py index 63870910..d352a4f7 100644 --- a/cvat/apps/engine/tests/test_rest_api.py +++ b/cvat/apps/engine/tests/test_rest_api.py @@ -843,15 +843,17 @@ class TaskPartialUpdateAPITestCase(TaskUpdateAPITestCase): def test_api_v1_tasks_id_admin_partial(self): data = { - "name": "new name for the task", - "owner": self.owner.id + "name": "new name for the task #2", } self._check_api_v1_tasks_id(self.admin, data) data = { - "name": "new name for the task #2", + "name": "new name for the task", + "owner": self.owner.id } self._check_api_v1_tasks_id(self.admin, data) + # Now owner is updated, but self.db_tasks are obsolete + # We can't do any tests without owner in data below def test_api_v1_tasks_id_user_partial(self):