Allow renaming DataFiles

arkindex/dataimport/serializers/files.py

@@ -27,7 +27,7 @@ class DataFileSerializer(serializers.ModelSerializer):
             'status',
             's3_url',
         )
-        read_only_fields = ('id', 'name', 'hash', 'size', 'content_type', 'images', 's3_url', )
+        read_only_fields = ('id', 'hash', 'size', 'content_type', 'images', 's3_url', )
 
     def validate_status(self, value):
         if value == S3FileStatus.Checked:

arkindex/dataimport/tests/test_datafile_api.py

@@ -101,7 +101,7 @@ class TestDataFileApi(FixtureAPITestCase):
         s3_object().content_type = 'test/testfile'
 
         self.client.force_login(self.user)
-        response = self.client.put(
+        response = self.client.patch(
             reverse('api:file-retrieve', kwargs={'pk': self.df.id}),
             {'status': 'checked'},
         )
@@ -128,7 +128,7 @@ class TestDataFileApi(FixtureAPITestCase):
     def test_check_wrong_md5(self, s3_object):
         s3_object().e_tag = 'wrong md5'
         self.client.force_login(self.user)
-        response = self.client.put(
+        response = self.client.patch(
             reverse('api:file-retrieve', kwargs={'pk': self.df.id}),
             {'status': 'checked'},
         )
@@ -138,7 +138,7 @@ class TestDataFileApi(FixtureAPITestCase):
     def test_set_error_status_on_failure(self, s3_object):
         s3_object().e_tag = 'wrong md5'
         self.client.force_login(self.user)
-        response = self.client.put(
+        response = self.client.patch(
             reverse('api:file-retrieve', kwargs={'pk': self.df.id}),
             {'status': 'checked'},
         )
@@ -152,7 +152,7 @@ class TestDataFileApi(FixtureAPITestCase):
         self.df.save()
         s3_object().e_tag = 'corrupted md5'
         self.client.force_login(self.user)
-        response = self.client.put(
+        response = self.client.patch(
             reverse('api:file-retrieve', kwargs={'pk': self.df.id}),
             {'status': 'checked'},
         )
@@ -182,3 +182,14 @@ class TestDataFileApi(FixtureAPITestCase):
         self.assertEqual(response.status_code, status.HTTP_200_OK)
         data = response.json()
         self.assertEqual(data['s3_url'], 'http://somewhere')
+
+    def test_rename_datafile(self):
+        self.client.force_login(self.user)
+        self.assertEqual(self.df.name, 'test.pdf')
+        response = self.client.patch(
+            reverse('api:file-retrieve', kwargs={'pk': self.df.id}),
+            {'name': 'taste.pdf'},
+        )
+        self.assertEqual(response.status_code, status.HTTP_200_OK)
+        self.df.refresh_from_db()
+        self.assertEqual(self.df.name, 'taste.pdf')