Commit 4472779b authored by Filipa Lacerda's avatar Filipa Lacerda

Adds tests to build view

Fix CHANGELOG entry
parent a0872833
...@@ -2,7 +2,7 @@ ...@@ -2,7 +2,7 @@
.header-content .header-content
= render 'ci/status/badge', status: @build.detailed_status(current_user) = render 'ci/status/badge', status: @build.detailed_status(current_user)
Build Build
%strong ##{@build.id} %strong.js-build-id ##{@build.id}
in pipeline in pipeline
= link_to pipeline_path(@build.pipeline) do = link_to pipeline_path(@build.pipeline) do
%strong ##{@build.pipeline.id} %strong ##{@build.pipeline.id}
......
--- ---
title: Add new ci status partial to render a badge without a link title: Add new ci status partial to render a badge without a link
merge_request: merge_request: 8740
author: 8740 author:
...@@ -15,6 +15,32 @@ describe 'projects/builds/show', :view do ...@@ -15,6 +15,32 @@ describe 'projects/builds/show', :view do
allow(view).to receive(:can?).and_return(true) allow(view).to receive(:can?).and_return(true)
end end
describe 'build information in header' do
let(:build) do
create(:ci_build, :success, environment: 'staging')
end
before do
render
end
it 'shows status name' do
expect(rendered).to have_css('.ci-status.ci-success', text: 'passed')
end
it 'shows build id' do
expect(rendered).to have_css('.js-build-id', text: build.id)
end
it 'shows a link to the pipeline' do
expect(rendered).to have_link(build.pipeline.id)
end
it 'shows a link to the commit' do
expect(rendered).to have_link(build.pipeline.short_sha)
end
end
describe 'environment info in build view' do describe 'environment info in build view' do
context 'build with latest deployment' do context 'build with latest deployment' do
let(:build) do let(:build) 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