-
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
Feat/provisioning module #486
Conversation
GitHub Action status on 692d6c1 generated by comment-failure-action Source code security scan using Bandit
Lint, format and test codeNo jobs failed 👍 Build containers CINo jobs failed 👍 |
members_details = kwargs.get("members_details", True) | ||
|
||
groups = [] | ||
match integration_source: |
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.
So much better than several if statements!
app/modules/provisioning/groups.py
Outdated
return groups | ||
|
||
|
||
# def get_matching_groups(source_groups, target_groups, matching_key): |
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 assume you want to keep the commented code here?
assert not mock_google_list_groups_with_members.called | ||
|
||
|
||
# @patch("modules.provisioning.groups.filter_tools.get_nested_value") |
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.
Do you need this commented code here?
GitHub Action status on b43c133 generated by comment-failure-action Lint, format and test code
Build containers CINo jobs failed 👍 |
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.
Excellent work! And love how many unit tests are there now!
Summary | Résumé