2014-09-02 18:07:02 +05:30
|
|
|
require 'spec_helper'
|
|
|
|
|
2017-09-10 17:25:29 +05:30
|
|
|
describe ExtractsPath do
|
|
|
|
include described_class
|
2015-09-11 14:41:01 +05:30
|
|
|
include RepoHelpers
|
2017-09-10 17:25:29 +05:30
|
|
|
include Gitlab::Routing
|
2014-09-02 18:07:02 +05:30
|
|
|
|
|
|
|
let(:project) { double('project') }
|
2016-11-03 12:29:30 +05:30
|
|
|
let(:request) { double('request') }
|
2014-09-02 18:07:02 +05:30
|
|
|
|
|
|
|
before do
|
|
|
|
@project = project
|
2015-09-11 14:41:01 +05:30
|
|
|
|
|
|
|
repo = double(ref_names: ['master', 'foo/bar/baz', 'v1.0.0', 'v2.0.0',
|
|
|
|
'release/app', 'release/app/v1.0.0'])
|
|
|
|
allow(project).to receive(:repository).and_return(repo)
|
2017-09-10 17:25:29 +05:30
|
|
|
allow(project).to receive(:full_path)
|
|
|
|
.and_return('gitlab/gitlab-ci')
|
2016-11-03 12:29:30 +05:30
|
|
|
allow(request).to receive(:format=)
|
2015-09-11 14:41:01 +05:30
|
|
|
end
|
|
|
|
|
2016-11-03 12:29:30 +05:30
|
|
|
describe '#assign_ref_vars' do
|
2015-09-11 14:41:01 +05:30
|
|
|
let(:ref) { sample_commit[:id] }
|
|
|
|
let(:params) { { path: sample_commit[:line_code_path], ref: ref } }
|
|
|
|
|
|
|
|
before do
|
2017-08-17 22:00:37 +05:30
|
|
|
@project = create(:project, :repository)
|
2015-09-11 14:41:01 +05:30
|
|
|
end
|
|
|
|
|
2016-09-13 17:45:13 +05:30
|
|
|
it "log tree path has no escape sequences" do
|
2015-09-11 14:41:01 +05:30
|
|
|
assign_ref_vars
|
2017-09-10 17:25:29 +05:30
|
|
|
expect(@logs_path).to eq("/#{@project.full_path}/refs/#{ref}/logs_tree/files/ruby/popen.rb")
|
2015-09-11 14:41:01 +05:30
|
|
|
end
|
|
|
|
|
2016-09-13 17:45:13 +05:30
|
|
|
context 'ref contains %20' do
|
|
|
|
let(:ref) { 'foo%20bar' }
|
2015-09-11 14:41:01 +05:30
|
|
|
|
2016-09-13 17:45:13 +05:30
|
|
|
it 'is not converted to a space in @id' do
|
|
|
|
@project.repository.add_branch(@project.owner, 'foo%20bar', 'master')
|
|
|
|
|
|
|
|
assign_ref_vars
|
|
|
|
|
|
|
|
expect(@id).to start_with('foo%20bar/')
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'path contains space' do
|
|
|
|
let(:params) { { path: 'with space', ref: '38008cb17ce1466d8fec2dfa6f6ab8dcfe5cf49e' } }
|
|
|
|
|
|
|
|
it 'is not converted to %20 in @path' do
|
2015-09-11 14:41:01 +05:30
|
|
|
assign_ref_vars
|
2016-09-13 17:45:13 +05:30
|
|
|
|
|
|
|
expect(@path).to eq(params[:path])
|
2015-09-11 14:41:01 +05:30
|
|
|
end
|
|
|
|
end
|
2016-09-29 09:46:39 +05:30
|
|
|
|
|
|
|
context 'subclass overrides get_id' do
|
|
|
|
it 'uses ref returned by get_id' do
|
2018-03-17 18:26:18 +05:30
|
|
|
allow_any_instance_of(self.class).to receive(:get_id) { '38008cb17ce1466d8fec2dfa6f6ab8dcfe5cf49e' }
|
2016-09-29 09:46:39 +05:30
|
|
|
|
|
|
|
assign_ref_vars
|
|
|
|
|
|
|
|
expect(@id).to eq(get_id)
|
|
|
|
end
|
|
|
|
end
|
2016-11-03 12:29:30 +05:30
|
|
|
|
|
|
|
context 'ref only exists without .atom suffix' do
|
|
|
|
context 'with a path' do
|
|
|
|
let(:params) { { ref: 'v1.0.0.atom', path: 'README.md' } }
|
|
|
|
|
|
|
|
it 'renders a 404' do
|
|
|
|
expect(self).to receive(:render_404)
|
|
|
|
|
|
|
|
assign_ref_vars
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'without a path' do
|
|
|
|
let(:params) { { ref: 'v1.0.0.atom' } }
|
2017-09-10 17:25:29 +05:30
|
|
|
|
|
|
|
before do
|
|
|
|
assign_ref_vars
|
|
|
|
end
|
2016-11-03 12:29:30 +05:30
|
|
|
|
|
|
|
it 'sets the un-suffixed version as @ref' do
|
|
|
|
expect(@ref).to eq('v1.0.0')
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'sets the request format to Atom' do
|
|
|
|
expect(request).to have_received(:format=).with(:atom)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'ref exists with .atom suffix' do
|
|
|
|
context 'with a path' do
|
|
|
|
let(:params) { { ref: 'master.atom', path: 'README.md' } }
|
|
|
|
|
|
|
|
before do
|
|
|
|
repository = @project.repository
|
|
|
|
allow(repository).to receive(:commit).and_call_original
|
|
|
|
allow(repository).to receive(:commit).with('master.atom').and_return(repository.commit('master'))
|
|
|
|
|
|
|
|
assign_ref_vars
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'sets the suffixed version as @ref' do
|
|
|
|
expect(@ref).to eq('master.atom')
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'does not change the request format' do
|
|
|
|
expect(request).not_to have_received(:format=)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'without a path' do
|
|
|
|
let(:params) { { ref: 'master.atom' } }
|
|
|
|
|
|
|
|
before do
|
|
|
|
repository = @project.repository
|
|
|
|
allow(repository).to receive(:commit).and_call_original
|
|
|
|
allow(repository).to receive(:commit).with('master.atom').and_return(repository.commit('master'))
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'sets the suffixed version as @ref' do
|
|
|
|
assign_ref_vars
|
|
|
|
|
|
|
|
expect(@ref).to eq('master.atom')
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'does not change the request format' do
|
|
|
|
expect(request).not_to receive(:format=)
|
|
|
|
|
|
|
|
assign_ref_vars
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
2014-09-02 18:07:02 +05:30
|
|
|
end
|
|
|
|
|
|
|
|
describe '#extract_ref' do
|
|
|
|
it "returns an empty pair when no @project is set" do
|
|
|
|
@project = nil
|
2015-04-26 12:48:37 +05:30
|
|
|
expect(extract_ref('master/CHANGELOG')).to eq(['', ''])
|
2014-09-02 18:07:02 +05:30
|
|
|
end
|
|
|
|
|
|
|
|
context "without a path" do
|
|
|
|
it "extracts a valid branch" do
|
2015-04-26 12:48:37 +05:30
|
|
|
expect(extract_ref('master')).to eq(['master', ''])
|
2014-09-02 18:07:02 +05:30
|
|
|
end
|
|
|
|
|
|
|
|
it "extracts a valid tag" do
|
2015-04-26 12:48:37 +05:30
|
|
|
expect(extract_ref('v2.0.0')).to eq(['v2.0.0', ''])
|
2014-09-02 18:07:02 +05:30
|
|
|
end
|
|
|
|
|
|
|
|
it "extracts a valid commit ref without a path" do
|
2015-04-26 12:48:37 +05:30
|
|
|
expect(extract_ref('f4b14494ef6abf3d144c28e4af0c20143383e062')).to eq(
|
2014-09-02 18:07:02 +05:30
|
|
|
['f4b14494ef6abf3d144c28e4af0c20143383e062', '']
|
2015-04-26 12:48:37 +05:30
|
|
|
)
|
2014-09-02 18:07:02 +05:30
|
|
|
end
|
|
|
|
|
|
|
|
it "falls back to a primitive split for an invalid ref" do
|
2015-04-26 12:48:37 +05:30
|
|
|
expect(extract_ref('stable')).to eq(['stable', ''])
|
2014-09-02 18:07:02 +05:30
|
|
|
end
|
2015-09-11 14:41:01 +05:30
|
|
|
|
|
|
|
it "extracts the longest matching ref" do
|
|
|
|
expect(extract_ref('release/app/v1.0.0/README.md')).to eq(
|
|
|
|
['release/app/v1.0.0', 'README.md'])
|
|
|
|
end
|
2014-09-02 18:07:02 +05:30
|
|
|
end
|
|
|
|
|
|
|
|
context "with a path" do
|
|
|
|
it "extracts a valid branch" do
|
2015-09-11 14:41:01 +05:30
|
|
|
expect(extract_ref('foo/bar/baz/CHANGELOG')).to eq(
|
|
|
|
['foo/bar/baz', 'CHANGELOG'])
|
2014-09-02 18:07:02 +05:30
|
|
|
end
|
|
|
|
|
|
|
|
it "extracts a valid tag" do
|
2015-04-26 12:48:37 +05:30
|
|
|
expect(extract_ref('v2.0.0/CHANGELOG')).to eq(['v2.0.0', 'CHANGELOG'])
|
2014-09-02 18:07:02 +05:30
|
|
|
end
|
|
|
|
|
|
|
|
it "extracts a valid commit SHA" do
|
2015-04-26 12:48:37 +05:30
|
|
|
expect(extract_ref('f4b14494ef6abf3d144c28e4af0c20143383e062/CHANGELOG')).to eq(
|
2017-08-17 22:00:37 +05:30
|
|
|
%w(f4b14494ef6abf3d144c28e4af0c20143383e062 CHANGELOG)
|
2015-04-26 12:48:37 +05:30
|
|
|
)
|
2014-09-02 18:07:02 +05:30
|
|
|
end
|
|
|
|
|
|
|
|
it "falls back to a primitive split for an invalid ref" do
|
2017-08-17 22:00:37 +05:30
|
|
|
expect(extract_ref('stable/CHANGELOG')).to eq(%w(stable CHANGELOG))
|
2014-09-02 18:07:02 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
2016-11-03 12:29:30 +05:30
|
|
|
|
|
|
|
describe '#extract_ref_without_atom' do
|
|
|
|
it 'ignores any matching refs suffixed with atom' do
|
|
|
|
expect(extract_ref_without_atom('master.atom')).to eq('master')
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns the longest matching ref' do
|
|
|
|
expect(extract_ref_without_atom('release/app/v1.0.0.atom')).to eq('release/app/v1.0.0')
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns nil if there are no matching refs' do
|
|
|
|
expect(extract_ref_without_atom('foo.atom')).to eq(nil)
|
|
|
|
end
|
|
|
|
end
|
2014-09-02 18:07:02 +05:30
|
|
|
end
|