Commit 4dd1f955 authored by winniehell's avatar winniehell

Add failing test for #20026

parent f4cb04da
...@@ -3,7 +3,7 @@ require 'spec_helper' ...@@ -3,7 +3,7 @@ require 'spec_helper'
describe Banzai::Filter::RelativeLinkFilter, lib: true do describe Banzai::Filter::RelativeLinkFilter, lib: true do
def filter(doc, contexts = {}) def filter(doc, contexts = {})
contexts.reverse_merge!({ contexts.reverse_merge!({
commit: project.commit, commit: commit,
project: project, project: project,
project_wiki: project_wiki, project_wiki: project_wiki,
ref: ref, ref: ref,
...@@ -28,6 +28,7 @@ describe Banzai::Filter::RelativeLinkFilter, lib: true do ...@@ -28,6 +28,7 @@ describe Banzai::Filter::RelativeLinkFilter, lib: true do
let(:project) { create(:project) } let(:project) { create(:project) }
let(:project_path) { project.path_with_namespace } let(:project_path) { project.path_with_namespace }
let(:ref) { 'markdown' } let(:ref) { 'markdown' }
let(:commit) { project.commit(ref) }
let(:project_wiki) { nil } let(:project_wiki) { nil }
let(:requested_path) { '/' } let(:requested_path) { '/' }
...@@ -77,7 +78,13 @@ describe Banzai::Filter::RelativeLinkFilter, lib: true do ...@@ -77,7 +78,13 @@ describe Banzai::Filter::RelativeLinkFilter, lib: true do
expect { filter(act) }.not_to raise_error expect { filter(act) }.not_to raise_error
end end
context 'with a valid repository' do it 'ignores ref if commit is passed' do
doc = filter(link('non/existent.file'), commit: project.commit('empty-branch') )
expect(doc.at_css('a')['href']).
to eq "/#{project_path}/#{ref}/non/existent.file" # non-existent files have no leading blob/raw/tree
end
shared_examples :valid_repository do
it 'rebuilds absolute URL for a file in the repo' do it 'rebuilds absolute URL for a file in the repo' do
doc = filter(link('/doc/api/README.md')) doc = filter(link('/doc/api/README.md'))
expect(doc.at_css('a')['href']). expect(doc.at_css('a')['href']).
...@@ -184,4 +191,13 @@ describe Banzai::Filter::RelativeLinkFilter, lib: true do ...@@ -184,4 +191,13 @@ describe Banzai::Filter::RelativeLinkFilter, lib: true do
include_examples :relative_to_requested include_examples :relative_to_requested
end end
end end
context 'with a valid commit' do
include_examples :valid_repository
end
context 'with a valid ref' do
let(:commit) { nil } # force filter to use ref instead of commit
include_examples :valid_repository
end
end end
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment