diff options
author | Toon Claes <toon@iotcl.com> | 2017-08-02 14:40:17 +0200 |
---|---|---|
committer | Toon Claes <toon@iotcl.com> | 2017-08-02 14:45:54 +0200 |
commit | 35081a77b0527e606c0835a979403ec4d55564d5 (patch) | |
tree | e3d5841cf0f0f8807b659e055ca9f618016f28a9 /spec/services | |
parent | 1b117e7f2d8cf4c2793bd2234688ad8739056894 (diff) | |
download | gitlab-ce-35081a77b0527e606c0835a979403ec4d55564d5.tar.gz |
Make Delete Merged Branches handle wildcard protected branches correctly
The "Delete Merged Branches" button should filter out protected
branches matching the wildcard patterns.
Closes gitlab-org/gitlab-ce#35592.
Diffstat (limited to 'spec/services')
-rw-r--r-- | spec/services/delete_merged_branches_service_spec.rb | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/spec/services/delete_merged_branches_service_spec.rb b/spec/services/delete_merged_branches_service_spec.rb index 4b872d667cf..03c682ae0d7 100644 --- a/spec/services/delete_merged_branches_service_spec.rb +++ b/spec/services/delete_merged_branches_service_spec.rb @@ -32,6 +32,14 @@ describe DeleteMergedBranchesService do expect(project.repository.branch_names).to include('improve/awesome') end + it 'keeps wildcard protected branches' do + create(:protected_branch, project: project, name: 'improve/*') + + service.execute + + expect(project.repository.branch_names).to include('improve/awesome') + end + context 'user without rights' do let(:user) { create(:user) } |