[WIP]Implemented automated test for add_provider_with_instance_without_name

[mmojzis]

{{ pytest: -k "test_add_provider_with_instance_without_name" -v }}

[dajoRH]

I detected some fixture changes in commit 1ef9180

The local fixture instance_without_name is used in the following files:

• cfme/tests/cloud/test_providers.py
  • test_add_provider_with_instance_without_name

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

[john-dupuy]

Q: Can you use the create_vm fixture?

integration_tests/cfme/fixtures/vm.py

Lines 111 to 118 in 950033f

	@pytest.fixture(scope="function")
	def create_vm(setup_provider, request, provider):
	if request.param:
	template_type = request.param
	else:
	pytest.error('Any appropriate Template was not passed to the fixture.')
	vm_name = _get_vm_name(request)
	return _create_vm(request, template_type, provider, vm_name)

Or just the _create_vm method? That way you can specify the template and name.

[dajoRH]

Would you mind rebasing this Pull Request against latest master, please?
CFME QE Bot

[mmojzis]

@kedark3 @john-dupuy @mshriver
So I would like to make this PR merged.
The problem is that create_vm by default doesn't handle vm without name.
I tried to use _create_vm(), but it didn't work as wrapanapi had no identificator for vm - by default it's name, but when instance doesn't have name it's identified by uuid, but _create_vm doesn't provide one.
As I am leaving by the end of this month I would like to make this PR merged without using create_vm.
I could try to fix the issue, but it would take me probably a lot of time and I don't see the point to use create_vm as it's premise is AFAIK that every VM created by this method has test assignee as a suffix in its name, which cannot be applied for this specific test.
Thanks for feedback.