-
Notifications
You must be signed in to change notification settings - Fork 180
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[tests-only] adding test for sharedWithMe end point for resource in project space and personal space #8292
Conversation
d5d8d7f
to
7a6ba63
Compare
f877061
to
80af4fb
Compare
16d96e9
to
84f35e1
Compare
84f35e1
to
8a808fa
Compare
|
@nirajacharya2 Please, update the PR. the fix PR has been merged |
8a808fa
to
55abef7
Compare
f71cfb8
to
4e43e0b
Compare
And the administrator has deleted user "Alice" using the provisioning API | ||
When user "Brian" lists the shares shared with him using the Graph API | ||
Then the HTTP status code should be "200" | ||
""" | ||
{ | ||
"type": "object", | ||
"required": [ | ||
"value" | ||
], | ||
"properties": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we have the share even the sharer is deleted?
CC @SagarGi
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
With the sharer deleted his/her resource is deleted AFAIK. And i think we should not have any shared responses. @ScharfViktor might have some details regarding it. i have created an issue for it which might be related to it. lets wait if its the actual bug> #8356
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we have the share even the sharer is deleted?
CC @SagarGi
yes, but for project space only. In personal space the share is deleted. We have different behaviour in project space and personal space. my assumption is that in project spaces the owner is set to a special user of the CS3 #8314 (comment) so deleting the user dose not delete the share. need clarification
CC @ScharfViktor
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I see that it's correct behavior
personal space: sharer is deleted - no share anymore
project space: sharer is deleted - share is exist. Another user manager can manage this share
d2ea2b4
to
8fb56db
Compare
5ab1d03
to
3687e59
Compare
319263a
to
475282c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
475282c
to
ba8ce37
Compare
ba8ce37
to
e190d69
Compare
Quality Gate passedIssues Measures |
""" | ||
|
||
|
||
Scenario: user lists the file shared with them from personal space after the sharer is deleted |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please, add scenarios per item from #7826 and the PR title would be meaningful.
Also, it will be easier to manage the PRs and scenarios
…wncloud#8292) (cherry picked from commit fb1d933)
Description
Scenarios added :
Related Issue
sharedWithMe
#7826Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: