Merge branch 'pass-namespace-gitlab-project-import' into 'master'
Fix GitLab project import when a user has access only to their default namespace ## What does this MR do? It fixes a bug when a namespace ID was not passed to `/import/gitlab_project/new` page. It occurred when a user have no choice of the namespace, so we did not render the input for namespace ID. This MR introduces a hidden input for the described case. ## Does this MR meet the acceptance criteria? - Tests - [x] Added for this feature/bug - [x] All builds are passing - [x] Conform by the [merge request performance guides](http://docs.gitlab.com/ce/development/merge_request_performance_guidelines.html) - [x] Conform by the [style guides](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CONTRIBUTING.md#style-guides) - [x] Branch has no merge conflicts with `master` (if it does - rebase it please) - [x] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits) ## What are the relevant issue numbers? Fixes #23507 See merge request !6995
This commit is contained in:
commit
672ab77bf4
|
@ -27,6 +27,7 @@
|
|||
- else
|
||||
.input-group-addon.static-namespace
|
||||
#{root_url}#{current_user.username}/
|
||||
= f.hidden_field :namespace_id, value: current_user.namespace_id
|
||||
.form-group.col-xs-12.col-sm-6.project-path
|
||||
= f.label :namespace_id, class: 'label-light' do
|
||||
%span
|
||||
|
|
|
@ -3,13 +3,8 @@ require 'spec_helper'
|
|||
feature 'Import/Export - project import integration test', feature: true, js: true do
|
||||
include Select2Helper
|
||||
|
||||
let(:admin) { create(:admin) }
|
||||
let(:normal_user) { create(:user) }
|
||||
let!(:namespace) { create(:namespace, name: "asd", owner: admin) }
|
||||
let(:file) { File.join(Rails.root, 'spec', 'features', 'projects', 'import_export', 'test_project_export.tar.gz') }
|
||||
let(:export_path) { "#{Dir::tmpdir}/import_file_spec" }
|
||||
let(:project) { Project.last }
|
||||
let(:project_hook) { Gitlab::Git::Hook.new('post-receive', project.repository.path) }
|
||||
|
||||
background do
|
||||
allow_any_instance_of(Gitlab::ImportExport).to receive(:storage_path).and_return(export_path)
|
||||
|
@ -19,41 +14,43 @@ feature 'Import/Export - project import integration test', feature: true, js: tr
|
|||
FileUtils.rm_rf(export_path, secure: true)
|
||||
end
|
||||
|
||||
context 'admin user' do
|
||||
context 'when selecting the namespace' do
|
||||
let(:user) { create(:admin) }
|
||||
let!(:namespace) { create(:namespace, name: "asd", owner: user) }
|
||||
|
||||
before do
|
||||
login_as(admin)
|
||||
login_as(user)
|
||||
end
|
||||
|
||||
scenario 'user imports an exported project successfully' do
|
||||
expect(Project.all.count).to be_zero
|
||||
|
||||
visit new_project_path
|
||||
|
||||
select2('2', from: '#project_namespace_id')
|
||||
select2(namespace.id, from: '#project_namespace_id')
|
||||
fill_in :project_path, with: 'test-project-path', visible: true
|
||||
click_link 'GitLab export'
|
||||
|
||||
expect(page).to have_content('GitLab project export')
|
||||
expect(URI.parse(current_url).query).to eq('namespace_id=2&path=test-project-path')
|
||||
expect(URI.parse(current_url).query).to eq("namespace_id=#{namespace.id}&path=test-project-path")
|
||||
|
||||
attach_file('file', file)
|
||||
|
||||
click_on 'Import project' # import starts
|
||||
expect { click_on 'Import project' }.to change { Project.count }.from(0).to(1)
|
||||
|
||||
project = Project.last
|
||||
expect(project).not_to be_nil
|
||||
expect(project.issues).not_to be_empty
|
||||
expect(project.merge_requests).not_to be_empty
|
||||
expect(project_hook).to exist
|
||||
expect(wiki_exists?).to be true
|
||||
expect(project_hook_exists?(project)).to be true
|
||||
expect(wiki_exists?(project)).to be true
|
||||
expect(project.import_status).to eq('finished')
|
||||
end
|
||||
|
||||
scenario 'invalid project' do
|
||||
project = create(:project, namespace_id: 2)
|
||||
project = create(:project, namespace: namespace)
|
||||
|
||||
visit new_project_path
|
||||
|
||||
select2('2', from: '#project_namespace_id')
|
||||
select2(namespace.id, from: '#project_namespace_id')
|
||||
fill_in :project_path, with: project.name, visible: true
|
||||
click_link 'GitLab export'
|
||||
|
||||
|
@ -66,11 +63,11 @@ feature 'Import/Export - project import integration test', feature: true, js: tr
|
|||
end
|
||||
|
||||
scenario 'project with no name' do
|
||||
create(:project, namespace_id: 2)
|
||||
create(:project, namespace: namespace)
|
||||
|
||||
visit new_project_path
|
||||
|
||||
select2('2', from: '#project_namespace_id')
|
||||
select2(namespace.id, from: '#project_namespace_id')
|
||||
|
||||
# click on disabled element
|
||||
find(:link, 'GitLab export').trigger('click')
|
||||
|
@ -81,24 +78,30 @@ feature 'Import/Export - project import integration test', feature: true, js: tr
|
|||
end
|
||||
end
|
||||
|
||||
context 'normal user' do
|
||||
context 'when limited to the default user namespace' do
|
||||
let(:user) { create(:user) }
|
||||
before do
|
||||
login_as(normal_user)
|
||||
login_as(user)
|
||||
end
|
||||
|
||||
scenario 'non-admin user is allowed to import a project' do
|
||||
expect(Project.all.count).to be_zero
|
||||
|
||||
scenario 'passes correct namespace ID in the URL' do
|
||||
visit new_project_path
|
||||
|
||||
fill_in :project_path, with: 'test-project-path', visible: true
|
||||
|
||||
expect(page).to have_content('GitLab export')
|
||||
click_link 'GitLab export'
|
||||
|
||||
expect(page).to have_content('GitLab project export')
|
||||
expect(URI.parse(current_url).query).to eq("namespace_id=#{user.namespace.id}&path=test-project-path")
|
||||
end
|
||||
end
|
||||
|
||||
def wiki_exists?
|
||||
def wiki_exists?(project)
|
||||
wiki = ProjectWiki.new(project)
|
||||
File.exist?(wiki.repository.path_to_repo) && !wiki.repository.empty?
|
||||
end
|
||||
|
||||
def project_hook_exists?(project)
|
||||
Gitlab::Git::Hook.new('post-receive', project.repository.path).exists?
|
||||
end
|
||||
end
|
||||
|
|
Loading…
Reference in New Issue