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

[1LP][RFR]Fix YAML data for scheduling a report #7994

Merged
merged 1 commit into from Oct 24, 2018

Conversation

valaparthvi
Copy link
Contributor

@valaparthvi valaparthvi commented Oct 15, 2018

This PR introduces fix to YAML data for scheduling a report. There is a change in the name of txt, csv and pdf. Until this change, txt, csv and pdf were not selected even if they were set to True in the data, since they were not configured properly.

{{ pytest: cfme/tests/intelligence/reports/test_crud.py::test_schedule_crud --use-template-cache -sqvvv }}

@valaparthvi valaparthvi changed the title [WIPTEST][NOTEST]Fix data for scheduling a report in YAML files [WIPTEST][NOTEST]Fix YAML data for scheduling a report Oct 15, 2018
@valaparthvi valaparthvi changed the title [WIPTEST][NOTEST]Fix YAML data for scheduling a report [RFR][NOTEST]Fix YAML data for scheduling a report Oct 15, 2018
Copy link
Contributor

@digitronik digitronik left a comment

Choose a reason for hiding this comment

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

LGTM 👍 @valaparthvi run one test which used this email options.

@valaparthvi valaparthvi changed the title [RFR][NOTEST]Fix YAML data for scheduling a report [WIPTEST]Fix YAML data for scheduling a report Oct 23, 2018
@valaparthvi valaparthvi changed the title [WIPTEST]Fix YAML data for scheduling a report [RFR]Fix YAML data for scheduling a report Oct 23, 2018
@digitronik digitronik changed the title [RFR]Fix YAML data for scheduling a report [1LP][RFR]Fix YAML data for scheduling a report Oct 23, 2018
@mshriver mshriver merged commit eff3eed into ManageIQ:master Oct 24, 2018
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.

None yet

3 participants