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

[1LP][RFR] Adding tower bundle retire test #9990

Merged
merged 1 commit into from
Mar 26, 2020

Conversation

niyazRedhat
Copy link
Contributor

@niyazRedhat niyazRedhat commented Mar 16, 2020

Purpose or Intent

PRT Run

{{pytest: cfme/tests/automate/test_ansible_tower_automate.py::test_retire_ansible_service_bundle -vvvv }}

@niyazRedhat niyazRedhat changed the title [WIPTEST] Adding tower bundle retire test [RFR] Adding tower bundle retire test Mar 26, 2020
@niyazRedhat
Copy link
Contributor Author

All ansible tower tests getting skipped or not collecting tests due to problems with Tower provider. Nandini trying to figure out the issue.
We tried this test locally as per Nandini suggestion and it passed.
Replaced version: 3.4 with version: 3.1 and --use-provider ansible_tower_34

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.

Test case LGTM, I'm just wondering if some of the setup can be moved into fixtures?

@john-dupuy john-dupuy changed the title [RFR] Adding tower bundle retire test [1LP][RFR] Adding tower bundle retire test Mar 26, 2020
@mshriver mshriver self-assigned this Mar 26, 2020
@mshriver mshriver merged commit 129a5d8 into ManageIQ:master Mar 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.

4 participants