diff options
author | Scott Hampton <shampton@gitlab.com> | 2019-02-19 13:27:53 -0700 |
---|---|---|
committer | Scott Hampton <shampton@gitlab.com> | 2019-02-19 13:27:53 -0700 |
commit | 54a5d513e5f068c53fad3b2dac04998f5e9afd88 (patch) | |
tree | f00f87a1549059dd2d84203dd8f4c730af41b823 /spec/views | |
parent | ab9f8785ebf5483e5d2cd02497b57fa05a47561e (diff) | |
download | gitlab-ce-54a5d513e5f068c53fad3b2dac04998f5e9afd88.tar.gz |
Fixing CI icon mismatch
MR list, and related MRs and branches were using a deprecated helper.
Created a new icon haml file to help move these forward.
Diffstat (limited to 'spec/views')
-rw-r--r-- | spec/views/ci/status/_icon.html.haml_spec.rb | 88 |
1 files changed, 88 insertions, 0 deletions
diff --git a/spec/views/ci/status/_icon.html.haml_spec.rb b/spec/views/ci/status/_icon.html.haml_spec.rb new file mode 100644 index 00000000000..43806446164 --- /dev/null +++ b/spec/views/ci/status/_icon.html.haml_spec.rb @@ -0,0 +1,88 @@ +require 'spec_helper' + +describe 'ci/status/_icon' do + let(:user) { create(:user) } + let(:project) { create(:project, :private) } + let(:pipeline) { create(:ci_pipeline, project: project) } + + context 'when rendering status for build' do + let(:build) do + create(:ci_build, :success, pipeline: pipeline) + end + + context 'when user has ability to see details' do + before do + project.add_developer(user) + end + + it 'has link to build details page' do + details_path = project_job_path(project, build) + + render_status(build) + + expect(rendered).to have_link(href: details_path) + end + end + + context 'when user do not have ability to see build details' do + before do + render_status(build) + end + + it 'contains build status text' do + expect(rendered).to have_css('.ci-status-icon.ci-status-icon-success') + end + + it 'does not contain links' do + expect(rendered).not_to have_link + end + end + end + + context 'when rendering status for external job' do + context 'when user has ability to see commit status details' do + before do + project.add_developer(user) + end + + context 'status has external target url' do + before do + external_job = create(:generic_commit_status, + status: :running, + pipeline: pipeline, + target_url: 'http://gitlab.com') + + render_status(external_job) + end + + it 'contains valid commit status text' do + expect(rendered).to have_css('.ci-status-icon.ci-status-icon-running') + end + + it 'has link to external status page' do + expect(rendered).to have_link(href: 'http://gitlab.com') + end + end + + context 'status do not have external target url' do + before do + external_job = create(:generic_commit_status, status: :canceled) + + render_status(external_job) + end + + it 'contains valid commit status text' do + expect(rendered).to have_css('.ci-status-icon.ci-status-icon-canceled') + end + + it 'has link to external status page' do + expect(rendered).not_to have_link + end + end + end + end + + def render_status(resource) + render 'ci/status/icon', status: resource.detailed_status(user) + end +end |