diff options
| author | Joas Schilling <coding@schilljs.com> | 2021-04-21 08:58:35 +0200 |
|---|---|---|
| committer | Joas Schilling <coding@schilljs.com> | 2021-05-12 08:16:07 +0200 |
| commit | b6c6527705695a343b055f89bdde5ec497914ff1 (patch) | |
| tree | b52adc3a0b203add9a971cd1e2bf0ef9666af23a /build | |
| parent | 0599a8060ceb6518bb3981c88fc14f215d80f562 (diff) | |
| download | nextcloud-server-b6c6527705695a343b055f89bdde5ec497914ff1.tar.gz nextcloud-server-b6c6527705695a343b055f89bdde5ec497914ff1.zip | |
Fix unauthorized OCS status in provisioning
Signed-off-by: Joas Schilling <coding@schilljs.com>
Diffstat (limited to 'build')
4 files changed, 18 insertions, 17 deletions
diff --git a/build/integration/features/provisioning-v1.feature b/build/integration/features/provisioning-v1.feature index 03aaad4b857..22feb7ef24f 100644 --- a/build/integration/features/provisioning-v1.feature +++ b/build/integration/features/provisioning-v1.feature @@ -176,8 +176,8 @@ Feature: provisioning When sending "PUT" to "/cloud/users/brand-new-user" with | key | phoneScope | | value | v2-private | - Then the OCS status code should be "997" - And the HTTP status code should be "401" + Then the OCS status code should be "103" + And the HTTP status code should be "200" Scenario: Search by phone number Given As an "admin" @@ -234,11 +234,12 @@ Feature: provisioning And the HTTP status code should be "200" Scenario: adding user to a group without privileges - Given As an "brand-new-user" + Given user "brand-new-user" exists + And As an "brand-new-user" When sending "POST" to "/cloud/users/brand-new-user/groups" with | groupid | new-group | - Then the OCS status code should be "997" - And the HTTP status code should be "401" + Then the OCS status code should be "403" + And the HTTP status code should be "200" Scenario: adding user to a group Given As an "admin" @@ -523,8 +524,8 @@ Feature: provisioning And Assure user "subadmin" is subadmin of group "new-group" And As an "subadmin" When sending "PUT" to "/cloud/users/user1/disable" - Then the OCS status code should be "997" - Then the HTTP status code should be "401" + Then the OCS status code should be "998" + Then the HTTP status code should be "200" And As an "admin" And user "user1" is enabled @@ -539,8 +540,8 @@ Feature: provisioning And Assure user "subadmin" is subadmin of group "new-group" And As an "subadmin" When sending "PUT" to "/cloud/users/another-admin/disable" - Then the OCS status code should be "997" - Then the HTTP status code should be "401" + Then the OCS status code should be "998" + Then the HTTP status code should be "200" And As an "admin" And user "another-admin" is enabled @@ -615,8 +616,8 @@ Feature: provisioning And user "user2" exists And As an "user1" When sending "PUT" to "/cloud/users/user2/disable" - Then the OCS status code should be "997" - And the HTTP status code should be "401" + Then the OCS status code should be "403" + And the HTTP status code should be "200" And As an "admin" And user "user2" is enabled @@ -627,8 +628,8 @@ Feature: provisioning And assure user "user2" is disabled And As an "user1" When sending "PUT" to "/cloud/users/user2/enable" - Then the OCS status code should be "997" - And the HTTP status code should be "401" + Then the OCS status code should be "403" + And the HTTP status code should be "200" And As an "admin" And user "user2" is disabled diff --git a/build/integration/sharing_features/sharing-v1-part2.feature b/build/integration/sharing_features/sharing-v1-part2.feature index 8fc06fbddeb..f9ebf6782b1 100644 --- a/build/integration/sharing_features/sharing-v1-part2.feature +++ b/build/integration/sharing_features/sharing-v1-part2.feature @@ -919,7 +919,7 @@ Feature: sharing And As an "user1" When Deleting last share Then the OCS status code should be "403" - And the HTTP status code should be "401" + And the HTTP status code should be "200" Scenario: Keep usergroup shares (#22143) Given As an "admin" diff --git a/build/integration/sharing_features/sharing-v1-part3.feature b/build/integration/sharing_features/sharing-v1-part3.feature index 5094111d025..42e0f1b6add 100644 --- a/build/integration/sharing_features/sharing-v1-part3.feature +++ b/build/integration/sharing_features/sharing-v1-part3.feature @@ -403,7 +403,7 @@ Feature: sharing And Updating last share with | permissions | 19 | Then the OCS status code should be "403" - And the HTTP status code should be "401" + And the HTTP status code should be "200" Scenario: do not allow to increase permissions on non received share with user with resharing rights Given As an "admin" @@ -427,7 +427,7 @@ Feature: sharing And Updating last share with | permissions | 19 | Then the OCS status code should be "403" - And the HTTP status code should be "401" + And the HTTP status code should be "200" Scenario: do not allow to increase link share permissions on reshare Given As an "admin" diff --git a/build/integration/sharing_features/sharing-v1.feature b/build/integration/sharing_features/sharing-v1.feature index 2c391805331..fbcbb50b7de 100644 --- a/build/integration/sharing_features/sharing-v1.feature +++ b/build/integration/sharing_features/sharing-v1.feature @@ -54,7 +54,7 @@ Feature: sharing | shareWith | a-room-token | | shareType | 10 | Then the OCS status code should be "403" - And the HTTP status code should be "401" + And the HTTP status code should be "200" Scenario: Creating a new mail share Given dummy mail server is listening |