Skip to content
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

[QA][Sharing Ng] Increase tests coverage for list permission #8336

Closed
18 tasks done
amrita-shrestha opened this issue Feb 1, 2024 · 4 comments · Fixed by #8959
Closed
18 tasks done

[QA][Sharing Ng] Increase tests coverage for list permission #8336

amrita-shrestha opened this issue Feb 1, 2024 · 4 comments · Fixed by #8959
Assignees
Labels

Comments

@amrita-shrestha
Copy link
Contributor

amrita-shrestha commented Feb 1, 2024

Describe the bug

Currently, only one scenario is present to test the list permission API endpoint
user lists permissions via the Graph API which covers the list role for personal drive folder resources.
Considering this issue #8131 as a base, different resource type shows different permission lists.

Edges Cases:

@nirajacharya2
Copy link
Contributor

nirajacharya2 commented Apr 8, 2024

The above PRs:
#8476
#8477
mentioned above in the description is blocked until issue #8331 is fixed

@nirajacharya2
Copy link
Contributor

The above PRs:
#8901
#8904
mentioned above in the description is blocked until issue #8922 is fixed

@amrita-shrestha
Copy link
Contributor Author

amrita-shrestha commented Apr 30, 2024

What you feel about closing this issue as #8901 and #8904 are the only PR left to merge blocked by same issue? We are following up on all share-ng reported issues at once in #8428
If you guys feel same please close this issue
@nirajacharya2 @SagarGi

@nirajacharya2
Copy link
Contributor

remaining PRs

The above PRs: #8901 #8904 mentioned above in the description is blocked until issue #8922 is fixed

are being tracked on #8428. So, closing this issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants