[full-ci] [tests-only] Added test for sending share invitation to disabled user #8139
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
36 tasks
grgprarup
force-pushed
the
add-test-for-invite-disabled-user
branch
from
8e05f65 to
e9771dd
Compare
grgprarup
changed the title
grgprarup
changed the title
grgprarup
changed the title
grgprarup
force-pushed
the
add-test-for-invite-disabled-user
branch
2 times, most recently
from
888023b to
5ff4b46
Compare
grgprarup
commented
Jan 10, 2024
| | sharee | Brian | | ||
| | shareType | user | | ||
| | permissionsRole | <permissions-role> | | ||
| Then the HTTP status code should be "200" |
There was a problem hiding this comment.
Note: We can share with disabled users. #7962 (comment)
grgprarup
commented
Jan 10, 2024
| @@ -276,7 +276,7 @@ Feature: add users to group | |||
| Given these groups have been created: | |||
| | groupname | comment | | |||
| | sales | normal group | | |||
| And the user "Admin" has disabled user "Alice" using the Graph API | |||
| And the user "Admin" has disabled user "Alice" | |||
There was a problem hiding this comment.
Refactored as we discussed, we are not using texts like using the Graph API for a given step.
grgprarup
force-pushed
the
add-test-for-invite-disabled-user
branch
from
5ff4b46 to
62baed3
Compare
grgprarup
commented
Jan 10, 2024
| | Co Owner | folder | FolderToShare | | ||
| | Uploader | folder | FolderToShare | | ||
| | Manager | folder | FolderToShare | | ||
| | permissions-role | resource-type | path | |
There was a problem hiding this comment.
Note: kebab-case for example table heading.
grgprarup
requested review from
ScharfViktor,
saw-jan,
amrita-shrestha,
PrajwolAmatya and
nirajacharya2
ScharfViktor
approved these changes
Jan 10, 2024
9 tasks
saw-jan
reviewed
Jan 10, 2024
tests/acceptance/features/apiSharingNg/shareInvitations.feature
Outdated
Show resolved
Hide resolved
grgprarup
force-pushed
the
add-test-for-invite-disabled-user
branch
from
62baed3 to
72fa0cd
Compare
|
PrajwolAmatya
approved these changes
Jan 11, 2024
delete-merged-branch
bot
deleted the
add-test-for-invite-disabled-user
branch
ownclouders
pushed a commit
that referenced
this pull request
Jan 11, 2024
[full-ci] [tests-only] Added test for sending share invitation to disabled user
9 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Scenarios Added:
Related Issue
How Has This Been Tested?
Types of changes
Checklist: