Commit 3e41ea46 authored by Alan (Maciej) Paruszewski's avatar Alan (Maciej) Paruszewski Committed by Heinrich Lee Yu

Handle blank default branch in security configuration

parent eb50a87b
...@@ -101,7 +101,7 @@ module Projects ...@@ -101,7 +101,7 @@ module Projects
return '' if project.empty_repo? return '' if project.empty_repo?
gitlab_ci = Gitlab::FileDetector::PATTERNS[:gitlab_ci] gitlab_ci = Gitlab::FileDetector::PATTERNS[:gitlab_ci]
Gitlab::Routing.url_helpers.project_blame_path(project, File.join(project.default_branch, gitlab_ci)) Gitlab::Routing.url_helpers.project_blame_path(project, File.join(project.default_branch_or_master, gitlab_ci))
end end
def features def features
......
---
title: Use `master` when default branch is blank in security configuration
merge_request: 48328
author:
type: fixed
...@@ -50,6 +50,16 @@ RSpec.describe Projects::Security::ConfigurationPresenter do ...@@ -50,6 +50,16 @@ RSpec.describe Projects::Security::ConfigurationPresenter do
end end
end end
context 'when the project has no default branch set' do
before do
allow(project).to receive(:default_branch).and_return(nil)
end
it 'includes the path to gitlab_ci history' do
expect(subject[:gitlab_ci_history_path]).to eq(project_blame_path(project, 'master/.gitlab-ci.yml'))
end
end
context "when the latest default branch pipeline's source is auto devops" do context "when the latest default branch pipeline's source is auto devops" do
before do before do
pipeline = create( pipeline = create(
......
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