Skip to content

Commit

Permalink
adjust tests
Browse files Browse the repository at this point in the history
Signed-off-by: jkoberg <[email protected]>
  • Loading branch information
kobergj committed Apr 3, 2023
1 parent 4f6784a commit 15ca806
Show file tree
Hide file tree
Showing 6 changed files with 20 additions and 19 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -54,6 +54,7 @@ The expected failures in this file are from features in the owncloud/ocis repo.
- [apiGraph/assignRole.feature:33](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraph/assignRole.feature#L33)

### [Group having percentage (%) can be created but cannot be GET](https://github.com/owncloud/ocis/issues/5083)
- [apiGraph/deleteGroup.feature:34](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraph/deleteGroup.feature#L34)
- [apiGraph/deleteGroup.feature:49](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraph/deleteGroup.feature#L49)
- [apiGraph/deleteGroup.feature:50](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraph/deleteGroup.feature#L50)
- [apiGraph/deleteGroup.feature:51](https://github.com/owncloud/ocis/blob/master/tests/acceptance/features/apiGraph/deleteGroup.feature#L51)
Expand Down
12 changes: 6 additions & 6 deletions tests/acceptance/features/apiGraph/addUserToGroup.feature
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ Feature: add users to group
| Finance (NP) | Space and brackets |
| Admin&Finance | Ampersand |
| maint+eng | Plus sign |
| $x<=>[y*z^2]! | Maths symbols |
| $x<>[y*z^2]! | Maths symbols |
| 😁 πŸ˜‚ | emoji |
| admin:Pokhara@Nepal | Colon and @ |
When the administrator adds the following users to the following groups using the Graph API
Expand All @@ -49,7 +49,7 @@ Feature: add users to group
| Alice | Finance (NP) |
| Alice | Admin&Finance |
| Alice | maint+eng |
| Alice | $x<=>[y*z^2]! |
| Alice | $x<>[y*z^2]! |
| Alice | 😁 πŸ˜‚ |
| Alice | admin:Pokhara@Nepal |
Then the HTTP status code of responses on all endpoints should be "204"
Expand All @@ -62,7 +62,7 @@ Feature: add users to group
| Alice | Finance (NP) |
| Alice | Admin&Finance |
| Alice | maint+eng |
| Alice | $x<=>[y*z^2]! |
| Alice | $x<>[y*z^2]! |
| Alice | 😁 πŸ˜‚ |
| Alice | admin:Pokhara@Nepal |

Expand All @@ -72,7 +72,7 @@ Feature: add users to group
| groupname | comment |
| maintenance#123 | Hash sign |
| 50%pass | Percent sign (special escaping happens) |
| 50%25=0 | %25 literal looks like an escaped "%" |
| 50%250 | %25 literal looks like an escaped "%" |
| 50%2Eagle | %2E literal looks like an escaped "." |
| 50%2Fix | %2F literal looks like an escaped slash |
| Mgmt\Middle | Backslash |
Expand All @@ -81,7 +81,7 @@ Feature: add users to group
| username | groupname |
| Alice | maintenance#123 |
| Alice | 50%pass |
| Alice | 50%25=0 |
| Alice | 50%250 |
| Alice | 50%2Eagle |
| Alice | 50%2Fix |
| Alice | Mgmt\Middle |
Expand All @@ -91,7 +91,7 @@ Feature: add users to group
| username | groupname |
| Alice | maintenance#123 |
| Alice | 50%pass |
| Alice | 50%25=0 |
| Alice | 50%250 |
| Alice | 50%2Eagle |
| Alice | 50%2Fix |
| Alice | Mgmt\Middle |
Expand Down
2 changes: 1 addition & 1 deletion tests/acceptance/features/apiGraph/createGroup.feature
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ Feature: create group
| simplegroup |
| EspaΓ±a§àôœ€ |
| ΰ€¨ΰ₯‡ΰ€ͺΰ€Ύΰ€²ΰ₯€ |
| $x<=>[y*z^2+1]! |
| $x<>[y*z^2+1]! |
| πŸ˜… πŸ˜† |
| comma,grp1 |
| Finance (NP) |
Expand Down
4 changes: 2 additions & 2 deletions tests/acceptance/features/apiGraph/deleteGroup.feature
Original file line number Diff line number Diff line change
Expand Up @@ -27,11 +27,11 @@ Feature: delete groups
| Admin&Finance | Ampersand |
| admin:Pokhara@Nepal | Colon and @ |
| maint+eng | Plus sign |
| $x<=>[y*z^2]! | Maths symbols |
| $x<>[y*z^2]! | Maths symbols |
| Mgmt\Middle | Backslash |
| 😁 πŸ˜‚ | emoji |
| maintenance#123 | Hash sign |
| 50%25=0 | %25 literal looks like an escaped "%" |
| 50%250 | %25 literal looks like an escaped "%" |
| staff?group | Question mark |
| Mgmt/Sydney | Slash (special escaping happens) |
| Mgmt//NSW/Sydney | Multiple slash |
Expand Down
4 changes: 2 additions & 2 deletions tests/acceptance/features/apiGraph/getGroup.feature
Original file line number Diff line number Diff line change
Expand Up @@ -186,8 +186,8 @@ Feature: get groups and their members
| group |
| EspaΓ±a§àôœ€ |
| ΰ€¨ΰ₯‡ΰ€ͺΰ€Ύΰ€²ΰ₯€ |
| $x<=>[y*z^2+1]! |
| Υ₯Γ²Ι΄Τͺ˯ΗՐΛɔπ |
| $x<>[y*z^2+1]! |
| Υ₯Γ²Ι΄Τͺ˯ΗՐΛɔπ |


Scenario: admin user tries to get group information of non-existing group
Expand Down
16 changes: 8 additions & 8 deletions tests/acceptance/features/apiGraph/removeUserFromGroup.feature
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ Feature: remove a user from a group
| Admin&Finance | Ampersand |
| admin:Pokhara@Nepal | Colon and @ |
| maint+eng | Plus sign |
| $x<=>[y*z^2]! | Maths symbols |
| $x<>[y*z^2]! | Maths symbols |
| Mgmt\Middle | Backslash |
| 😁 πŸ˜‚ | emoji |
And the following users have been added to the following groups
Expand All @@ -56,7 +56,7 @@ Feature: remove a user from a group
| Alice | Admin&Finance |
| Alice | admin:Pokhara@Nepal |
| Alice | maint+eng |
| Alice | $x<=>[y*z^2]! |
| Alice | $x<>[y*z^2]! |
| Alice | Mgmt\Middle |
| Alice | 😁 πŸ˜‚ |
When the administrator removes the following users from the following groups using the Graph API
Expand All @@ -69,7 +69,7 @@ Feature: remove a user from a group
| Alice | Admin&Finance |
| Alice | admin:Pokhara@Nepal |
| Alice | maint+eng |
| Alice | $x<=>[y*z^2]! |
| Alice | $x<>[y*z^2]! |
| Alice | Mgmt\Middle |
| Alice | 😁 πŸ˜‚ |
Then the HTTP status code of responses on all endpoints should be "204"
Expand All @@ -83,7 +83,7 @@ Feature: remove a user from a group
| Alice | Admin&Finance |
| Alice | admin:Pokhara@Nepal |
| Alice | maint+eng |
| Alice | $x<=>[y*z^2]! |
| Alice | $x<>[y*z^2]! |
| Alice | Mgmt\Middle |
| Alice | 😁 πŸ˜‚ |

Expand All @@ -92,23 +92,23 @@ Feature: remove a user from a group
Given these groups have been created:
| groupname | comment |
| maintenance#123 | Hash sign |
| 50%25=0 | %25 literal looks like an escaped "%" |
| 50%250 | %25 literal looks like an escaped "%" |
| staff?group | Question mark |
| 50%pass | Percent sign (special escaping happens) |
| 50%2Eagle | %2E literal looks like an escaped "." |
| 50%2Fix | %2F literal looks like an escaped slash |
And the following users have been added to the following groups
| username | groupname |
| Alice | maintenance#123 |
| Alice | 50%25=0 |
| Alice | 50%250 |
| Alice | staff?group |
| Alice | 50%pass |
| Alice | 50%2Eagle |
| Alice | 50%2Fix |
When the administrator removes the following users from the following groups using the Graph API
| username | groupname |
| Alice | maintenance#123 |
| Alice | 50%25=0 |
| Alice | 50%250 |
| Alice | staff?group |
| Alice | 50%pass |
| Alice | 50%2Eagle |
Expand All @@ -117,7 +117,7 @@ Feature: remove a user from a group
And the following users should not belong to the following groups
| username | groupname |
| Alice | maintenance#123 |
| Alice | 50%25=0 |
| Alice | 50%250 |
| Alice | staff?group |
| Alice | 50%pass |
| Alice | 50%2Eagle |
Expand Down

0 comments on commit 15ca806

Please sign in to comment.