2015-04-26 12:48:37 +05:30
|
|
|
require 'spec_helper'
|
|
|
|
|
2015-12-23 02:04:40 +05:30
|
|
|
describe Repository, models: true do
|
2015-04-26 12:48:37 +05:30
|
|
|
include RepoHelpers
|
|
|
|
|
|
|
|
let(:repository) { create(:project).repository }
|
2015-12-23 02:04:40 +05:30
|
|
|
let(:user) { create(:user) }
|
2016-04-02 18:10:28 +05:30
|
|
|
let(:commit_options) do
|
|
|
|
author = repository.user_to_committer(user)
|
|
|
|
{ message: 'Test message', committer: author, author: author }
|
|
|
|
end
|
|
|
|
let(:merge_commit) do
|
|
|
|
source_sha = repository.find_branch('feature').target
|
|
|
|
merge_commit_id = repository.merge(user, source_sha, 'master', commit_options)
|
|
|
|
repository.commit(merge_commit_id)
|
|
|
|
end
|
2015-04-26 12:48:37 +05:30
|
|
|
|
|
|
|
describe :branch_names_contains do
|
|
|
|
subject { repository.branch_names_contains(sample_commit.id) }
|
|
|
|
|
|
|
|
it { is_expected.to include('master') }
|
|
|
|
it { is_expected.not_to include('feature') }
|
|
|
|
it { is_expected.not_to include('fix') }
|
|
|
|
end
|
|
|
|
|
|
|
|
describe :tag_names_contains do
|
|
|
|
subject { repository.tag_names_contains(sample_commit.id) }
|
|
|
|
|
|
|
|
it { is_expected.to include('v1.1.0') }
|
|
|
|
it { is_expected.not_to include('v1.0.0') }
|
|
|
|
end
|
|
|
|
|
|
|
|
describe :last_commit_for_path do
|
|
|
|
subject { repository.last_commit_for_path(sample_commit.id, '.gitignore').id }
|
|
|
|
|
|
|
|
it { is_expected.to eq('c1acaa58bbcbc3eafe538cb8274ba387047b69f8') }
|
|
|
|
end
|
2015-09-11 14:41:01 +05:30
|
|
|
|
2015-11-26 14:37:03 +05:30
|
|
|
describe :find_commits_by_message do
|
|
|
|
subject { repository.find_commits_by_message('submodule').map{ |k| k.id } }
|
|
|
|
|
|
|
|
it { is_expected.to include('5937ac0a7beb003549fc5fd26fc247adbce4a52e') }
|
|
|
|
it { is_expected.to include('6f6d7e7ed97bb5f0054f2b1df789b39ca89b6ff9') }
|
|
|
|
it { is_expected.to include('cfe32cf61b73a0d5e9f13e774abde7ff789b1660') }
|
|
|
|
it { is_expected.not_to include('913c66a37b4a45b9769037c55c2d238bd0942d2e') }
|
|
|
|
end
|
|
|
|
|
2015-09-11 14:41:01 +05:30
|
|
|
describe :blob_at do
|
|
|
|
context 'blank sha' do
|
|
|
|
subject { repository.blob_at(Gitlab::Git::BLANK_SHA, '.gitignore') }
|
|
|
|
|
|
|
|
it { is_expected.to be_nil }
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe :merged_to_root_ref? do
|
|
|
|
context 'merged branch' do
|
|
|
|
subject { repository.merged_to_root_ref?('improve/awesome') }
|
|
|
|
|
|
|
|
it { is_expected.to be_truthy }
|
|
|
|
end
|
2015-09-25 12:07:36 +05:30
|
|
|
end
|
|
|
|
|
|
|
|
describe :can_be_merged? do
|
|
|
|
context 'mergeable branches' do
|
|
|
|
subject { repository.can_be_merged?('0b4bc9a49b562e85de7cc9e834518ea6828729b9', 'master') }
|
|
|
|
|
|
|
|
it { is_expected.to be_truthy }
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'non-mergeable branches' do
|
|
|
|
subject { repository.can_be_merged?('bb5206fee213d983da88c47f9cf4cc6caf9c66dc', 'feature') }
|
|
|
|
|
|
|
|
it { is_expected.to be_falsey }
|
|
|
|
end
|
2015-09-11 14:41:01 +05:30
|
|
|
|
|
|
|
context 'non merged branch' do
|
|
|
|
subject { repository.merged_to_root_ref?('fix') }
|
|
|
|
|
|
|
|
it { is_expected.to be_falsey }
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'non existent branch' do
|
|
|
|
subject { repository.merged_to_root_ref?('non_existent_branch') }
|
|
|
|
|
|
|
|
it { is_expected.to be_nil }
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe "search_files" do
|
|
|
|
let(:results) { repository.search_files('feature', 'master') }
|
|
|
|
subject { results }
|
|
|
|
|
|
|
|
it { is_expected.to be_an Array }
|
|
|
|
|
|
|
|
describe 'result' do
|
|
|
|
subject { results.first }
|
|
|
|
|
|
|
|
it { is_expected.to be_an String }
|
|
|
|
it { expect(subject.lines[2]).to eq("master:CHANGELOG:188: - Feature: Replace teams with group membership\n") }
|
|
|
|
end
|
|
|
|
|
|
|
|
describe 'parsing result' do
|
|
|
|
subject { repository.parse_search_result(results.first) }
|
|
|
|
|
|
|
|
it { is_expected.to be_an OpenStruct }
|
|
|
|
it { expect(subject.filename).to eq('CHANGELOG') }
|
|
|
|
it { expect(subject.ref).to eq('master') }
|
|
|
|
it { expect(subject.startline).to eq(186) }
|
|
|
|
it { expect(subject.data.lines[2]).to eq(" - Feature: Replace teams with group membership\n") }
|
|
|
|
end
|
2015-12-23 02:04:40 +05:30
|
|
|
|
|
|
|
end
|
|
|
|
|
|
|
|
describe "#license" do
|
|
|
|
before do
|
|
|
|
repository.send(:cache).expire(:license)
|
|
|
|
TestBlob = Struct.new(:name)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'test selection preference' do
|
|
|
|
files = [TestBlob.new('file'), TestBlob.new('license'), TestBlob.new('copying')]
|
|
|
|
expect(repository.tree).to receive(:blobs).and_return(files)
|
|
|
|
|
|
|
|
expect(repository.license.name).to eq('license')
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'also accepts licence instead of license' do
|
|
|
|
expect(repository.tree).to receive(:blobs).and_return([TestBlob.new('licence')])
|
|
|
|
|
|
|
|
expect(repository.license.name).to eq('licence')
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe :add_branch do
|
|
|
|
context 'when pre hooks were successful' do
|
|
|
|
it 'should run without errors' do
|
|
|
|
hook = double(trigger: true)
|
|
|
|
expect(Gitlab::Git::Hook).to receive(:new).exactly(3).times.and_return(hook)
|
|
|
|
|
|
|
|
expect { repository.add_branch(user, 'new_feature', 'master') }.not_to raise_error
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'should create the branch' do
|
|
|
|
allow_any_instance_of(Gitlab::Git::Hook).to receive(:trigger).and_return(true)
|
|
|
|
|
|
|
|
branch = repository.add_branch(user, 'new_feature', 'master')
|
|
|
|
|
|
|
|
expect(branch.name).to eq('new_feature')
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when pre hooks failed' do
|
|
|
|
it 'should get an error' do
|
|
|
|
allow_any_instance_of(Gitlab::Git::Hook).to receive(:trigger).and_return(false)
|
|
|
|
|
|
|
|
expect do
|
|
|
|
repository.add_branch(user, 'new_feature', 'master')
|
|
|
|
end.to raise_error(GitHooksService::PreReceiveError)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'should not create the branch' do
|
|
|
|
allow_any_instance_of(Gitlab::Git::Hook).to receive(:trigger).and_return(false)
|
|
|
|
|
|
|
|
expect do
|
|
|
|
repository.add_branch(user, 'new_feature', 'master')
|
|
|
|
end.to raise_error(GitHooksService::PreReceiveError)
|
|
|
|
expect(repository.find_branch('new_feature')).to be_nil
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe :rm_branch do
|
|
|
|
context 'when pre hooks were successful' do
|
|
|
|
it 'should run without errors' do
|
|
|
|
allow_any_instance_of(Gitlab::Git::Hook).to receive(:trigger).and_return(true)
|
|
|
|
|
|
|
|
expect { repository.rm_branch(user, 'feature') }.not_to raise_error
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'should delete the branch' do
|
|
|
|
allow_any_instance_of(Gitlab::Git::Hook).to receive(:trigger).and_return(true)
|
|
|
|
|
|
|
|
expect { repository.rm_branch(user, 'feature') }.not_to raise_error
|
|
|
|
|
|
|
|
expect(repository.find_branch('feature')).to be_nil
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when pre hooks failed' do
|
|
|
|
it 'should get an error' do
|
|
|
|
allow_any_instance_of(Gitlab::Git::Hook).to receive(:trigger).and_return(false)
|
|
|
|
|
|
|
|
expect do
|
|
|
|
repository.rm_branch(user, 'new_feature')
|
|
|
|
end.to raise_error(GitHooksService::PreReceiveError)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'should not delete the branch' do
|
|
|
|
allow_any_instance_of(Gitlab::Git::Hook).to receive(:trigger).and_return(false)
|
|
|
|
|
|
|
|
expect do
|
|
|
|
repository.rm_branch(user, 'feature')
|
|
|
|
end.to raise_error(GitHooksService::PreReceiveError)
|
|
|
|
expect(repository.find_branch('feature')).not_to be_nil
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe :commit_with_hooks do
|
|
|
|
context 'when pre hooks were successful' do
|
2016-04-02 18:10:28 +05:30
|
|
|
before do
|
|
|
|
expect_any_instance_of(GitHooksService).to receive(:execute).
|
|
|
|
and_return(true)
|
|
|
|
end
|
2015-12-23 02:04:40 +05:30
|
|
|
|
2016-04-02 18:10:28 +05:30
|
|
|
it 'should run without errors' do
|
2015-12-23 02:04:40 +05:30
|
|
|
expect do
|
|
|
|
repository.commit_with_hooks(user, 'feature') { sample_commit.id }
|
|
|
|
end.not_to raise_error
|
|
|
|
end
|
2016-04-02 18:10:28 +05:30
|
|
|
|
|
|
|
it 'should ensure the autocrlf Git option is set to :input' do
|
|
|
|
expect(repository).to receive(:update_autocrlf_option)
|
|
|
|
|
|
|
|
repository.commit_with_hooks(user, 'feature') { sample_commit.id }
|
|
|
|
end
|
2015-12-23 02:04:40 +05:30
|
|
|
end
|
|
|
|
|
|
|
|
context 'when pre hooks failed' do
|
|
|
|
it 'should get an error' do
|
|
|
|
allow_any_instance_of(Gitlab::Git::Hook).to receive(:trigger).and_return(false)
|
|
|
|
|
|
|
|
expect do
|
|
|
|
repository.commit_with_hooks(user, 'feature') { sample_commit.id }
|
|
|
|
end.to raise_error(GitHooksService::PreReceiveError)
|
|
|
|
end
|
|
|
|
end
|
2015-09-11 14:41:01 +05:30
|
|
|
end
|
2016-02-05 20:25:01 +05:30
|
|
|
|
2016-04-02 18:10:28 +05:30
|
|
|
describe '#exists?' do
|
|
|
|
it 'returns true when a repository exists' do
|
|
|
|
expect(repository.exists?).to eq(true)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns false when a repository does not exist' do
|
|
|
|
expect(repository.raw_repository).to receive(:rugged).
|
|
|
|
and_raise(Gitlab::Git::Repository::NoRepository)
|
|
|
|
|
|
|
|
expect(repository.exists?).to eq(false)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns false when there is no namespace' do
|
|
|
|
allow(repository).to receive(:path_with_namespace).and_return(nil)
|
|
|
|
|
|
|
|
expect(repository.exists?).to eq(false)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2016-02-05 20:25:01 +05:30
|
|
|
describe '#has_visible_content?' do
|
|
|
|
subject { repository.has_visible_content? }
|
|
|
|
|
|
|
|
describe 'when there are no branches' do
|
|
|
|
before do
|
|
|
|
allow(repository.raw_repository).to receive(:branch_count).and_return(0)
|
|
|
|
end
|
|
|
|
|
|
|
|
it { is_expected.to eq(false) }
|
|
|
|
end
|
|
|
|
|
|
|
|
describe 'when there are branches' do
|
2016-04-02 18:10:28 +05:30
|
|
|
it 'returns true' do
|
|
|
|
expect(repository.raw_repository).to receive(:branch_count).and_return(3)
|
|
|
|
|
|
|
|
expect(subject).to eq(true)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'caches the output' do
|
|
|
|
expect(repository.raw_repository).to receive(:branch_count).
|
|
|
|
once.
|
|
|
|
and_return(3)
|
|
|
|
|
|
|
|
repository.has_visible_content?
|
|
|
|
repository.has_visible_content?
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe '#update_autocrlf_option' do
|
|
|
|
describe 'when autocrlf is not already set to :input' do
|
2016-02-05 20:25:01 +05:30
|
|
|
before do
|
2016-04-02 18:10:28 +05:30
|
|
|
repository.raw_repository.autocrlf = true
|
2016-02-05 20:25:01 +05:30
|
|
|
end
|
|
|
|
|
2016-04-02 18:10:28 +05:30
|
|
|
it 'sets autocrlf to :input' do
|
|
|
|
repository.update_autocrlf_option
|
|
|
|
|
|
|
|
expect(repository.raw_repository.autocrlf).to eq(:input)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe 'when autocrlf is already set to :input' do
|
|
|
|
before do
|
|
|
|
repository.raw_repository.autocrlf = :input
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'does nothing' do
|
|
|
|
expect(repository.raw_repository).to_not receive(:autocrlf=).
|
|
|
|
with(:input)
|
|
|
|
|
|
|
|
repository.update_autocrlf_option
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe '#empty?' do
|
|
|
|
let(:empty_repository) { create(:project_empty_repo).repository }
|
|
|
|
|
|
|
|
it 'returns true for an empty repository' do
|
|
|
|
expect(empty_repository.empty?).to eq(true)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns false for a non-empty repository' do
|
|
|
|
expect(repository.empty?).to eq(false)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'caches the output' do
|
|
|
|
expect(repository.raw_repository).to receive(:empty?).
|
|
|
|
once.
|
|
|
|
and_return(false)
|
|
|
|
|
|
|
|
repository.empty?
|
|
|
|
repository.empty?
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe '#root_ref' do
|
|
|
|
it 'returns a branch name' do
|
|
|
|
expect(repository.root_ref).to be_an_instance_of(String)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'caches the output' do
|
|
|
|
expect(repository.raw_repository).to receive(:root_ref).
|
|
|
|
once.
|
|
|
|
and_return('master')
|
|
|
|
|
|
|
|
repository.root_ref
|
|
|
|
repository.root_ref
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe '#expire_cache' do
|
|
|
|
it 'expires all caches' do
|
|
|
|
expect(repository).to receive(:expire_branch_cache)
|
|
|
|
|
|
|
|
repository.expire_cache
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'expires the caches for a specific branch' do
|
|
|
|
expect(repository).to receive(:expire_branch_cache).with('master')
|
|
|
|
|
|
|
|
repository.expire_cache('master')
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'expires the emptiness cache for an empty repository' do
|
|
|
|
expect(repository).to receive(:empty?).and_return(true)
|
|
|
|
expect(repository).to receive(:expire_emptiness_caches)
|
|
|
|
|
|
|
|
repository.expire_cache
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'does not expire the emptiness cache for a non-empty repository' do
|
|
|
|
expect(repository).to receive(:empty?).and_return(false)
|
|
|
|
expect(repository).to_not receive(:expire_emptiness_caches)
|
|
|
|
|
|
|
|
repository.expire_cache
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe '#expire_root_ref_cache' do
|
|
|
|
it 'expires the root reference cache' do
|
|
|
|
repository.root_ref
|
|
|
|
|
|
|
|
expect(repository.raw_repository).to receive(:root_ref).
|
|
|
|
once.
|
|
|
|
and_return('foo')
|
|
|
|
|
|
|
|
repository.expire_root_ref_cache
|
|
|
|
|
|
|
|
expect(repository.root_ref).to eq('foo')
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe '#expire_has_visible_content_cache' do
|
|
|
|
it 'expires the visible content cache' do
|
|
|
|
repository.has_visible_content?
|
|
|
|
|
|
|
|
expect(repository.raw_repository).to receive(:branch_count).
|
|
|
|
once.
|
|
|
|
and_return(0)
|
|
|
|
|
|
|
|
repository.expire_has_visible_content_cache
|
|
|
|
|
|
|
|
expect(repository.has_visible_content?).to eq(false)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe '#expire_branch_ache' do
|
|
|
|
# This method is private but we need it for testing purposes. Sadly there's
|
|
|
|
# no other proper way of testing caching operations.
|
|
|
|
let(:cache) { repository.send(:cache) }
|
|
|
|
|
|
|
|
it 'expires the cache for all branches' do
|
|
|
|
expect(cache).to receive(:expire).
|
|
|
|
at_least(repository.branches.length).
|
|
|
|
times
|
|
|
|
|
|
|
|
repository.expire_branch_cache
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'expires the cache for all branches when the root branch is given' do
|
|
|
|
expect(cache).to receive(:expire).
|
|
|
|
at_least(repository.branches.length).
|
|
|
|
times
|
|
|
|
|
|
|
|
repository.expire_branch_cache(repository.root_ref)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'expires the cache for a specific branch' do
|
|
|
|
expect(cache).to receive(:expire).once
|
|
|
|
|
|
|
|
repository.expire_branch_cache('foo')
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe '#expire_emptiness_caches' do
|
|
|
|
let(:cache) { repository.send(:cache) }
|
|
|
|
|
|
|
|
it 'expires the caches' do
|
|
|
|
expect(cache).to receive(:expire).with(:empty?)
|
|
|
|
expect(repository).to receive(:expire_has_visible_content_cache)
|
|
|
|
|
|
|
|
repository.expire_emptiness_caches
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe :skip_merged_commit do
|
|
|
|
subject { repository.commits(Gitlab::Git::BRANCH_REF_PREFIX + "'test'", nil, 100, 0, true).map{ |k| k.id } }
|
|
|
|
|
|
|
|
it { is_expected.not_to include('e56497bb5f03a90a51293fc6d516788730953899') }
|
|
|
|
end
|
|
|
|
|
|
|
|
describe '#merge' do
|
|
|
|
it 'should merge the code and return the commit id' do
|
|
|
|
expect(merge_commit).to be_present
|
|
|
|
expect(repository.blob_at(merge_commit.id, 'files/ruby/feature.rb')).to be_present
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe '#revert_merge' do
|
|
|
|
it 'should revert the changes' do
|
|
|
|
repository.revert(user, merge_commit, 'master')
|
|
|
|
|
|
|
|
expect(repository.blob_at_branch('master', 'files/ruby/feature.rb')).not_to be_present
|
2016-02-05 20:25:01 +05:30
|
|
|
end
|
|
|
|
end
|
2015-04-26 12:48:37 +05:30
|
|
|
end
|