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

[1LP][RFR] Automate: test_retire_multiple_services #10107

Merged
merged 1 commit into from
May 19, 2020

Conversation

valaparthvi
Copy link
Contributor

@valaparthvi valaparthvi commented May 12, 2020

Purpose or Intent

  • Adding tests
    1. test_retire_multiple_services
  • Enhancement
    1. Add teardown to catalog_item fixture.
    2. enhance services helper function to take a service_template.
    3. Remove unnecessary code from helper functions in gen_data.py.

PRT Run

{{ pytest: cfme/tests/services/test_myservice.py -k "test_retire_multiple_services" --long-running}}

@valaparthvi valaparthvi added the test-automation To be applied on PR's which are automating existing manual cases label May 12, 2020
@dajoRH
Copy link
Contributor

dajoRH commented May 14, 2020

I detected some fixture changes in commit 961fa06

Show fixtures

The global fixture catalog_item is used in the following files:

  • cfme/tests/ansible/test_embedded_ansible_automate.py
  • cfme/tests/automate/custom_button/test_catalog_custom_button.py
  • cfme/tests/automate/test_ansible_tower_automate.py
  • cfme/tests/automate/test_service_automate.py
  • cfme/tests/cloud_infra_common/test_html5_vm_console_ssui.py
  • cfme/tests/services/test_dialog_element_in_catalog.py
  • cfme/tests/services/test_dialog_regex_validation_in_catalog.py
  • cfme/tests/services/test_dynamicdd_dialogelement.py
  • cfme/tests/services/test_myservice.py
    • test_service_state
    • services_vms_list
    • test_retire_service_bundle_and_vms
  • cfme/tests/services/test_request.py
    • test_copy_request_bz1194479
  • cfme/tests/services/test_service_catalogs.py
    • test_order_catalog_item
    • test_order_catalog_bundle
    • test_request_with_orphaned_template
    • test_order_service_after_deleting_provider
  • cfme/tests/services/test_service_customer_bz.py
    • test_edit_bundle_entry_point
  • cfme/tests/services/test_service_manual_approval.py
  • cfme/tests/services/test_service_rbac.py
    • test_service_rbac_catalog_item
    • test_service_rbac_request
  • cfme/tests/ssui/test_ssui_myservice.py
  • cfme/tests/ssui/test_ssui_service_catalogs.py

The local fixture services_vms_list is used in the following files:

  • cfme/tests/services/test_myservice.py
    • test_retire_multiple_services

Please, consider creating a PRT run to make sure your fixture changes do not break existing usage 😃

@dajoRH dajoRH added lint-ok and removed needs-lint labels May 14, 2020
@valaparthvi valaparthvi changed the title [WIPTEST] Automate: test_retire_multiple_services [RFR] Automate: test_retire_multiple_services May 18, 2020
Copy link
Contributor

@niyazRedhat niyazRedhat left a comment

Choose a reason for hiding this comment

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

LGTM

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, thanks @valaparthvi

@john-dupuy john-dupuy changed the title [RFR] Automate: test_retire_multiple_services [1LP][RFR] Automate: test_retire_multiple_services May 19, 2020
@mshriver mshriver merged commit 764635b into ManageIQ:master May 19, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lint-ok test-automation To be applied on PR's which are automating existing manual cases
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants