Commit 0ccdfaaf authored by Kamil Trzciński's avatar Kamil Trzciński

Merge branch '7757-fix-license-mamgement-mr-widget-for-forks' into 'master'

Fix managed licences path for fork cases

Closes #7757

See merge request gitlab-org/gitlab-ee!8700
parents 21f143fc 3ec8558c
......@@ -66,7 +66,7 @@ module EE
end
expose :managed_licenses_path do |merge_request|
api_v4_projects_managed_licenses_path(id: merge_request.source_project.id)
api_v4_projects_managed_licenses_path(id: merge_request.target_project.id)
end
expose :can_manage_licenses do |merge_request|
......
---
title: Fixed license managment path in MR widget for fork cases
merge_request: 8700
author:
type: fixed
require 'spec_helper'
describe MergeRequestWidgetEntity do
include ProjectForksHelper
set(:user) { create(:user) }
set(:project) { create :project, :repository }
set(:merge_request) { create(:merge_request, source_project: project, target_project: project) }
......@@ -150,6 +152,33 @@ describe MergeRequestWidgetEntity do
expect(subject.as_json[:license_management]).to include(:license_management_full_report_path)
end
end
describe '#managed_licenses_path' do
let(:managed_licenses_path) { api_v4_projects_managed_licenses_path(id: project.id) }
before do
create(:ee_ci_build, :legacy_license_management, pipeline: pipeline)
end
it 'should be a path for target project' do
expect(subject.as_json[:license_management][:managed_licenses_path]).to eq(managed_licenses_path)
end
context 'with fork' do
let(:source_project) { fork_project(project, user, repository: true) }
let(:fork_merge_request) { create(:merge_request, source_project: source_project, target_project: project) }
let(:subject_json) { described_class.new(fork_merge_request, current_user: user, request: request).as_json }
before do
allow(fork_merge_request).to receive_messages(head_pipeline: pipeline)
stub_licensed_features(license_management: true)
end
it 'should be a path for target project' do
expect(subject_json[:license_management][:managed_licenses_path]).to eq(managed_licenses_path)
end
end
end
end
it 'has vulnerability feedbacks path' do
......
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