Commit 4ab83117 authored by Filipa Lacerda's avatar Filipa Lacerda

Changes after review

parent 4472779b
...@@ -6,6 +6,4 @@ ...@@ -6,6 +6,4 @@
= custom_icon(status.icon) = custom_icon(status.icon)
= status.text = status.text
- else - else
%span{ class: css_classes } = render "ci/status/badge", status: status
= custom_icon(status.icon)
= status.text
...@@ -28,6 +28,10 @@ describe 'projects/builds/show', :view do ...@@ -28,6 +28,10 @@ describe 'projects/builds/show', :view do
expect(rendered).to have_css('.ci-status.ci-success', text: 'passed') expect(rendered).to have_css('.ci-status.ci-success', text: 'passed')
end end
it 'does not render a link to the build' do
expect(rendered).not_to have_link('passed')
end
it 'shows build id' do it 'shows build id' do
expect(rendered).to have_css('.js-build-id', text: build.id) expect(rendered).to have_css('.js-build-id', text: build.id)
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