Merge branch 'fix-test-coverage-api-groups-spec' into 'master'

Add missing test coverage on Group API spec See merge request gitlab-org/gitlab!52055

Merge branch 'fix-test-coverage-api-groups-spec' into 'master'
Add missing test coverage on Group API spec See merge request gitlab-org/gitlab!52055
74f25e12 · Arturo Herrero · e83fec82 · 492e8d56 · 74f25e12
Commit 74f25e12 authored Jan 20, 2021 by Arturo Herrero
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

ee/spec/requests/api/groups_spec.rb ee/spec/requests/api/groups_spec.rb +2 -2

No files found.
--- a/ee/spec/requests/api/groups_spec.rb
+++ b/ee/spec/requests/api/groups_spec.rb
@@ -243,7 +243,7 @@ RSpec.describe API::Groups do
        let(:user) { admin }

        where(:feature_enabled, :setting_enabled, :default_branch_protection) do
-          false | false | Gitlab::Access::PROTECTION_NONE
+          true  | true  | Gitlab::Access::PROTECTION_NONE
          false | true  | Gitlab::Access::PROTECTION_NONE
          true  | false | Gitlab::Access::PROTECTION_NONE
          false | false | Gitlab::Access::PROTECTION_NONE
@@ -266,7 +266,7 @@ RSpec.describe API::Groups do

      context 'authenticated a normal user' do
        where(:feature_enabled, :setting_enabled, :default_branch_protection) do
-          false | false | Gitlab::Access::PROTECTION_NONE
+          true  | true  | Gitlab::Access::PROTECTION_NONE
          false | true  | Gitlab::Access::PROTECTION_NONE
          true  | false | Gitlab::Access::PROTECTION_FULL
          false | false | Gitlab::Access::PROTECTION_NONE