Merge pull request #8989 from izapolsk/multi-region-tests

[1LP][RFR] Adding Multi-Region FA requirement and appropriate tests

Author: mshriver

cfme/test_requirements.py

@@ -120,12 +120,6 @@ def test_quota_alert():
     assignee_id='ndhandre',
 )
 
-multi_tenancy = pytest.mark.requirement(
-    "MIQ Multi-Tenancy",
-    description='Support for Tenants in MIQ/CFME',
-    assignee_id='nachandr',
-)
-
 chargeback = pytest.mark.requirement(
     "Chargeback",
     description='Chargeback rates, calculations, and reports',
@@ -234,6 +228,18 @@ def test_quota_alert():
     assignee_id='anikifor',
 )
 
+multi_region = pytest.mark.requirement(
+    "MIQ Multi-Region deployments",
+    description='Multi-Region deployment and Central Administration',
+    assignee_id='izapolsk',
+)
+
+multi_tenancy = pytest.mark.requirement(
+    "MIQ Multi-Tenancy",
+    description='Support for Tenants in MIQ/CFME',
+    assignee_id='nachandr',
+)
+
 ownership = pytest.mark.requirement(
     "VM/Instance Ownership",
     description='Ownership settings for VM/Instances',

cfme/tests/cloud_infra_common/test_power_control_manual.py

@@ -2,6 +2,9 @@
 import pytest
 
 from cfme import test_requirements
+from cfme.utils.appliance import ViaREST
+from cfme.utils.appliance import ViaUI
+
 
 pytestmark = [test_requirements.power, pytest.mark.manual]
 
@@ -221,16 +224,19 @@ def test_power_controls_on_vm_in_stack_cloud():
     pass
 
 
+@pytest.mark.manual
 @pytest.mark.tier(2)
-@pytest.mark.manual('manualonly')
-def test_power_operations_from_global_region():
+@pytest.mark.parametrize('context', [ViaREST, ViaUI])
+@test_requirements.multi_region
+@test_requirements.power
+def test_power_operations_from_global_region(context):
     """
     This test case is to check power operations from Global region
     Setup is 2 or more appliances(DB should be configured manually). One
     is Global region, others are Remote. To get this working enable Central Admin.
 
     Polarion:
-        assignee: ghubale
+        assignee: izapolsk
         initialEstimate: 1/2h
         caseimportance: medium
         caseposneg: positive

cfme/tests/cloud_infra_common/test_provisioning.py

@@ -17,6 +17,8 @@
 from cfme.infrastructure.provider.scvmm import SCVMMProvider
 from cfme.markers.env_markers.provider import providers
 from cfme.utils import normalize_text
+from cfme.utils.appliance import ViaREST
+from cfme.utils.appliance import ViaUI
 from cfme.utils.appliance.implementations.ui import navigate_to
 from cfme.utils.blockers import BZ
 from cfme.utils.generators import random_vm_name
@@ -616,3 +618,19 @@ def test_provision_with_tag(appliance, vm_name, tag, provider, request):
         instance_tag.category.display_name == tag.category.display_name and
         instance_tag.display_name == tag.display_name for instance_tag in tags), (
         "{}: {} not in ({})".format(tag.category.display_name, tag.display_name, str(tags)))
+
+
+@pytest.mark.manual
+@pytest.mark.tier(2)
+@pytest.mark.parametrize('context', [ViaREST, ViaUI])
+@test_requirements.multi_region
+@test_requirements.provision
+def test_provision_from_template_from_global_region(context):
+    """
+    Polarion:
+        assignee: izapolsk
+        caseimportance: medium
+        casecomponent: Provisioning
+        initialEstimate: 1/3h
+    """
+    pass

cfme/tests/cloud_infra_common/test_retirement.py

@@ -11,6 +11,8 @@
 from cfme.cloud.provider.ec2 import EC2Provider
 from cfme.infrastructure.provider import InfraProvider
 from cfme.markers.env_markers.provider import providers
+from cfme.utils.appliance import ViaREST
+from cfme.utils.appliance import ViaUI
 from cfme.utils.appliance.implementations.ui import navigate_to
 from cfme.utils.appliance.implementations.ui import navigator
 from cfme.utils.generators import random_vm_name
@@ -294,3 +296,28 @@ def test_resume_retired_instance(retire_vm, provider, remove_date):
 
     verify_retirement_date(retire_vm, expected_date=retire_date if retire_date else 'Never')
     assert retire_vm.is_retired is False
+
+
+@pytest.mark.manual
+@pytest.mark.tier(2)
+@pytest.mark.parametrize('context', [ViaREST, ViaUI])
+@test_requirements.multi_region
+@test_requirements.retirement
+def test_vm_retirement_from_global_region(context):
+    """
+    retire a vm via CA
+
+    Polarion:
+        assignee: izapolsk
+        casecomponent: Provisioning
+        initialEstimate: 1/3h
+        testSteps:
+            1. Have a VM created in the provider in the Remote region
+               subscribed to Global.
+            2. Retire the VM using the Global appliance.
+        expectedResults:
+            1.
+            2. VM transitions to Retired state in the Global and Remote region.
+
+    """
+    pass

cfme/tests/distributed/test_appliance_replication.py

@@ -370,16 +370,3 @@ def finalize():
         soft_assert(
             not vm_obj.mgmt.is_running,
             "vm running")
-
-
-@pytest.mark.manual
-@pytest.mark.tier(1)
-def test_replication_central_admin_adhoc_provision_template():
-    """
-    Polarion:
-        assignee: jhenner
-        caseimportance: medium
-        casecomponent: Replication
-        initialEstimate: 1/6h
-    """
-    pass

cfme/tests/infrastructure/test_vm_reconfigure.py

@@ -4,6 +4,8 @@
 from cfme import test_requirements
 from cfme.infrastructure.provider.rhevm import RHEVMProvider
 from cfme.infrastructure.provider.virtualcenter import VMwareProvider
+from cfme.utils.appliance import ViaREST
+from cfme.utils.appliance import ViaUI
 from cfme.utils.blockers import BZ
 from cfme.utils.generators import random_vm_name
 from cfme.utils.rest import assert_response
@@ -615,3 +617,28 @@ def test_vm_disk_reconfig_via_rest(appliance, full_vm):
             ), "Number of disks expected was {expected}, found {actual}".format(
                 expected=len(initial_disks), actual=full_vm.configuration.num_disks
             )
+
+
+@pytest.mark.manual
+@pytest.mark.tier(2)
+@pytest.mark.parametrize('context', [ViaREST, ViaUI])
+@test_requirements.multi_region
+@test_requirements.reconfigure
+def test_vm_reconfigure_from_global_region(context):
+    """
+    reconfigure a VM via CA
+
+    Polarion:
+        assignee: izapolsk
+        casecomponent: Infra
+        initialEstimate: 1/3h
+        testSteps:
+            1. Have a VM created in the provider in the Remote region which is
+               subscribed to Global.
+            2. Reconfigure the VM using the Global appliance.
+        expectedResults:
+            1.
+            2.
+            3. VM reconfigured, no errors.
+    """
+    pass

cfme/tests/intelligence/reports/test_reports_manual.py

@@ -3,6 +3,8 @@
 import pytest
 
 from cfme import test_requirements
+from cfme.utils.appliance import ViaREST
+from cfme.utils.appliance import ViaUI
 
 
 @pytest.mark.manual
@@ -413,3 +415,56 @@ def test_report_new_tags():
         1504155
     """
     pass
+
+
+@pytest.mark.manual
+@test_requirements.report
+@test_requirements.multi_region
+@pytest.mark.tier(2)
+@pytest.mark.parametrize('context', [ViaREST, ViaUI])
+def test_reports_create_in_global_region(context):
+    """
+    This test case tests report creation and rendering from global region
+    based on data from remote regions.
+
+    Polarion:
+        assignee: izapolsk
+        casecomponent: Reporting
+        caseimportance: medium
+        initialEstimate: 1/2h
+        testSteps:
+            1. Set up Multi-Region deployment
+            2. Add provider to remote appliance
+            3. Create and render report in global region
+        expectedResults:
+            1. Report should be created and rendered successfully.
+
+    """
+    pass
+
+
+@pytest.mark.manual
+@test_requirements.report
+@test_requirements.multi_region
+@pytest.mark.tier(2)
+@pytest.mark.parametrize('context', [ViaREST, ViaUI])
+def test_reports_use_remote_in_global_region(context):
+    """
+    This test case tests report creation in remote region and rendering it in global region
+    based on data from remote regions.
+
+    Polarion:
+        assignee: izapolsk
+        casecomponent: Reporting
+        caseimportance: medium
+        initialEstimate: 1/2h
+        testSteps:
+            1. Set up Multi-Region deployment
+            2. Add provider to remote appliance
+            3. Create report in remote region
+            4. Render report in global region
+        expectedResults:
+            1. Report should be created and rendered successfully.
+
+    """
+    pass

cfme/tests/services/test_service_catalogs.py

@@ -7,6 +7,8 @@
 from cfme.markers.env_markers.provider import ONE_PER_TYPE
 from cfme.services.service_catalogs import ServiceCatalogs
 from cfme.services.workloads import VmsInstances
+from cfme.utils.appliance import ViaREST
+from cfme.utils.appliance import ViaUI
 from cfme.utils.appliance.implementations.ui import navigate_to
 from cfme.utils.log import logger
 from cfme.utils.rest import assert_response
@@ -545,3 +547,35 @@ def test_generic_object_details_displayed_from_a_service_do_not_include_associat
         1576828
     """
     pass
+
+
+@pytest.mark.manual
+@pytest.mark.tier(2)
+@pytest.mark.parametrize('context', [ViaREST, ViaUI])
+@test_requirements.multi_region
+@test_requirements.service
+def test_service_provisioning_from_global_region(context):
+    """
+    Polarion:
+        assignee: izapolsk
+        caseimportance: medium
+        casecomponent: Services
+        initialEstimate: 1/4h
+    """
+    pass
+
+
+@pytest.mark.manual
+@pytest.mark.tier(2)
+@pytest.mark.parametrize('context', [ViaREST, ViaUI])
+@test_requirements.multi_region
+@test_requirements.service
+def test_service_retirement_from_global_region(context):
+    """
+    Polarion:
+        assignee: izapolsk
+        caseimportance: medium
+        casecomponent: Services
+        initialEstimate: 1/4h
+    """
+    pass

cfme/tests/test_replication.py

@@ -29,40 +29,6 @@ def test_replication_powertoggle():
     pass
 
 
-@pytest.mark.manual
-def test_replication_central_admin_vm_retirement():
-    """
-    retire a vm via CA
-
-    Polarion:
-        assignee: mnadeem
-        casecomponent: Replication
-        initialEstimate: 1/12h
-        testSteps:
-            1. Have a VM created in the provider in the Remote region
-               subscribed to Global.
-            2. Retire the VM using the Global appliance.
-        expectedResults:
-            1.
-            2. VM transitions to Retired state in the Global and Remote region.
-
-    """
-    pass
-
-
-@pytest.mark.manual
-@pytest.mark.tier(1)
-def test_replication_central_admin_service_provisioning():
-    """
-    Polarion:
-        assignee: mnadeem
-        caseimportance: medium
-        casecomponent: Replication
-        initialEstimate: 1/4h
-    """
-    pass
-
-
 @pytest.mark.manual
 @pytest.mark.tier(2)
 def test_replication_appliance_add_single_subscription():
@@ -147,7 +113,7 @@ def test_replication_central_admin_ansible_playbook_service_from_global():
     Playbook service is ordered from the master region catalog.
 
     Polarion:
-        assignee: mnadeem
+        assignee: izapolsk
         casecomponent: Replication
         subcomponent: Ansible
         caseimportance: medium
@@ -284,40 +250,6 @@ def test_replication_global_to_remote_new_vm_from_template():
     pass
 
 
-@pytest.mark.manual
-def test_replication_central_admin_vm_reconfigure():
-    """
-    reconfigure a VM via CA
-
-    Polarion:
-        assignee: mnadeem
-        casecomponent: Replication
-        initialEstimate: 1/12h
-        testSteps:
-            1. Have a VM created in the provider in the Remote region which is
-               subscribed to Global.
-            2. Reconfigure the VM using the Global appliance.
-        expectedResults:
-            1.
-            2.
-            3. VM reconfigured, no errors.
-    """
-    pass
-
-
-@pytest.mark.manual
-@pytest.mark.tier(1)
-def test_replication_central_admin_adhoc_provision_template():
-    """
-    Polarion:
-        assignee: mnadeem
-        caseimportance: medium
-        initialEstimate: 1/6h
-        casecomponent: Replication
-    """
-    pass
-
-
 @pytest.mark.manual
 @pytest.mark.tier(1)
 def test_replication_subscription_revalidation_pglogical():