-
Notifications
You must be signed in to change notification settings - Fork 1
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
Add function to remove share invite using root endpoint #227
Conversation
23cffe4
to
36c98c1
Compare
e83bed1
to
1958911
Compare
b50f27c
to
13ded81
Compare
13ded81
to
56af5dc
Compare
$managerRole = null; | ||
|
||
if (getenv('OCIS_VERSION') === "stable") { | ||
$this->markTestSkipped('Ocis version < 6.0.0 does not support creation of drive share invite so delete drive share test has been skipped'); |
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.
This is correct. It is difficult, or "impossible", to test what happens in ocis V5 if someone tries to delete a drive share, because it is not possible to create a drive share in the first place.
Someone could try to delete a "bogus" drive share, putting some non-existent id. But that would be the same as any "completely wrong" attempt to use the non-existent end-point, and that is already covered by ocis tests.
3408d8a
to
197e7f8
Compare
Quality Gate passedIssues Measures |
Add function to remove share invite using root endpoint
This PR refactor delete function of Share to work on both permission and root endpoint.
In this PR, ResourceShare has been introduced. Share class is parent class for DriveShare and ResourceShare. ResourceShare will be parent class for ShareLink, ShareCreated.
Part of