2015-04-26 12:48:37 +05:30
|
|
|
require 'spec_helper'
|
|
|
|
|
2017-09-10 17:25:29 +05:30
|
|
|
describe Projects::ForkService do
|
|
|
|
let(:gitlab_shell) { Gitlab::Shell.new }
|
|
|
|
|
2017-08-17 22:00:37 +05:30
|
|
|
describe 'fork by user' do
|
2015-04-26 12:48:37 +05:30
|
|
|
before do
|
2017-09-10 17:25:29 +05:30
|
|
|
@from_user = create(:user)
|
|
|
|
@from_namespace = @from_user.namespace
|
2017-08-17 22:00:37 +05:30
|
|
|
avatar = fixture_file_upload(Rails.root + "spec/fixtures/dk.png", "image/png")
|
2015-09-11 14:41:01 +05:30
|
|
|
@from_project = create(:project,
|
2017-08-17 22:00:37 +05:30
|
|
|
:repository,
|
2015-09-11 14:41:01 +05:30
|
|
|
creator_id: @from_user.id,
|
|
|
|
namespace: @from_namespace,
|
|
|
|
star_count: 107,
|
2017-08-17 22:00:37 +05:30
|
|
|
avatar: avatar,
|
2015-04-26 12:48:37 +05:30
|
|
|
description: 'wow such project')
|
2017-09-10 17:25:29 +05:30
|
|
|
@to_user = create(:user)
|
|
|
|
@to_namespace = @to_user.namespace
|
2016-10-01 15:18:49 +05:30
|
|
|
@from_project.add_user(@to_user, :developer)
|
2015-04-26 12:48:37 +05:30
|
|
|
end
|
|
|
|
|
|
|
|
context 'fork project' do
|
2016-10-01 15:18:49 +05:30
|
|
|
context 'when forker is a guest' do
|
|
|
|
before do
|
|
|
|
@guest = create(:user)
|
|
|
|
@from_project.add_user(@guest, :guest)
|
|
|
|
end
|
|
|
|
subject { fork_project(@from_project, @guest) }
|
|
|
|
|
|
|
|
it { is_expected.not_to be_persisted }
|
|
|
|
it { expect(subject.errors[:forked_from_project_id]).to eq(['is forbidden']) }
|
|
|
|
end
|
|
|
|
|
2015-04-26 12:48:37 +05:30
|
|
|
describe "successfully creates project in the user namespace" do
|
|
|
|
let(:to_project) { fork_project(@from_project, @to_user) }
|
|
|
|
|
2016-10-01 15:18:49 +05:30
|
|
|
it { expect(to_project).to be_persisted }
|
|
|
|
it { expect(to_project.errors).to be_empty }
|
2015-04-26 12:48:37 +05:30
|
|
|
it { expect(to_project.owner).to eq(@to_user) }
|
|
|
|
it { expect(to_project.namespace).to eq(@to_user.namespace) }
|
|
|
|
it { expect(to_project.star_count).to be_zero }
|
|
|
|
it { expect(to_project.description).to eq(@from_project.description) }
|
2017-08-17 22:00:37 +05:30
|
|
|
it { expect(to_project.avatar.file).to be_exists }
|
|
|
|
|
|
|
|
# This test is here because we had a bug where the from-project lost its
|
|
|
|
# avatar after being forked.
|
|
|
|
# https://gitlab.com/gitlab-org/gitlab-ce/issues/26158
|
|
|
|
it "after forking the from-project still has its avatar" do
|
|
|
|
# If we do not fork the project first we cannot detect the bug.
|
|
|
|
expect(to_project).to be_persisted
|
|
|
|
|
|
|
|
expect(@from_project.avatar.file).to be_exists
|
|
|
|
end
|
2017-09-10 17:25:29 +05:30
|
|
|
|
|
|
|
it 'flushes the forks count cache of the source project' do
|
|
|
|
expect(@from_project.forks_count).to be_zero
|
|
|
|
|
|
|
|
fork_project(@from_project, @to_user)
|
|
|
|
|
|
|
|
expect(@from_project.forks_count).to eq(1)
|
|
|
|
end
|
2015-04-26 12:48:37 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'project already exists' do
|
2016-09-13 17:45:13 +05:30
|
|
|
it "fails due to validation, not transaction failure" do
|
2017-08-17 22:00:37 +05:30
|
|
|
@existing_project = create(:project, :repository, creator_id: @to_user.id, name: @from_project.name, namespace: @to_namespace)
|
2015-04-26 12:48:37 +05:30
|
|
|
@to_project = fork_project(@from_project, @to_user)
|
2016-10-01 15:18:49 +05:30
|
|
|
expect(@existing_project).to be_persisted
|
|
|
|
|
|
|
|
expect(@to_project).not_to be_persisted
|
2015-09-11 14:41:01 +05:30
|
|
|
expect(@to_project.errors[:name]).to eq(['has already been taken'])
|
|
|
|
expect(@to_project.errors[:path]).to eq(['has already been taken'])
|
2015-04-26 12:48:37 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2017-09-10 17:25:29 +05:30
|
|
|
context 'repository already exists' do
|
|
|
|
let(:repository_storage_path) { Gitlab.config.repositories.storages['default']['path'] }
|
|
|
|
|
|
|
|
before do
|
|
|
|
gitlab_shell.add_repository(repository_storage_path, "#{@to_user.namespace.full_path}/#{@from_project.path}")
|
|
|
|
end
|
|
|
|
|
|
|
|
after do
|
|
|
|
gitlab_shell.remove_repository(repository_storage_path, "#{@to_user.namespace.full_path}/#{@from_project.path}")
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'does not allow creation' do
|
|
|
|
to_project = fork_project(@from_project, @to_user)
|
|
|
|
|
|
|
|
expect(to_project).not_to be_persisted
|
|
|
|
expect(to_project.errors.messages).to have_key(:base)
|
|
|
|
expect(to_project.errors.messages[:base].first).to match('There is already a repository with that name on disk')
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2015-04-26 12:48:37 +05:30
|
|
|
context 'GitLab CI is enabled' do
|
2016-09-13 17:45:13 +05:30
|
|
|
it "forks and enables CI for fork" do
|
2015-10-24 18:46:33 +05:30
|
|
|
@from_project.enable_ci
|
|
|
|
@to_project = fork_project(@from_project, @to_user)
|
2015-11-26 14:37:03 +05:30
|
|
|
expect(@to_project.builds_enabled?).to be_truthy
|
2015-04-26 12:48:37 +05:30
|
|
|
end
|
|
|
|
end
|
2016-06-16 23:09:34 +05:30
|
|
|
|
|
|
|
context "when project has restricted visibility level" do
|
|
|
|
context "and only one visibility level is restricted" do
|
|
|
|
before do
|
|
|
|
@from_project.update_attributes(visibility_level: Gitlab::VisibilityLevel::INTERNAL)
|
|
|
|
stub_application_setting(restricted_visibility_levels: [Gitlab::VisibilityLevel::INTERNAL])
|
|
|
|
end
|
|
|
|
|
|
|
|
it "creates fork with highest allowed level" do
|
|
|
|
forked_project = fork_project(@from_project, @to_user)
|
|
|
|
|
|
|
|
expect(forked_project.visibility_level).to eq(Gitlab::VisibilityLevel::PUBLIC)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context "and all visibility levels are restricted" do
|
|
|
|
before do
|
|
|
|
stub_application_setting(restricted_visibility_levels: [Gitlab::VisibilityLevel::PUBLIC, Gitlab::VisibilityLevel::INTERNAL, Gitlab::VisibilityLevel::PRIVATE])
|
|
|
|
end
|
|
|
|
|
|
|
|
it "creates fork with private visibility levels" do
|
|
|
|
forked_project = fork_project(@from_project, @to_user)
|
|
|
|
|
|
|
|
expect(forked_project.visibility_level).to eq(Gitlab::VisibilityLevel::PRIVATE)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
2015-04-26 12:48:37 +05:30
|
|
|
end
|
|
|
|
|
2017-08-17 22:00:37 +05:30
|
|
|
describe 'fork to namespace' do
|
2015-04-26 12:48:37 +05:30
|
|
|
before do
|
|
|
|
@group_owner = create(:user)
|
|
|
|
@developer = create(:user)
|
2017-08-17 22:00:37 +05:30
|
|
|
@project = create(:project, :repository,
|
|
|
|
creator_id: @group_owner.id,
|
|
|
|
star_count: 777,
|
|
|
|
description: 'Wow, such a cool project!')
|
2015-04-26 12:48:37 +05:30
|
|
|
@group = create(:group)
|
|
|
|
@group.add_user(@group_owner, GroupMember::OWNER)
|
|
|
|
@group.add_user(@developer, GroupMember::DEVELOPER)
|
2016-10-01 15:18:49 +05:30
|
|
|
@project.add_user(@developer, :developer)
|
|
|
|
@project.add_user(@group_owner, :developer)
|
2015-04-26 12:48:37 +05:30
|
|
|
@opts = { namespace: @group }
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'fork project for group' do
|
|
|
|
it 'group owner successfully forks project into the group' do
|
2015-11-26 14:37:03 +05:30
|
|
|
to_project = fork_project(@project, @group_owner, @opts)
|
2016-10-01 15:18:49 +05:30
|
|
|
|
|
|
|
expect(to_project).to be_persisted
|
|
|
|
expect(to_project.errors).to be_empty
|
2015-04-26 12:48:37 +05:30
|
|
|
expect(to_project.owner).to eq(@group)
|
|
|
|
expect(to_project.namespace).to eq(@group)
|
|
|
|
expect(to_project.name).to eq(@project.name)
|
|
|
|
expect(to_project.path).to eq(@project.path)
|
|
|
|
expect(to_project.description).to eq(@project.description)
|
2016-10-01 15:18:49 +05:30
|
|
|
expect(to_project.star_count).to be_zero
|
2015-04-26 12:48:37 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'fork project for group when user not owner' do
|
2016-09-13 17:45:13 +05:30
|
|
|
it 'group developer fails to fork project into the group' do
|
2015-11-26 14:37:03 +05:30
|
|
|
to_project = fork_project(@project, @developer, @opts)
|
2015-09-11 14:41:01 +05:30
|
|
|
expect(to_project.errors[:namespace]).to eq(['is not valid'])
|
2015-04-26 12:48:37 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'project already exists in group' do
|
2016-09-13 17:45:13 +05:30
|
|
|
it 'fails due to validation, not transaction failure' do
|
2017-08-17 22:00:37 +05:30
|
|
|
existing_project = create(:project, :repository,
|
|
|
|
name: @project.name,
|
|
|
|
namespace: @group)
|
2015-11-26 14:37:03 +05:30
|
|
|
to_project = fork_project(@project, @group_owner, @opts)
|
2015-04-26 12:48:37 +05:30
|
|
|
expect(existing_project.persisted?).to be_truthy
|
|
|
|
expect(to_project.errors[:name]).to eq(['has already been taken'])
|
|
|
|
expect(to_project.errors[:path]).to eq(['has already been taken'])
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2015-11-26 14:37:03 +05:30
|
|
|
def fork_project(from_project, user, params = {})
|
|
|
|
allow(RepositoryForkWorker).to receive(:perform_async).and_return(true)
|
2015-09-11 14:41:01 +05:30
|
|
|
Projects::ForkService.new(from_project, user, params).execute
|
2015-04-26 12:48:37 +05:30
|
|
|
end
|
|
|
|
end
|