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

Update FA assignee #9882

Merged
merged 2 commits into from
Jan 28, 2020
Merged

Update FA assignee #9882

merged 2 commits into from
Jan 28, 2020

Conversation

spusateri
Copy link
Contributor

@spusateri spusateri commented Jan 27, 2020 •
edited
Loading

Updated assignee for ownership FA and genealogy test cleanup.

@mshriver mshriver added the test-cleanup Test removal, collection changes, re-organization label Jan 28, 2020
@dajoRH
Copy link
Contributor

dajoRH commented Jan 28, 2020

I detected some fixture changes in commit 816ab7c13a7ed230e6bfb88d0bb0bd687b1a1b21

The local fixture setup_provider_min_hosts is used in the following files:

  • cfme/tests/infrastructure/test_host.py
    • test_infrastructure_hosts_refresh_multi
    • test_compare_hosts_from_provider_allhosts
    • test_infrastructure_hosts_crud

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

@spusateri
Updated VM ownership assignee field
50853d5 
Removed incomplete test

Removed extra spacing and new lines
@mshriver mshriver merged commit adefc3b into ManageIQ:master Jan 28, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
lint-ok test-cleanup Test removal, collection changes, re-organization
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@spusateri @dajoRH @mshriver