Merge pull request #7285 from readthedocs/humitos/apiv3-update-hidden

readthedocs · Jul 23, 2020 · dd1ba8c · dd1ba8c
2 parents 97124cf + b6f31ae
commit dd1ba8c
Show file tree

Hide file tree

Showing 3 changed files with 32 additions and 0 deletions.
diff --git a/readthedocs/api/v3/serializers.py b/readthedocs/api/v3/serializers.py
@@ -261,6 +261,7 @@ class Meta:
         model = Version
         fields = [
             'active',
+            'hidden',
         ]
 
 

diff --git a/readthedocs/api/v3/tests/mixins.py b/readthedocs/api/v3/tests/mixins.py
@@ -73,6 +73,7 @@ def setUp(self):
             verbose_name='v1.0',
             identifier='a1b2c3',
             project=self.project,
+            hidden=False,
             active=True,
             built=True,
             type=TAG,

diff --git a/readthedocs/api/v3/tests/test_versions.py b/readthedocs/api/v3/tests/test_versions.py
@@ -95,3 +95,33 @@ def test_projects_versions_detail_unique(self):
             ),
         )
         self.assertEqual(response.status_code, 200)
+
+    def test_projects_versions_partial_update(self):
+        self.assertTrue(self.version.active)
+        self.assertFalse(self.version.hidden)
+        data = {
+            'active': False,
+            'hidden': True,
+        }
+
+        self.client.credentials(HTTP_AUTHORIZATION=f'Token {self.token.key}')
+        response = self.client.patch(
+            reverse(
+                'projects-versions-detail',
+                kwargs={
+                    'parent_lookup_project__slug': self.project.slug,
+                    'version_slug': self.version.slug,
+                },
+            ),
+            data,
+        )
+        self.assertEqual(response.status_code, 204)
+
+        self.version.refresh_from_db()
+        self.assertEqual(self.version.verbose_name, 'v1.0')
+        self.assertEqual(self.version.slug, 'v1.0')
+        self.assertEqual(self.version.identifier, 'a1b2c3')
+        self.assertFalse(self.version.active)
+        self.assertTrue(self.version.hidden)
+        self.assertTrue(self.version.built)
+        self.assertEqual(self.version.type, TAG)