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

[1LP][RFR] CLI - Verifying log messages on failure of DB configuration #9884

Merged

Conversation

dgaikwad
Copy link
Contributor

Purpose or Intent

adding_test to verifying log messages on the failure of DB configuration

PRT Run

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

@dgaikwad dgaikwad force-pushed the test_appliance_console_vmdb_log_negative branch 2 times, most recently from 5339a42 to c19188f Compare January 28, 2020 10:23
@dgaikwad dgaikwad changed the title [WIPTEST] CLI - Verifying log messages on failure of DB configuration [RFR] CLI - Verifying log messages on failure of DB configuration Jan 28, 2020
Copy link
Contributor

@sbulage sbulage left a comment

Choose a reason for hiding this comment

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

Thanks for the PR @dgaikwad. LGTM 👍

@jawatts
Copy link
Contributor

jawatts commented Jan 28, 2020

Looks good, @dgaikwad can you fix your commit history.

@jawatts jawatts changed the title [RFR] CLI - Verifying log messages on failure of DB configuration [1LP][WIPTEST] CLI - Verifying log messages on failure of DB configuration Jan 28, 2020
@dgaikwad dgaikwad force-pushed the test_appliance_console_vmdb_log_negative branch from c19188f to 7be734a Compare February 3, 2020 13:16
@dgaikwad dgaikwad force-pushed the test_appliance_console_vmdb_log_negative branch from 7be734a to 91633a4 Compare February 11, 2020 13:04
@dgaikwad dgaikwad force-pushed the test_appliance_console_vmdb_log_negative branch from 91633a4 to 11accfa Compare February 12, 2020 13:05
@dgaikwad dgaikwad changed the title [1LP][WIPTEST] CLI - Verifying log messages on failure of DB configuration [1LP][RFR] CLI - Verifying log messages on failure of DB configuration Feb 12, 2020
@jawatts jawatts merged commit 50ca060 into ManageIQ:master Feb 12, 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.

5 participants