[1LP][RFR] Manual test cleanup and qe-test-coverage test cases

[valaparthvi]

Purpose or Intent

• Adding tests

  1. test_widget_generate_content_via_rest - BZ 1761836
  2. test_service_refresh_dialog_fields_default_values - BZ 1730813
  3. test_schedule_automation_request - BZ 1740340
  4. test_update_roles_via_rest_name_change - BZ 1596142
  5. test_compare_vm_from_datastore_relationships - BZ 1733120

• Removing tests

  1. test_automation_request_task [already automated]
  2. test_date_should_be_change_in_editing_reports_scheduled [already automated]
  3. test_custom_reports_with_timelines

[izapolsk]

LGTM, one thing needs a correction though.

[izapolsk]

REQUIRED: There are already defined appropriate classes. and please use context instead for better readability
@pytest.mark.parametrize('context', [ViaSSUI, ViaUI])

[mshriver]

@valaparthvi I think there was a misunderstanding here - there are actual ViaUI / ViaRest classes that should be parametrized against here, and those context classes can be used directly. Your update still just parametrizes the strings.

Example:

integration_tests/cfme/tests/cloud/test_instance_reconfigure.py

Line 18 in d752d18

@pytest.mark.parametrize('context', [ViaREST, ViaUI])

[valaparthvi]

I understood that, but um okay here's my question: If I use the classes instead of strings, will I also need to use those contexts? Because I am only using this parametrization so that I know which way to verify my response. Should I stil use the classes?

[izapolsk]

@valaparthvi, you aren't required to use these classes in test if it isn't necessary. It will only be used for generating tests.

[mshriver]

Contexts for parametrization should be Via* classes, not strings.