[1LP][RFR] Fix test_embedded_ansible_actions

[Gauravtalreja1]

Purpose or Intent

PRT Run

{{pytest: cfme/tests/ansible/test_embedded_ansible_actions.py --long-running }}

[valaparthvi]

One comment, if you agree, make sure to change it everywhere. LGTM otherwise.

[dajoRH]

I detected some fixture changes in commit 6988abb

The local fixture ansible_action is used in the following files:

• cfme/tests/ansible/test_embedded_ansible_actions.py
  • policy_for_testing

The local fixture policy_for_testing is used in the following files:

• cfme/tests/ansible/test_embedded_ansible_actions.py

The local fixture ansible_credential is used in the following files:

• cfme/tests/ansible/test_embedded_ansible_actions.py

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

[mshriver]

When you apply black formatting to the module in a PR that is fixing the test, its hard to tell what is actually changing test behavior and what's just auto-formatted.

Please, in future PRs, refrain from including auto-formatting changes along with functional changes.

pre-commit config is excluding black formatting from most of the repo - if you're interested in including more of the repository in our auto-formatting, that's great! Just open separate PRs updating the pre-commit configuration for black, and applying it to the given directories.

Otherwise the next person that modifies this file won't be adhering to the same formatting as what you've applied here.