integration_tests icon indicating copy to clipboard operation
integration_tests copied to clipboard
verified publisher icon ManageIQ

[WIPTEST] Fixing infra_hosts_power_tests for the CFME 5.10

Open dkholodo opened this issue 7 years ago • 4 comments

Were fixed infra provider's host_off and host_on tests for the CFME 5.10

{{ pytest: cfme/tests/openstack/infrastructure/test_host_power_control.py }}

dkholodo avatar Dec 28 '18 11:12 dkholodo

@dkholodo please sign commit with your GPG key.

digitronik avatar Dec 28 '18 13:12 digitronik

This needs rebase, we don't want to include any merge commits. Please rebase/squash to one (new) commit.

mshriver avatar Jan 03 '19 20:01 mshriver

Lint report for commit 40fbd817efd7dc25659c4086fe83072b30539853:

cfme/infrastructure/host.py:

  • :bangbang: Line 666:101: E501 line too long (106 > 100 characters)

Please, rectify these issues :smirk: .

No commit flaws detected.

CFME QE Bot

dajoRH avatar Jan 17 '19 23:01 dajoRH

I detected some fixture changes in commit 40fbd817efd7dc25659c4086fe83072b30539853

The local fixture host_on is used in the following files:

  • cfme/tests/openstack/infrastructure/test_host_power_control.py
    • test_host_power_off

The local fixture host_off is used in the following files:

  • cfme/tests/openstack/infrastructure/test_host_power_control.py
    • test_host_power_on

Please, consider creating a PRT run against these tests make sure your fixture changes do not break existing usage :smiley:

dajoRH avatar Aug 07 '19 12:08 dajoRH