[WIPTEST] Added fixture to validate repo url from yaml

[ganeshhubale]

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

Purpose or Intent

• Enhanced test cases in test_git_import.py by adding fixture to validate git repo url from cfme_data.yml file
• As per this comment
• Updated skip message too.

PRT Run

{{ pytest: cfme/tests/automate/test_git_import.py -k 'test_automate_git_domain_removed_from_disk or test_automate_git_domain_displayed_in_service or test_automate_git_import_multiple_domains or test_domain_import_git or test_domain_import_git_rest or test_automate_git_import_case_insensitive or test_automate_git_domain_import_connection or test_automate_git_import_without_master or test_git_refresh_with_renamed_yaml' -vvv }}

[dajoRH]

I detected some fixture changes in commit ad253517aa6eb4811023d4a41b9939aba35253b0

The local fixture git_repo_url is used in the following files:

• cfme/tests/automate/test_git_import.py
  • imported_domain
  • test_automate_git_domain_removed_from_disk
  • test_automate_git_domain_displayed_in_service
  • test_automate_git_import_multiple_domains
  • test_domain_import_git
  • test_domain_import_git_rest
  • test_automate_git_import_case_insensitive
  • test_automate_git_domain_import_connection
  • test_automate_git_import_without_master
  • test_git_refresh_with_renamed_yaml

The local fixture imported_domain is used in the following files:

• cfme/tests/automate/test_git_import.py
  • test_automate_git_domain_removed_from_disk
  • test_refresh_git_current_user
  • test_automate_git_verify_ssl
  • test_automate_git_import_deleted_tag

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

[jarovo]

Required: I don't think there is a reason for use of delattr when we can use del

Suggested change

	delattr(self.obj, "locked")
	del self.obj.locked

This may help with code analysis in some code editors and re-factoring tools.