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

[1LP][RFR] Automate policy_profiles test for rest #10208

Merged
merged 1 commit into from
Jun 26, 2020

Conversation

valaparthvi
Copy link
Contributor

@valaparthvi valaparthvi commented Jun 24, 2020

Purpose or Intent

  • Adding tests

    1. TestPolicyProfilesRESTAPI
  • Enhancement

    1. Add policy_profiles helper function in cfme/rest/gen_data.py to create policy_profile via REST.

PRT Run

{{ pytest: cfme/tests/control/test_rest_control.py -k "TestPolicyProfilesREST" -svvv }}

@dajoRH
Copy link
Contributor

dajoRH commented Jun 24, 2020

I detected some fixture changes in commit ec5fc13

The local fixture policy_profiles is used in the following files:

  • cfme/tests/control/test_rest_control.py
    • test_query_policy_attributes
    • test_create_policy_profiles
    • test_edit_policy_profiles
    • test_delete_policy_profiles_from_detail
    • test_delete_policy_profiles_from_collection

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

@valaparthvi valaparthvi changed the title [WIPTEST] Automate policy_profiles test for rest [RFR] Automate policy_profiles test for rest Jun 24, 2020
Copy link
Member

@ganeshhubale ganeshhubale left a comment

Choose a reason for hiding this comment

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

LGTM :)

@ganeshhubale ganeshhubale changed the title [RFR] Automate policy_profiles test for rest [1LP][RFR] Automate policy_profiles test for rest Jun 25, 2020
@digitronik digitronik self-assigned this Jun 26, 2020
Copy link
Contributor

@digitronik digitronik left a comment

Choose a reason for hiding this comment

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

I like edit name indexing :)

@digitronik digitronik merged commit 42ab189 into ManageIQ:master Jun 26, 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.

None yet

4 participants