More attributes when importing a Project

Author: humitos

readthedocs/api/v3/serializers.py

@@ -394,14 +394,16 @@ class ProjectCreateSerializer(FlexFieldsModelSerializer):
     """Serializer used to Import a Project."""
 
     repository = RepositorySerializer(source='*')
+    homepage = serializers.URLField(source='project_url', required=False)
 
     class Meta:
         model = Project
         fields = (
             'name',
             'language',
+            'programming_language',
             'repository',
-            'project_url',  # project_homepage
+            'homepage',
         )
 
 

readthedocs/api/v3/tests/responses/projects-list_POST.json

@@ -23,8 +23,8 @@
         "name": "Public"
     },
     "programming_language": {
-        "code": "words",
-        "name": "Only Words"
+        "code": "py",
+        "name": "Python"
     },
     "repository": {
         "type": "git",
@@ -36,7 +36,7 @@
     "translation_of": null,
     "urls": {
         "documentation": "http://readthedocs.org/docs/test-project/en/latest/",
-        "project_homepage": null
+        "project_homepage": "http://template.readthedocs.io/"
     },
     "users": [
         {

readthedocs/api/v3/tests/test_projects.py

@@ -540,6 +540,8 @@ def test_import_project(self):
                 'url': 'https://github.com/rtfd/template',
                 'type': 'git',
             },
+            'homepage': 'http://template.readthedocs.io/',
+            'programming_language': 'py',
         }
 
         self.client.credentials(HTTP_AUTHORIZATION=f'Token {self.token.key}')
@@ -554,6 +556,9 @@ def test_import_project(self):
         self.assertEqual(project.slug, 'test-project')
         self.assertEqual(project.repo, 'https://github.com/rtfd/template')
         self.assertEqual(project.language, 'en')
+        self.assertEqual(project.programming_language, 'py')
+        self.assertEqual(project.privacy_level, 'public')
+        self.assertEqual(project.project_url, 'http://template.readthedocs.io/')
         self.assertIn(self.me, project.users.all())
         self.assertEqual(project.builds.count(), 1)
 
@@ -565,3 +570,27 @@ def test_import_project(self):
             response_json,
             self._get_response_dict('projects-list_POST'),
         )
+
+    def test_import_project_with_extra_fields(self):
+        data = {
+            'name': 'Test Project',
+            'repository': {
+                'url': 'https://github.com/rtfd/template',
+                'type': 'git',
+            },
+            'default_version': 'v1.0',  # ignored: field not allowed
+        }
+
+        self.client.credentials(HTTP_AUTHORIZATION=f'Token {self.token.key}')
+        response = self.client.post(reverse('projects-list'), data)
+        self.assertEqual(response.status_code, 201)
+
+        query = Project.objects.filter(slug='test-project')
+        self.assertTrue(query.exists())
+
+        project = query.first()
+        self.assertEqual(project.name, 'Test Project')
+        self.assertEqual(project.slug, 'test-project')
+        self.assertEqual(project.repo, 'https://github.com/rtfd/template')
+        self.assertNotEqual(project.default_version, 'v1.0')
+        self.assertIn(self.me, project.users.all())