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

[1LP][RFR] Fix test: test_ssa_cluster_relationships #10168

Merged
merged 1 commit into from
Jun 5, 2020

Conversation

valaparthvi
Copy link
Contributor

Purpose or Intent

  • Fixing
    1. test_ssa_cluster_relationships

PRT Run

{{ pytest: cfme/tests/cloud_infra_common/test_relationships.py::test_ssa_cluster_relationships -svvv}}

@dajoRH
Copy link
Contributor

dajoRH commented Jun 5, 2020

I detected some fixture changes in commit 23435c3

The local fixture testing_vm is used in the following files:

  • cfme/tests/cloud_infra_common/test_relationships.py
    • test_datastore_relationships
    • test_ssa_cluster_relationships

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

@valaparthvi valaparthvi changed the title [WIPTEST] Fix test: test_ssa_cluster_relationships [RFR] Fix test: test_ssa_cluster_relationships Jun 5, 2020
@dajoRH dajoRH removed the WIP-testing label Jun 5, 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] Fix test: test_ssa_cluster_relationships [1LP][RFR] Fix test: test_ssa_cluster_relationships Jun 5, 2020
@jawatts jawatts merged commit 21b33d6 into ManageIQ:master Jun 5, 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