-
Notifications
You must be signed in to change notification settings - Fork 186
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][full-ci] added test to remove access to share after the share role has been disabled #10759
Conversation
3c3c571
to
4a260ce
Compare
4a260ce
to
31f43da
Compare
tests/acceptance/features/apiSharingNg1/removeAccessToDrive.feature
Outdated
Show resolved
Hide resolved
tests/acceptance/features/apiSharingNg1/removeAccessToDriveItem.feature
Outdated
Show resolved
Hide resolved
tests/acceptance/features/apiSharingNg1/removeAccessToDriveItem.feature
Outdated
Show resolved
Hide resolved
tests/acceptance/features/apiSharingNg1/removeAccessToDriveItem.feature
Outdated
Show resolved
Hide resolved
tests/acceptance/features/apiSharingNg1/removeAccessToDriveItem.feature
Outdated
Show resolved
Hide resolved
31f43da
to
3c75699
Compare
3c75699
to
1e6455c
Compare
Quality Gate passedIssues Measures |
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 but do we have tests coverage for group sharee
We don't but is it necessary for these cases? |
yeah need to add test for group too but You can do that in next PR |
[tests-only][full-ci] added test to remove access to share after the share role has been disabled
Description
This PR adds the test to remove access to share after the share role has been disabled.
Related Issue
Types of changes
Checklist: