ec4423665c
Direct disk access is done through Gitaly now, so the legacy path was deprecated. This path was used in Gitlab::Shell however. This required the refactoring in this commit. Added is the removal of direct path access on the project model, as that lookup wasn't needed anymore is most cases. Closes https://gitlab.com/gitlab-org/gitaly/issues/1111
269 lines
11 KiB
Ruby
269 lines
11 KiB
Ruby
require 'spec_helper'
|
|
|
|
describe Projects::ForkService do
|
|
include ProjectForksHelper
|
|
let(:gitlab_shell) { Gitlab::Shell.new }
|
|
context 'when forking a new project' do
|
|
describe 'fork by user' do
|
|
before do
|
|
@from_user = create(:user)
|
|
@from_namespace = @from_user.namespace
|
|
avatar = fixture_file_upload(Rails.root + "spec/fixtures/dk.png", "image/png")
|
|
@from_project = create(:project,
|
|
:repository,
|
|
creator_id: @from_user.id,
|
|
namespace: @from_namespace,
|
|
star_count: 107,
|
|
avatar: avatar,
|
|
description: 'wow such project')
|
|
@to_user = create(:user)
|
|
@to_namespace = @to_user.namespace
|
|
@from_project.add_user(@to_user, :developer)
|
|
end
|
|
|
|
context 'fork project' do
|
|
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
|
|
|
|
describe "successfully creates project in the user namespace" do
|
|
let(:to_project) { fork_project(@from_project, @to_user, namespace: @to_user.namespace) }
|
|
|
|
it { expect(to_project).to be_persisted }
|
|
it { expect(to_project.errors).to be_empty }
|
|
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) }
|
|
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
|
|
|
|
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
|
|
|
|
it 'creates a fork network with the new project and the root project set' do
|
|
to_project
|
|
fork_network = @from_project.reload.fork_network
|
|
|
|
expect(fork_network).not_to be_nil
|
|
expect(fork_network.root_project).to eq(@from_project)
|
|
expect(fork_network.projects).to contain_exactly(@from_project, to_project)
|
|
end
|
|
end
|
|
|
|
context 'creating a fork of a fork' do
|
|
let(:from_forked_project) { fork_project(@from_project, @to_user) }
|
|
let(:other_namespace) do
|
|
group = create(:group)
|
|
group.add_owner(@to_user)
|
|
group
|
|
end
|
|
let(:to_project) { fork_project(from_forked_project, @to_user, namespace: other_namespace) }
|
|
|
|
it 'sets the root of the network to the root project' do
|
|
expect(to_project.fork_network.root_project).to eq(@from_project)
|
|
end
|
|
|
|
it 'sets the forked_from_project on the membership' do
|
|
expect(to_project.fork_network_member.forked_from_project).to eq(from_forked_project)
|
|
end
|
|
end
|
|
end
|
|
|
|
context 'project already exists' do
|
|
it "fails due to validation, not transaction failure" do
|
|
@existing_project = create(:project, :repository, creator_id: @to_user.id, name: @from_project.name, namespace: @to_namespace)
|
|
@to_project = fork_project(@from_project, @to_user, namespace: @to_namespace)
|
|
expect(@existing_project).to be_persisted
|
|
|
|
expect(@to_project).not_to be_persisted
|
|
expect(@to_project.errors[:name]).to eq(['has already been taken'])
|
|
expect(@to_project.errors[:path]).to eq(['has already been taken'])
|
|
end
|
|
end
|
|
|
|
context 'repository already exists' do
|
|
let(:repository_storage) { 'default' }
|
|
let(:repository_storage_path) { Gitlab.config.repositories.storages[repository_storage].legacy_disk_path }
|
|
|
|
before do
|
|
gitlab_shell.create_repository(repository_storage, "#{@to_user.namespace.full_path}/#{@from_project.path}")
|
|
end
|
|
|
|
after do
|
|
gitlab_shell.remove_repository(repository_storage, "#{@to_user.namespace.full_path}/#{@from_project.path}")
|
|
end
|
|
|
|
it 'does not allow creation' do
|
|
to_project = fork_project(@from_project, @to_user, namespace: @to_user.namespace)
|
|
|
|
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
|
|
|
|
context 'GitLab CI is enabled' do
|
|
it "forks and enables CI for fork" do
|
|
@from_project.enable_ci
|
|
@to_project = fork_project(@from_project, @to_user)
|
|
expect(@to_project.builds_enabled?).to be_truthy
|
|
end
|
|
end
|
|
|
|
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 lowest level" do
|
|
forked_project = fork_project(@from_project, @to_user)
|
|
|
|
expect(forked_project.visibility_level).to eq(Gitlab::VisibilityLevel::PRIVATE)
|
|
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
|
|
end
|
|
|
|
describe 'fork to namespace' do
|
|
before do
|
|
@group_owner = create(:user)
|
|
@developer = create(:user)
|
|
@project = create(:project, :repository,
|
|
creator_id: @group_owner.id,
|
|
star_count: 777,
|
|
description: 'Wow, such a cool project!')
|
|
@group = create(:group)
|
|
@group.add_user(@group_owner, GroupMember::OWNER)
|
|
@group.add_user(@developer, GroupMember::DEVELOPER)
|
|
@project.add_user(@developer, :developer)
|
|
@project.add_user(@group_owner, :developer)
|
|
@opts = { namespace: @group }
|
|
end
|
|
|
|
context 'fork project for group' do
|
|
it 'group owner successfully forks project into the group' do
|
|
to_project = fork_project(@project, @group_owner, @opts)
|
|
|
|
expect(to_project).to be_persisted
|
|
expect(to_project.errors).to be_empty
|
|
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)
|
|
expect(to_project.star_count).to be_zero
|
|
end
|
|
end
|
|
|
|
context 'fork project for group when user not owner' do
|
|
it 'group developer fails to fork project into the group' do
|
|
to_project = fork_project(@project, @developer, @opts)
|
|
expect(to_project.errors[:namespace]).to eq(['is not valid'])
|
|
end
|
|
end
|
|
|
|
context 'project already exists in group' do
|
|
it 'fails due to validation, not transaction failure' do
|
|
existing_project = create(:project, :repository,
|
|
name: @project.name,
|
|
namespace: @group)
|
|
to_project = fork_project(@project, @group_owner, @opts)
|
|
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
|
|
|
|
context 'when the namespace has a lower visibility level than the project' do
|
|
it 'creates the project with the lower visibility level' do
|
|
public_project = create(:project, :public)
|
|
private_group = create(:group, :private)
|
|
group_owner = create(:user)
|
|
private_group.add_owner(group_owner)
|
|
|
|
forked_project = fork_project(public_project, group_owner, namespace: private_group)
|
|
|
|
expect(forked_project.visibility_level).to eq(Gitlab::VisibilityLevel::PRIVATE)
|
|
end
|
|
end
|
|
end
|
|
end
|
|
|
|
context 'when linking fork to an existing project' do
|
|
let(:fork_from_project) { create(:project, :public) }
|
|
let(:fork_to_project) { create(:project, :public) }
|
|
let(:user) { create(:user) }
|
|
|
|
subject { described_class.new(fork_from_project, user) }
|
|
|
|
def forked_from_project(project)
|
|
project.fork_network_member&.forked_from_project
|
|
end
|
|
|
|
context 'if project is already forked' do
|
|
it 'does not create fork relation' do
|
|
allow(fork_to_project).to receive(:forked?).and_return(true)
|
|
expect(forked_from_project(fork_to_project)).to be_nil
|
|
expect(subject.execute(fork_to_project)).to be_nil
|
|
expect(forked_from_project(fork_to_project)).to be_nil
|
|
end
|
|
end
|
|
|
|
context 'if project is not forked' do
|
|
it 'creates fork relation' do
|
|
expect(fork_to_project.forked?).to be false
|
|
expect(forked_from_project(fork_to_project)).to be_nil
|
|
|
|
subject.execute(fork_to_project)
|
|
|
|
expect(fork_to_project.forked?).to be true
|
|
expect(forked_from_project(fork_to_project)).to eq fork_from_project
|
|
expect(fork_to_project.forked_from_project).to eq fork_from_project
|
|
end
|
|
|
|
it 'flushes the forks count cache of the source project' do
|
|
expect(fork_from_project.forks_count).to be_zero
|
|
|
|
subject.execute(fork_to_project)
|
|
|
|
expect(fork_from_project.forks_count).to eq(1)
|
|
end
|
|
end
|
|
end
|
|
end
|