Commit a06e9f48 authored by Dmitriy Zaporozhets's avatar Dmitriy Zaporozhets

Update merge_request_widget_entity_spec to match refactoring made in serializer

Signed-off-by: default avatarDmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
parent be8b8cff
...@@ -7,6 +7,10 @@ describe MergeRequestWidgetEntity do ...@@ -7,6 +7,10 @@ describe MergeRequestWidgetEntity do
let(:request) { double('request', current_user: user) } let(:request) { double('request', current_user: user) }
let(:pipeline) { create(:ci_empty_pipeline, project: project) } let(:pipeline) { create(:ci_empty_pipeline, project: project) }
before do
project.add_developer(user)
end
subject do subject do
described_class.new(merge_request, request: request) described_class.new(merge_request, request: request)
end end
...@@ -23,7 +27,7 @@ describe MergeRequestWidgetEntity do ...@@ -23,7 +27,7 @@ describe MergeRequestWidgetEntity do
it 'has performance data' do it 'has performance data' do
build = create(:ci_build, name: 'job') build = create(:ci_build, name: 'job')
allow(subject).to receive(:expose_performance_data?).and_return(true) allow(merge_request).to receive(:expose_performance_data?).and_return(true)
allow(merge_request).to receive(:base_performance_artifact).and_return(build) allow(merge_request).to receive(:base_performance_artifact).and_return(build)
allow(merge_request).to receive(:head_performance_artifact).and_return(build) allow(merge_request).to receive(:head_performance_artifact).and_return(build)
...@@ -31,9 +35,9 @@ describe MergeRequestWidgetEntity do ...@@ -31,9 +35,9 @@ describe MergeRequestWidgetEntity do
end end
it 'has sast data' do it 'has sast data' do
build = create(:ci_build, name: 'sast') build = create(:ci_build, name: 'sast', pipeline: pipeline)
allow(subject).to receive(:expose_sast_data?).and_return(true) allow(merge_request).to receive(:expose_sast_data?).and_return(true)
allow(merge_request).to receive(:has_base_sast_data?).and_return(true) allow(merge_request).to receive(:has_base_sast_data?).and_return(true)
allow(merge_request).to receive(:base_sast_artifact).and_return(build) allow(merge_request).to receive(:base_sast_artifact).and_return(build)
allow(merge_request).to receive(:head_sast_artifact).and_return(build) allow(merge_request).to receive(:head_sast_artifact).and_return(build)
...@@ -44,9 +48,9 @@ describe MergeRequestWidgetEntity do ...@@ -44,9 +48,9 @@ describe MergeRequestWidgetEntity do
end end
it 'has sast_container data' do it 'has sast_container data' do
build = create(:ci_build, name: 'sast:image') build = create(:ci_build, name: 'sast:image', pipeline: pipeline)
allow(subject).to receive(:expose_sast_container_data?).and_return(true) allow(merge_request).to receive(:expose_sast_container_data?).and_return(true)
allow(merge_request).to receive(:sast_container_artifact).and_return(build) allow(merge_request).to receive(:sast_container_artifact).and_return(build)
expect(subject.as_json).to include(:sast_container) expect(subject.as_json).to include(:sast_container)
......
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