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

[1LP][RFR]Automate: test_schedule_automation_request #9913

Merged
merged 1 commit into from
Feb 14, 2020

Conversation

valaparthvi
Copy link
Contributor

Purpose or Intent

  • Adding tests
    1. test_schedule_automation_request
  • Enhancement
    1. Add model page for Automation Requests.

PRT Run

{{ pytest: cfme/tests/automate/test_rest.py -k "test_schedule_automation_request" -vvv }}

@valaparthvi valaparthvi added enhancement test-automation To be applied on PR's which are automating existing manual cases labels Feb 11, 2020
@valaparthvi valaparthvi changed the title Automate: test_schedule_automation_request [WIPTEST]Automate: test_schedule_automation_request Feb 11, 2020
@valaparthvi valaparthvi changed the title [WIPTEST]Automate: test_schedule_automation_request [RFR]Automate: test_schedule_automation_request Feb 11, 2020
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.

LGMT 👍

@john-dupuy john-dupuy changed the title [RFR]Automate: test_schedule_automation_request [1LP][RFR]Automate: test_schedule_automation_request Feb 14, 2020
@mshriver mshriver self-assigned this Feb 14, 2020
view = navigate_to(automate_request, "Details")
assert view.is_displayed

def _convert(date):
Copy link
Member

Choose a reason for hiding this comment

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

Bit of overkill putting this in a function, could just define date_format as 'constant' and use the same datetime.strptime call using it.

@mshriver
Copy link
Member

Great work!

@mshriver mshriver merged commit ae8d85d into ManageIQ:master Feb 14, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement 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