Commit 8d23b964 authored by Grzegorz Bizon's avatar Grzegorz Bizon

Simplify view specs for commit status badge partial

parent 79a37423
...@@ -10,7 +10,7 @@ describe 'ci/status/_badge', :view do ...@@ -10,7 +10,7 @@ describe 'ci/status/_badge', :view do
create(:ci_build, :success, pipeline: pipeline) create(:ci_build, :success, pipeline: pipeline)
end end
context 'when status has details' do context 'when user has ability to see details' do
before do before do
project.add_developer(user) project.add_developer(user)
end end
...@@ -25,7 +25,7 @@ describe 'ci/status/_badge', :view do ...@@ -25,7 +25,7 @@ describe 'ci/status/_badge', :view do
end end
end end
context 'when status does not have details' do context 'when user do not have ability to see build details' do
before do before do
render_status(build) render_status(build)
end end
...@@ -41,17 +41,19 @@ describe 'ci/status/_badge', :view do ...@@ -41,17 +41,19 @@ describe 'ci/status/_badge', :view do
end end
context 'when rendering status for external job' do context 'when rendering status for external job' do
context 'when user has ability to see commit status details' do
before do before do
project.add_developer(use) project.add_developer(user)
render_status(external_job)
end end
context 'status has external target url' do context 'status has external target url' do
let(:external_job) do before do
create(:generic_commit_status, status: :running, external_job = create(:generic_commit_status,
status: :running,
pipeline: pipeline, pipeline: pipeline,
target_url: 'http://gitlab.com') target_url: 'http://gitlab.com')
render_status(external_job)
end end
it 'contains valid commit status text' do it 'contains valid commit status text' do
...@@ -64,8 +66,10 @@ describe 'ci/status/_badge', :view do ...@@ -64,8 +66,10 @@ describe 'ci/status/_badge', :view do
end end
context 'status do not have external target url' do context 'status do not have external target url' do
let(:external_job) do before do
create(:generic_commit_status, status: :canceled) external_job = create(:generic_commit_status, status: :canceled)
render_status(external_job)
end end
it 'contains valid commit status text' do it 'contains valid commit status text' do
...@@ -77,6 +81,7 @@ describe 'ci/status/_badge', :view do ...@@ -77,6 +81,7 @@ describe 'ci/status/_badge', :view do
end end
end end
end end
end
def render_status(resource) def render_status(resource)
render 'ci/status/badge', status: resource.detailed_status(user) render 'ci/status/badge', status: resource.detailed_status(user)
......
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