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

[RFR]Automate: test_database_wildcard_should_work_and_be_included_in_the_query #10137

Merged
merged 1 commit into from
Jun 1, 2020

Conversation

valaparthvi
Copy link
Contributor

Purpose or Intent

  • Extending
    1. cfme.rest.gen_data.vm to take kwargs and a custom name.
  • Adding tests
    2. Automate : test_database_wildcard_should_work_and_be_included_in_the_query

PRT Run

{{ pytest: cfme/tests/infrastructure/test_vm_rest.py -k "test_database_wildcard_should_work_and_be_included_in_the_query" -svvv }}

@valaparthvi valaparthvi added the test-automation To be applied on PR's which are automating existing manual cases label May 21, 2020
@valaparthvi valaparthvi changed the title [WIPTEST]Automate: test_database_wildcard_should_work_and_be_included_in_the_query [RFR]Automate: test_database_wildcard_should_work_and_be_included_in_the_query May 26, 2020
@dgaikwad
Copy link
Contributor

LGTM

initialEstimate: 1/4h
startsin: 5.10
testSteps:
1. Create a VM with some name, for e.g test-25-xyz.
Copy link
Member

Choose a reason for hiding this comment

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

For this test and others in the module, its unnecessary to create a VM if there is one that exists on the provider already that has been inventoried by MIQ.

Not required for this PR, but consider using a module scoped fixture to provide the VM for this test and others within this module that don't actually make any modifications to the VM. The fixture can save a lot of wasted time in these test functions by just using an existing VM.

@mshriver mshriver merged commit 8965060 into ManageIQ:master Jun 1, 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