Add latest changes from gitlab-org/gitlab@16-0-stable-ee
This commit is contained in:
parent
d7b53d074b
commit
fefc7abcfc
|
|
@ -10,6 +10,7 @@ module Sidebars
|
|||
add_item(background_migrations_menu_item)
|
||||
add_item(background_jobs_menu_item)
|
||||
add_item(health_check_menu_item)
|
||||
add_item(metrics_dashboard_menu_item)
|
||||
true
|
||||
end
|
||||
|
||||
|
|
@ -65,6 +66,18 @@ module Sidebars
|
|||
item_id: :health_check
|
||||
)
|
||||
end
|
||||
|
||||
def metrics_dashboard_menu_item
|
||||
return ::Sidebars::NilMenuItem.new(item_id: :metrics_dashboard) unless
|
||||
Gitlab::CurrentSettings.current_application_settings.grafana_enabled?
|
||||
|
||||
::Sidebars::MenuItem.new(
|
||||
title: _('Metrics Dashboard'),
|
||||
link: Gitlab::CurrentSettings.current_application_settings.grafana_url,
|
||||
active_routes: { path: Gitlab::CurrentSettings.current_application_settings.grafana_url },
|
||||
item_id: :metrics_dashboard
|
||||
)
|
||||
end
|
||||
end
|
||||
end
|
||||
end
|
||||
|
|
|
|||
|
|
@ -3,10 +3,38 @@
|
|||
require 'spec_helper'
|
||||
|
||||
RSpec.describe Sidebars::Admin::Menus::MonitoringMenu, feature_category: :navigation do
|
||||
let_it_be(:user) { create(:user, :admin) }
|
||||
let(:context) { Sidebars::Context.new(current_user: user, container: nil) }
|
||||
let(:menu) { described_class.new(context) }
|
||||
|
||||
it_behaves_like 'Admin menu',
|
||||
link: '/admin/system_info',
|
||||
title: s_('Admin|Monitoring'),
|
||||
icon: 'monitor'
|
||||
|
||||
it_behaves_like 'Admin menu with sub menus'
|
||||
|
||||
describe 'Menu items' do
|
||||
subject { described_class.new(context).renderable_items.index { |e| e.item_id == item_id } }
|
||||
|
||||
describe 'Metrics Dashboard' do
|
||||
let(:item_id) { :metrics_dashboard }
|
||||
|
||||
before do
|
||||
stub_application_setting(grafana_enabled: grafana_enabled)
|
||||
end
|
||||
|
||||
context 'when grafana is enabled' do
|
||||
let(:grafana_enabled) { true }
|
||||
|
||||
specify { is_expected.not_to be_nil }
|
||||
end
|
||||
|
||||
context 'when grafana is disabled' do
|
||||
let(:grafana_enabled) { false }
|
||||
|
||||
specify { is_expected.to be_nil }
|
||||
end
|
||||
end
|
||||
end
|
||||
end
|
||||
|
|
|
|||
Loading…
Reference in New Issue