-
Notifications
You must be signed in to change notification settings - Fork 7
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
List roles #2496
List roles #2496
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2496 +/- ##
===========================================
- Coverage 85.33% 74.92% -10.42%
===========================================
Files 71 66 -5
Lines 11828 11073 -755
Branches 2293 2146 -147
===========================================
- Hits 10094 8296 -1798
- Misses 1356 2474 +1118
+ Partials 378 303 -75
Continue to review full report at Codecov.
|
@@ -102,6 +102,16 @@ async def get_shares( | |||
ret.append(Share(item["user"], Permission(uri, action))) | |||
return ret | |||
|
|||
async def get_subroles( |
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.
Would you write a test for this method?
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.
Sorry, forgot about it. Done.
ca0376d
to
c003511
Compare
Blocked by https://github.com/neuro-inc/platform-auth/pull/692
Closes #2395