Skip to content
This repository has been archived by the owner on Apr 7, 2022. It is now read-only.

[1LP][RFR] Remove references to has_no_cloud_providers and has_no_infra_providers #10167

Merged
merged 1 commit into from
Jun 5, 2020

Conversation

valaparthvi
Copy link
Contributor

@valaparthvi valaparthvi commented Jun 5, 2020

Purpose or Intent

  • Updating tests
    1. Removing reference to fixtures such as has_no_cloud_providers and has_no_infra_providers and replacing them with has_no_providers.

PRT Run

{{ pytest: cfme/tests -k "test_add_cancelled_validation_cloud or test_cloud_provider_add_with_bad_credentials or test_openstack_provider_has_dashboard or test_select_key_pair_none_while_provisioning or test_create_azure_vm_from_azure_image or test_public_images_enable_disable or test_api_port_max_character_validation_infra or test_providers_discovery or test_infra_provider_add_with_bad_credentials or test_infra_provider_crud or test_no_template_catalog_item" --long-running -svvv }}

@dajoRH dajoRH added lint-ok and removed needs-lint labels Jun 5, 2020
@valaparthvi valaparthvi changed the title [WIPTEST] Remove references to has_no_cloud_providers and has_no_infra_providers [RFR] Remove references to has_no_cloud_providers and has_no_infra_providers Jun 5, 2020
@dajoRH dajoRH removed the WIP-testing label Jun 5, 2020
Copy link
Contributor

@john-dupuy john-dupuy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@john-dupuy john-dupuy changed the title [RFR] Remove references to has_no_cloud_providers and has_no_infra_providers [1LP][RFR] Remove references to has_no_cloud_providers and has_no_infra_providers Jun 5, 2020
@digitronik digitronik merged commit 20d46ee into ManageIQ:master Jun 5, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants