Commit eeeea489 authored by Zeger-Jan van de Weg's avatar Zeger-Jan van de Weg

AutoDevOps banner hidden on explicit CI config

Extends the helper method to no show the banner as soon as the project
has a `.gitlab-ci.yml` file on the default branch.

Fixes gitlab-org/gitlab-ce#37652
parent 9c9b1774
...@@ -3,6 +3,8 @@ module AutoDevopsHelper ...@@ -3,6 +3,8 @@ module AutoDevopsHelper
Feature.get(:auto_devops_banner_disabled).off? && Feature.get(:auto_devops_banner_disabled).off? &&
show_callout?('auto_devops_settings_dismissed') && show_callout?('auto_devops_settings_dismissed') &&
can?(current_user, :admin_pipeline, project) && can?(current_user, :admin_pipeline, project) &&
project.has_auto_devops_implicitly_disabled? project.has_auto_devops_implicitly_disabled? &&
!project.repository.gitlab_ci_yml &&
project.ci_services.active.none?
end end
end end
---
title: Do not show the Auto DevOps banner when the project has a .gitlab-ci.yml on
master
merge_request:
author:
type: fixed
...@@ -65,5 +65,21 @@ describe AutoDevopsHelper do ...@@ -65,5 +65,21 @@ describe AutoDevopsHelper do
it { is_expected.to eq(false) } it { is_expected.to eq(false) }
end end
context 'when master contains a .gitlab-ci.yml file' do
before do
allow(project.repository).to receive(:gitlab_ci_yml).and_return("script: [ 'test']")
end
it { is_expected.to eq(false) }
end
context 'when another service is enabled' do
before do
create(:service, project: project, category: :ci, active: true)
end
it { is_expected.to eq(false) }
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