Commit 9d15814e authored by Rémy Coutable's avatar Rémy Coutable

Fix specs

parent 73881224
...@@ -11,7 +11,7 @@ describe LabelsHelper do ...@@ -11,7 +11,7 @@ describe LabelsHelper do
end end
it 'uses the instance variable' do it 'uses the instance variable' do
expect(link_to_label(label)).to match %r{<a href="/#{@project.to_reference}/issues\?label_name=#{label.name}">.*</a>} expect(link_to_label(label)).to match %r{<a href="/#{@project.to_reference}/issues\?label_name=#{label.name}"><span class="[\w\s\-]*has_tooltip".*</span></a>}
end end
end end
...@@ -39,6 +39,14 @@ describe LabelsHelper do ...@@ -39,6 +39,14 @@ describe LabelsHelper do
end end
end end
context 'with a tooltip argument' do
context 'set to false' do
it 'does not include the has_tooltip class' do
expect(link_to_label(label, tooltip: false)).not_to match %r{has_tooltip}
end
end
end
context 'with block' do context 'with block' do
it 'passes the block to link_to' do it 'passes the block to link_to' do
link = link_to_label(label) { 'Foo' } link = link_to_label(label) { 'Foo' }
...@@ -49,7 +57,7 @@ describe LabelsHelper do ...@@ -49,7 +57,7 @@ describe LabelsHelper do
context 'without block' do context 'without block' do
it 'uses render_colored_label as the link content' do it 'uses render_colored_label as the link content' do
expect(self).to receive(:render_colored_label). expect(self).to receive(:render_colored_label).
with(label).and_return('Foo') with(label, tooltip: true).and_return('Foo')
expect(link_to_label(label)).to match('Foo') expect(link_to_label(label)).to match('Foo')
end end
end end
......
...@@ -56,7 +56,7 @@ describe Banzai::Filter::LabelReferenceFilter, lib: true do ...@@ -56,7 +56,7 @@ describe Banzai::Filter::LabelReferenceFilter, lib: true do
describe 'label span element' do describe 'label span element' do
it 'includes default classes' do it 'includes default classes' do
doc = reference_filter("Label #{reference}") doc = reference_filter("Label #{reference}")
expect(doc.css('a span').first.attr('class')).to eq 'label color-label' expect(doc.css('a span').first.attr('class')).to eq 'label color-label has_tooltip'
end end
it 'includes a style attribute' do it 'includes a style attribute' do
......
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