From da04c0baceb07ceaf32521a7c8ac2c1657df50f7 Mon Sep 17 00:00:00 2001
From: Boris Sekachev <40690378+bsekachev@users.noreply.github.com>
Date: Tue, 3 Sep 2019 18:19:51 +0300
Subject: [PATCH] Fixed update of a task via REST (#686)

* Fixed update of a task via REST

* Fixed unit tests
---
 cvat/apps/engine/serializers.py         | 1 +
 cvat/apps/engine/tests/test_rest_api.py | 8 +++++---
 2 files changed, 6 insertions(+), 3 deletions(-)

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):