Commit 860c8cfc authored by Phil Hughes's avatar Phil Hughes

Added header to protected branches access dropdowns

CE part of https://gitlab.com/gitlab-org/gitlab-ee/issues/1294
parent fadeefb6
...@@ -68,8 +68,12 @@ class Projects::ProtectedBranchesController < Projects::ApplicationController ...@@ -68,8 +68,12 @@ class Projects::ProtectedBranchesController < Projects::ApplicationController
def access_levels_options def access_levels_options
{ {
push_access_levels: ProtectedBranch::PushAccessLevel.human_access_levels.map { |id, text| { id: id, text: text, before_divider: true } }, push_access_levels: {
merge_access_levels: ProtectedBranch::MergeAccessLevel.human_access_levels.map { |id, text| { id: id, text: text, before_divider: true } } "Roles" => ProtectedBranch::PushAccessLevel.human_access_levels.map { |id, text| { id: id, text: text, before_divider: true } },
},
merge_access_levels: {
"Roles" => ProtectedBranch::MergeAccessLevel.human_access_levels.map { |id, text| { id: id, text: text, before_divider: true } }
}
} }
end end
......
---
title: Added headers to protected branch access dropdowns
merge_request:
author:
...@@ -26,7 +26,11 @@ RSpec.shared_examples "protected branches > access control > CE" do ...@@ -26,7 +26,11 @@ RSpec.shared_examples "protected branches > access control > CE" do
within(".protected-branches-list") do within(".protected-branches-list") do
find(".js-allowed-to-push").click find(".js-allowed-to-push").click
within('.js-allowed-to-push-container') { click_on access_type_name }
within('.js-allowed-to-push-container') do
expect(first("li")).to have_content("Roles")
click_on access_type_name
end
end end
wait_for_ajax wait_for_ajax
...@@ -61,7 +65,11 @@ RSpec.shared_examples "protected branches > access control > CE" do ...@@ -61,7 +65,11 @@ RSpec.shared_examples "protected branches > access control > CE" do
within(".protected-branches-list") do within(".protected-branches-list") do
find(".js-allowed-to-merge").click find(".js-allowed-to-merge").click
within('.js-allowed-to-merge-container') { click_on access_type_name }
within('.js-allowed-to-merge-container') do
expect(first("li")).to have_content("Roles")
click_on access_type_name
end
end end
wait_for_ajax wait_for_ajax
......
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