Commit 24dd70d3 authored by Grzegorz Bizon's avatar Grzegorz Bizon

Extend tests for pipeline detailed status helpers

parent 8f743ede
...@@ -2,29 +2,35 @@ require 'spec_helper' ...@@ -2,29 +2,35 @@ require 'spec_helper'
describe Gitlab::Ci::Status::Pipeline::Common do describe Gitlab::Ci::Status::Pipeline::Common do
let(:user) { create(:user) } let(:user) { create(:user) }
let(:project) { create(:empty_project) } let(:project) { create(:empty_project, :private) }
let(:pipeline) { create(:ci_pipeline, project: project) } let(:pipeline) { create(:ci_pipeline, project: project) }
subject do subject do
Class.new(Gitlab::Ci::Status::Core) Gitlab::Ci::Status::Core
.new(pipeline, user) .new(pipeline, user)
.extend(described_class) .extend(described_class)
end end
before do describe '#has_action?' do
project.team << [user, :developer] it { is_expected.not_to have_action }
end end
it 'does not have action' do describe '#has_details?' do
expect(subject).not_to have_action context 'when user has access to read pipeline' do
end before { project.team << [user, :developer] }
it { is_expected.to have_details }
end
it 'has details' do context 'when user does not have access to read pipeline' do
expect(subject).to have_details it { is_expected.not_to have_details }
end
end end
it 'links to the pipeline details page' do describe '#details_path' do
expect(subject.details_path) it 'links to the pipeline details page' do
.to include "pipelines/#{pipeline.id}" expect(subject.details_path)
.to include "pipelines/#{pipeline.id}"
end
end end
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