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

[1LP][RFR] CLI - Test to fail DB config command - negative #9836

Merged
merged 1 commit into from
Jan 14, 2020

Conversation

dgaikwad
Copy link
Contributor

Purpose or Intent

Adding tests testing DB configuration command by providing the disk which is not available on the host

PRT Run

{{pytest: ./cfme/tests/cli/test_appliance_console.py::test_ap_failed_dbconfig_status -v}}

@pytest.mark.tier(2)
@pytest.mark.meta(coverage=[1753687])
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Bug 1753687 is closed so removed this marker

Copy link
Contributor

Choose a reason for hiding this comment

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

please add @pytest.mark.meta(automates=[BZ(1753687)]) as this BZ is now automated.

@dgaikwad dgaikwad changed the title [WIPTEST] CLI - Test to fail DB config command by providing disk - negative [WIPTEST] CLI - Test to fail DB config command - negative Jan 13, 2020
@dgaikwad dgaikwad changed the title [WIPTEST] CLI - Test to fail DB config command - negative [RFR] CLI - Test to fail DB config command - negative Jan 13, 2020
@john-dupuy john-dupuy added the test-automation To be applied on PR's which are automating existing manual cases label Jan 13, 2020
@pytest.mark.tier(2)
@pytest.mark.meta(coverage=[1753687])
Copy link
Contributor

Choose a reason for hiding this comment

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

please add @pytest.mark.meta(automates=[BZ(1753687)]) as this BZ is now automated.

Copy link
Contributor

@sshveta sshveta left a comment

Choose a reason for hiding this comment

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

Looks good @dgaikwad , one small comment

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.

Just one question, awesome stuff automating these test cases 👍

cfme/tests/cli/test_appliance_console.py Show resolved Hide resolved
@dgaikwad dgaikwad force-pushed the test_ap_failed_dbconfig_status branch from 173950f to b5ada29 Compare January 14, 2020 09:05
@sshveta sshveta changed the title [RFR] CLI - Test to fail DB config command - negative [1LP][RFR] CLI - Test to fail DB config command - negative Jan 14, 2020
@jawatts jawatts self-assigned this Jan 14, 2020
@jawatts jawatts merged commit 8b426c8 into ManageIQ:master Jan 14, 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.

5 participants