-
Notifications
You must be signed in to change notification settings - Fork 55
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 unit tests for Minion Manager RPC Server #258
Conversation
0cfd1bf
to
e62e76f
Compare
e62e76f
to
1d0e33c
Compare
1d0e33c
to
a2abaef
Compare
@Dany9966 I removed the manual exception mappings using |
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.
Also please don't forget to sync/rebase with origin, so the pep8 action applies to this PR too
coriolis/tests/minion_manager/rpc/data/validate_minion_pool_selections_for_action_config.yaml
Show resolved
Hide resolved
f4d8b77
to
9189250
Compare
@Dany9966 All changes from conversions marked as resolved are now pushed |
coriolis/tests/minion_manager/rpc/data/validate_minion_pool_selections_for_action_config.yaml
Show resolved
Hide resolved
coriolis/tests/minion_manager/rpc/data/make_minion_machine_allocation_subflow_for_action.yaml
Show resolved
Hide resolved
coriolis/tests/minion_manager/rpc/data/get_minion_pool_refresh_flow.yaml
Show resolved
Hide resolved
coriolis/tests/minion_manager/rpc/data/get_minion_pool_refresh_flow.yaml
Show resolved
Hide resolved
coriolis/tests/minion_manager/rpc/data/get_minion_pool_refresh_flow.yaml
Outdated
Show resolved
Hide resolved
coriolis/tests/minion_manager/rpc/data/get_minion_pool_refresh_flow.yaml
Outdated
Show resolved
Hide resolved
9189250
to
8ee1eb0
Compare
@Dany9966 All conversations have been resolved. |
8ee1eb0
to
343dbe1
Compare
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! Thank you!
No description provided.