Exposing name property in imports APIexpose-name-property-in-import-api

Updating import API documentation. Updating import API tests.

4 files changed, 49 insertions, 5 deletions

diff --git a/changelogs/unreleased/expose-name-property-in-import-api.yml b/changelogs/unreleased/expose-name-property-in-import-api.yml
new file mode 100644
index 00000000000..43adfa05c0c
--- /dev/null
+++ b/changelogs/unreleased/expose-name-property-in-import-api.yml
@@ -0,0 +1,5 @@
+---
+title: Exposing name property in imports API
+merge_request: 32886
+author:
+type: added
diff --git a/doc/api/project_import_export.md b/doc/api/project_import_export.md
index 5155e996158..7e9926a4bae 100644
--- a/doc/api/project_import_export.md
+++ b/doc/api/project_import_export.md
@@ -111,6 +111,7 @@ POST /projects/import
 | Attribute | Type           | Required | Description                              |
 | --------- | -------------- | -------- | ---------------------------------------- |
 | `namespace` | integer/string | no | The ID or path of the namespace that the project will be imported to. Defaults to the current user's namespace |
+| `name` | string | no | The name of the project to be imported. Defaults to the path of the project if not provided |
 | `file` | string | yes | The file to be uploaded |
 | `path` | string | yes | Name and path for new project |
 | `overwrite` | boolean | no | If there is a project with the same path the import will overwrite it. Default to false |
@@ -131,14 +132,12 @@ cURL doesn't support posting a file from a remote server. Importing a project fr
 
 ```python
 import requests
-import urllib
-import json
-import sys
+from io import BytesIO
 
-s3_file = urllib.urlopen(presigned_url)
+s3_file = requests.get(presigned_url)
 
 url =  'https://gitlab.example.com/api/v4/projects/import'
-files = {'file': s3_file}
+files = {'file': BytesIO(s3_file.content)}
 data = {
     "path": "example-project",
     "namespace": "example-group"
diff --git a/lib/api/project_import.rb b/lib/api/project_import.rb
index 9b5e0727184..cb4ddd9bf44 100644
--- a/lib/api/project_import.rb
+++ b/lib/api/project_import.rb
@@ -29,6 +29,7 @@ module API
         requires :path, type: String, desc: 'The new project path and name'
         # TODO: remove rubocop disable - https://gitlab.com/gitlab-org/gitlab-ee/issues/14960
         requires :file, type: File, desc: 'The project export file to be imported' # rubocop:disable Scalability/FileUploads
+        optional :name, type: String, desc: 'The name of the project to be imported'
         optional :namespace, type: String, desc: "The ID or name of the namespace that the project will be imported into. Defaults to the current user's namespace."
         optional :overwrite, type: Boolean, default: false, desc: 'If there is a project in the same namespace and with the same name overwrite it'
         optional :override_params,
@@ -55,6 +56,7 @@ module API
         project_params = {
             path: import_params[:path],
             namespace_id: namespace.id,
+            name: import_params[:name],
             file: import_params[:file]['tempfile'],
             overwrite: import_params[:overwrite]
         }
diff --git a/spec/requests/api/project_import_spec.rb b/spec/requests/api/project_import_spec.rb
index 594b42bb6c0..bfd5e49e0e6 100644
--- a/spec/requests/api/project_import_spec.rb
+++ b/spec/requests/api/project_import_spec.rb
@@ -33,6 +33,44 @@ describe API::ProjectImport do
       expect(response).to have_gitlab_http_status(201)
     end
 
+    context 'when a name is explicitly set' do
+      let(:expected_name) { 'test project import' }
+
+      it 'schedules an import using a namespace and a different name' do
+        stub_import(namespace)
+
+        post api('/projects/import', user), params: { path: 'test-import', file: fixture_file_upload(file), namespace: namespace.id, name: expected_name }
+
+        expect(response).to have_gitlab_http_status(201)
+      end
+
+      it 'schedules an import using the namespace path and a different name' do
+        stub_import(namespace)
+
+        post api('/projects/import', user), params: { path: 'test-import', file: fixture_file_upload(file), namespace: namespace.full_path, name: expected_name }
+
+        expect(response).to have_gitlab_http_status(201)
+      end
+
+      it 'sets name correctly' do
+        stub_import(namespace)
+
+        post api('/projects/import', user), params: { path: 'test-import', file: fixture_file_upload(file), namespace: namespace.full_path, name: expected_name }
+
+        project = Project.find(json_response['id'])
+        expect(project.name).to eq(expected_name)
+      end
+
+      it 'sets name correctly with an overwrite' do
+        stub_import(namespace)
+
+        post api('/projects/import', user), params: { path: 'test-import', file: fixture_file_upload(file), namespace: namespace.full_path, name: 'new project name', overwrite: true }
+
+        project = Project.find(json_response['id'])
+        expect(project.name).to eq('new project name')
+      end
+    end
+
     it 'schedules an import at the user namespace level' do
       stub_import(user.namespace)