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

[1LP][RFR] Automating bundle test #bz 1671522 #9986

Merged
merged 1 commit into from
Mar 24, 2020

Conversation

niyazRedhat
Copy link
Contributor

Purpose or Intent

PRT Run

{{pytest: cfme/tests/services/test_catalog_item.py::test_add_bundle_in_bundle --long-running }}

@niyazRedhat niyazRedhat changed the title [WIPTEST] Automating bundle test #bz 1671522 [RFR] Automating bundle test #bz 1671522 Mar 16, 2020
@ganeshhubale ganeshhubale added the test-automation To be applied on PR's which are automating existing manual cases label Mar 18, 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] Automating bundle test #bz 1671522 [1LP][RFR] Automating bundle test #bz 1671522 Mar 18, 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.

LGTM 👍
try to avoid browser refresh.

3. There should be no bundle in resources list
"""
view = navigate_to(appliance.collections.catalog_bundles, "Add")
request.addfinalizer(lambda: view.browser.refresh())
Copy link
Contributor

Choose a reason for hiding this comment

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

I would not refresh browser. In state of this
is it ok to click on cancel button if view is displayed?

@digitronik digitronik self-assigned this Mar 24, 2020
@digitronik digitronik changed the title [1LP][RFR] Automating bundle test #bz 1671522 [1LP][WIPTEST] Automating bundle test #bz 1671522 Mar 24, 2020
@niyazRedhat niyazRedhat changed the title [1LP][WIPTEST] Automating bundle test #bz 1671522 [1LP][RFR] Automating bundle test #bz 1671522 Mar 24, 2020
@digitronik digitronik merged commit b5bc6f9 into ManageIQ:master Mar 24, 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.

4 participants