-
Notifications
You must be signed in to change notification settings - Fork 115
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
Remove cli.factory dependencies from the repo #11544
Conversation
36a9477
to
be44675
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.
We are not suppose to use 'cli/factory' module at all. Everything related CLI helpers should be coming from 'cli_factory' object.
This pull request has not been updated in the past 45 days. |
@shweta83 Are you ok to resume on this ? Just FYI, Automation FG is planning for this kind of cleanup in Q3, this PR would be considered one of them. |
Yes. I am planning to resume this in later this week. |
This pull request has not been updated in the past 45 days. |
ffae036
to
33bb1cf
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.
@shweta83 Looks like the changes made here are doing the opposite of what we are supposed to do :)
Please refer comments or else we can sync up !
33bb1cf
to
4063004
Compare
Remove cli_factory
This PR will remove the dependency of cli.factory from robottelo and cli_factory will be used in place of that.
All the test modules using cli.factory method should be replaced with cli_factory methods as we are deprecating cli.factory.
Old methods -> New methods
make_activation_key -> make_activation_key
make_architecture -> make_architecture
make_content_view -> make_content_view
make_content_view_with_credentials -> content_view_with_credentials
make_content_view_filter -> content_view_filter
make_content_view_filter_rule -> content_view_filter_rule
make_discoveryrule -> discoveryrule
make_content_credential -> make_content_credential
make_location -> make_location
make_model -> model
make_partition_table -> make_partition_table
make_product -> make_product
make_product_with_credentials -> product_with_credentials
make_product_wait -> make_product_wait
make_repository_with_credentials -> repository_with_credentials
make_role -> role
make_filter -> make_filter
make_scap_policy -> scap_policy
make_subnet -> subnet
make_sync_plan -> sync_plan
make_host -> make_host
make_fake_host -> make_fake_host
make_host_collection -> host_collection
make_job_invocation -> job_invocation
make_job_invocation_with_credentials -> Removed as not used anywhere
make_job_template -> job_template
make_user -> user
make_usergroup -> usergroup
make_usergroup_external --> usergroup_external
make_compute_resource -> compute_resource
make_org -> make_org
make_org_with_credentials -> org_with_credentials
make_realm -> realm
make_reporttemplate -> reporttemplate
make_os -> os
make_scapcontent -> scapcontent
make_domain -> make_domain
make_hostgroup -> hostgroup
make_medium -> make_medium
make_environment -> ?
make_lifecycle_environment -> make_lifecycle_environment
make_tailoringfile -> tailoringfile
make_template -> make_template
make_template_input -> template_input
make_virt_who_config -> virt_who_config
activationkey_add_subscription_to_repo -> activationkey_add_subscription_to_repo
setup_org_for_a_custom_repo -> setup_org_for_a_custom_repo
_setup_org_for_a_rh_repo -> _setup_org_for_a_rh_repo
setup_org_for_a_rh_repo -> setup_org_for_a_rh_repo
add_role_permissions -> add_role_permissions
setup_cdn_and_custom_repositories -> setup_cdn_and_custom_repositories
setup_cdn_and_custom_repos_content -> setup_cdn_and_custom_repos_content
make_http_proxy -> Not used anywhere