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

[WIPTEST] Automating MA test case #9938

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ganeshhubale
Copy link
Member

@ganeshhubale ganeshhubale commented Feb 20, 2020

Signed-off-by: Ganesh Hubale [email protected]

Purpose or Intent

  • Automating MA test case:
    • test_enable_migration_analytics
    • test_confirm_environment_summary_data
  • Adding views, models, collections related to it

PRT Run

{{ pytest: cfme/tests/migration_analytics/test_migration_analytics.py -k 'test_enable_migration_analytics or test_confirm_environment_summary_data' --use-template-cache -qsvvv }}

@ganeshhubale ganeshhubale added enhancement test-automation To be applied on PR's which are automating existing manual cases labels Feb 20, 2020
@ganeshhubale ganeshhubale changed the title [WIP] Automating MA test case [WIPTEST] Automating MA test case Feb 20, 2020
@dajoRH dajoRH added WIP-testing and removed WIP labels Feb 20, 2020
… to it

1. test_enable_migration_analytics
2. test_confirm_environment_summary_data

Signed-off-by: Ganesh Hubale <[email protected]>
@dajoRH
Copy link
Contributor

dajoRH commented Feb 20, 2020

I detected some fixture changes in commit 89f182d

The local fixture enable_migration is used in the following files:

  • cfme/tests/migration_analytics/test_migration_analytics.py
    • test_enable_migration_analytics
    • test_confirm_environment_summary_data

Please, consider creating a PRT run to make sure your fixture changes do not break existing usage 😃

def enable(is_enable):
yaml_data = {"prototype": {"migration_analytics": {"enabled": is_enable}}}
appliance.update_advanced_settings(yaml_data)
appliance.ssh_client.run_command('systemctl restart evmserverd')
Copy link
Contributor

Choose a reason for hiding this comment

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

Optional: We can use appliance.evmserverd.restart() here.

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 WIP-testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants