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

[RFR] Added two button fixture and automated manual test#10258

Merged
mshriver merged 2 commits intoManageIQ:masterfrom
spusateri:Generic_obj_custom_button
Jul 17, 2020
Merged

[RFR] Added two button fixture and automated manual test#10258
mshriver merged 2 commits intoManageIQ:masterfrom
spusateri:Generic_obj_custom_button

Conversation

@spusateri
Copy link
Contributor

@spusateri spusateri commented Jul 17, 2020

Purpose or Intent

PRT Run

{{pytest: cfme/tests/automate/generic_objects/buttons/test_generic_class_custom_button.py -k "test_generic_object_button_delete_multiple" -v}}

@dajoRH
Copy link
Contributor

dajoRH commented Jul 17, 2020

I detected some fixture changes in commit 4c28e5a

The local fixture two_buttons_without_group is used in the following files:

  • cfme/tests/automate/generic_objects/buttons/test_generic_class_custom_button.py
    • test_generic_object_button_delete_multiple

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

@spusateri spusateri changed the title [WIPTEST] Added two button fixture and automated manual test [RFR] Added two button fixture and automated manual test Jul 17, 2020
@mshriver mshriver added the test-automation To be applied on PR's which are automating existing manual cases label Jul 17, 2020
@spusateri spusateri changed the title [RFR] Added two button fixture and automated manual test [WIPTEST] Added two button fixture and automated manual test Jul 17, 2020
@mshriver
Copy link
Member

PRT failing at the moment because of wharf connection timeouts.

@spusateri spusateri changed the title [WIPTEST] Added two button fixture and automated manual test [RFR] Added two button fixture and automated manual test Jul 17, 2020


@pytest.mark.manual
@pytest.mark.meta(blockers=[BZ(1753281), BZ(1753388)], coverage=[1753281, 1753388])
Copy link
Member

Choose a reason for hiding this comment

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

The coverage list here should be updated to automates, as the meta marker processes them differently for BZ reports.

@mshriver mshriver merged commit 52be709 into ManageIQ:master Jul 17, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

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.

3 participants