Commit f8625b8c authored by Mayra Cabrera's avatar Mayra Cabrera

Merge branch 'refactor-compare-license-test' into 'master'

Refactor compare license scanning reports service test

See merge request gitlab-org/gitlab!28197
parents 5d032667 72031abe
...@@ -3,9 +3,8 @@ ...@@ -3,9 +3,8 @@
require 'spec_helper' require 'spec_helper'
describe Ci::CompareLicenseScanningReportsService do describe Ci::CompareLicenseScanningReportsService do
let(:current_user) { project.users.take } let(:service) { described_class.new(project, nil) }
let(:service) { described_class.new(project, current_user) } let(:project) { build(:project, :repository) }
let(:project) { create(:project, :repository) }
before do before do
stub_licensed_features(license_scanning: true) stub_licensed_features(license_scanning: true)
...@@ -50,8 +49,8 @@ describe Ci::CompareLicenseScanningReportsService do ...@@ -50,8 +49,8 @@ describe Ci::CompareLicenseScanningReportsService do
end end
context 'when head pipeline has corrupted license scanning reports' do context 'when head pipeline has corrupted license scanning reports' do
let!(:base_pipeline) { create(:ee_ci_pipeline, :with_corrupted_license_management_report, project: project) } let!(:base_pipeline) { build(:ee_ci_pipeline, :with_corrupted_license_management_report, project: project) }
let!(:head_pipeline) { create(:ee_ci_pipeline, :with_corrupted_license_management_report, project: project) } let!(:head_pipeline) { build(:ee_ci_pipeline, :with_corrupted_license_management_report, project: project) }
it 'returns status and error message' do it 'returns status and error message' do
expect(subject[:status]).to eq(:error) expect(subject[:status]).to eq(:error)
......
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