From 219ae90ead1af5c5e99761098693c673a418b1d7 Mon Sep 17 00:00:00 2001 From: shwsingh Date: Tue, 9 Apr 2024 19:53:32 +0530 Subject: [PATCH] Mark UI tests as end-to-end for Rocket Team Components --- tests/foreman/cli/test_leapp_client.py | 4 +++- tests/foreman/ui/test_computeprofiles.py | 1 + tests/foreman/ui/test_computeresource.py | 5 +++-- tests/foreman/ui/test_computeresource_azurerm.py | 7 ++----- tests/foreman/ui/test_computeresource_ec2.py | 1 + tests/foreman/ui/test_computeresource_gce.py | 9 ++++----- tests/foreman/ui/test_computeresource_libvirt.py | 6 ++++-- tests/foreman/ui/test_config_group.py | 1 + tests/foreman/ui/test_discoveryrule.py | 2 ++ tests/foreman/ui/test_operatingsystem.py | 1 + tests/foreman/ui/test_puppetclass.py | 1 + tests/foreman/ui/test_puppetenvironment.py | 1 + tests/foreman/ui/test_smartclassparameter.py | 1 + tests/foreman/ui/test_subnet.py | 1 + 14 files changed, 26 insertions(+), 15 deletions(-) diff --git a/tests/foreman/cli/test_leapp_client.py b/tests/foreman/cli/test_leapp_client.py index e4402d3f6a9..5c908697202 100644 --- a/tests/foreman/cli/test_leapp_client.py +++ b/tests/foreman/cli/test_leapp_client.py @@ -6,7 +6,7 @@ :Team: Rocket -:CaseImportance: High +:CaseImportance: Critical :CaseAutomation: Automated @@ -219,6 +219,7 @@ def precondition_check_upgrade_and_install_leapp_tool(custom_leapp_host): ) +@pytest.mark.e2e @pytest.mark.skip_if_open('SAT-24023') @pytest.mark.parametrize( 'upgrade_path', @@ -300,6 +301,7 @@ def test_leapp_upgrade_rhel( assert new_ver == upgrade_path['target_version'] +@pytest.mark.e2e @pytest.mark.skip_if_open('SAT-24023') @pytest.mark.parametrize( 'upgrade_path', diff --git a/tests/foreman/ui/test_computeprofiles.py b/tests/foreman/ui/test_computeprofiles.py index 0f0cb99c1ea..551f003316f 100644 --- a/tests/foreman/ui/test_computeprofiles.py +++ b/tests/foreman/ui/test_computeprofiles.py @@ -16,6 +16,7 @@ import pytest +@pytest.mark.e2e @pytest.mark.tier2 @pytest.mark.upgrade def test_positive_end_to_end(session, module_location, module_org, module_target_sat): diff --git a/tests/foreman/ui/test_computeresource.py b/tests/foreman/ui/test_computeresource.py index b39cc00e3a6..e9372012d69 100644 --- a/tests/foreman/ui/test_computeresource.py +++ b/tests/foreman/ui/test_computeresource.py @@ -48,7 +48,7 @@ def rhev_data(): @pytest.mark.tier2 -def test_positive_end_to_end(session, rhev_data, module_org, module_location, module_target_sat): +def test_positive_end_to_end(session, rhev_data, module_target_sat): """Perform end to end testing for compute resource RHEV. :id: 3c079675-e5d3-490e-9b7e-1c2950f9965d @@ -361,8 +361,9 @@ def test_positive_update_organization(session, rhev_data, module_location, modul assert new_organization.name in resource_values['organizations']['resources']['assigned'] +@pytest.mark.e2e @pytest.mark.tier2 -def test_positive_image_end_to_end(session, rhev_data, module_location, target_sat): +def test_positive_image_end_to_end(session, rhev_data, target_sat): """Perform end to end testing for compute resource RHV component image. :id: 62a5c52f-dd15-45e7-8200-c64bb335474f diff --git a/tests/foreman/ui/test_computeresource_azurerm.py b/tests/foreman/ui/test_computeresource_azurerm.py index 5d09ad88a69..7ee5c3b80c9 100644 --- a/tests/foreman/ui/test_computeresource_azurerm.py +++ b/tests/foreman/ui/test_computeresource_azurerm.py @@ -74,14 +74,13 @@ def module_azure_hg( ).create() +@pytest.mark.e2e @pytest.mark.tier4 @pytest.mark.parametrize('sat_azure', ['sat'], indirect=True) def test_positive_end_to_end_azurerm_ft_host_provision( - session, sat_azure, azurermclient, module_azurerm_custom_finishimg, - module_azurerm_cr, sat_azure_domain, sat_azure_org, sat_azure_loc, @@ -151,19 +150,17 @@ def test_positive_end_to_end_azurerm_ft_host_provision( raise error +@pytest.mark.e2e @pytest.mark.tier3 @pytest.mark.upgrade @pytest.mark.parametrize( 'sat_azure', ['sat', 'puppet_sat'], indirect=True, ids=['satellite', 'puppet_enabled'] ) def test_positive_azurerm_host_provision_ud( - session, sat_azure, azurermclient, module_azurerm_cloudimg, - module_azurerm_cr, sat_azure_domain, - sat_azure_default_os, sat_azure_org, sat_azure_loc, module_azure_hg, diff --git a/tests/foreman/ui/test_computeresource_ec2.py b/tests/foreman/ui/test_computeresource_ec2.py index 1dae4eb7492..cf6f1f29237 100644 --- a/tests/foreman/ui/test_computeresource_ec2.py +++ b/tests/foreman/ui/test_computeresource_ec2.py @@ -40,6 +40,7 @@ def module_ec2_settings(): ) +@pytest.mark.e2e @pytest.mark.tier2 @pytest.mark.skip_if_not_set('http_proxy') @pytest.mark.skip_if_open("BZ:2032530") diff --git a/tests/foreman/ui/test_computeresource_gce.py b/tests/foreman/ui/test_computeresource_gce.py index 4c48e902809..9b8bb585bad 100644 --- a/tests/foreman/ui/test_computeresource_gce.py +++ b/tests/foreman/ui/test_computeresource_gce.py @@ -29,11 +29,12 @@ ) +@pytest.mark.e2e @pytest.mark.tier2 @pytest.mark.upgrade @pytest.mark.skip_if_not_set('http_proxy', 'gce') def test_positive_default_end_to_end_with_custom_profile( - session, sat_gce_org, sat_gce_loc, gce_cert, sat_gce + sat_gce_org, sat_gce_loc, gce_cert, sat_gce ): """Create GCE compute resource with default properties and apply it's basic functionality. @@ -140,12 +141,12 @@ def test_positive_default_end_to_end_with_custom_profile( assert not session.computeresource.search(new_cr_name) +@pytest.mark.e2e @pytest.mark.tier4 @pytest.mark.run_in_one_thread @pytest.mark.skip_if_not_set('gce') @pytest.mark.parametrize('sat_gce', ['sat', 'puppet_sat'], indirect=True) def test_positive_gce_provision_end_to_end( - session, request, sat_gce, sat_gce_org, @@ -154,7 +155,6 @@ def test_positive_gce_provision_end_to_end( gce_domain, gce_hostgroup_resource_image, googleclient, - gce_setting_update, ): """Provision Host on GCE compute resource @@ -225,13 +225,13 @@ def _finalize(): assert gceapi_vm.is_stopping or gceapi_vm.is_stopped +@pytest.mark.e2e @pytest.mark.tier4 @pytest.mark.upgrade @pytest.mark.run_in_one_thread @pytest.mark.skip_if_not_set('gce') @pytest.mark.parametrize('sat_gce', ['sat', 'puppet_sat'], indirect=True) def test_positive_gce_cloudinit_provision_end_to_end( - session, request, sat_gce, sat_gce_org, @@ -240,7 +240,6 @@ def test_positive_gce_cloudinit_provision_end_to_end( gce_domain, gce_hostgroup_resource_image, googleclient, - gce_setting_update, ): """Provision Host on GCE compute resource diff --git a/tests/foreman/ui/test_computeresource_libvirt.py b/tests/foreman/ui/test_computeresource_libvirt.py index 8328f1a4cf1..22e5ffdf0f4 100644 --- a/tests/foreman/ui/test_computeresource_libvirt.py +++ b/tests/foreman/ui/test_computeresource_libvirt.py @@ -118,13 +118,13 @@ def test_positive_end_to_end(session, module_target_sat, module_org, module_loca assert not session.computeresource.search(new_cr_name) +@pytest.mark.e2e @pytest.mark.on_premises_provisioning @pytest.mark.tier4 @pytest.mark.rhel_ver_match('[^6]') @pytest.mark.parametrize('setting_update', ['destroy_vm_on_host_delete=True'], indirect=True) def test_positive_provision_end_to_end( request, - session, setting_update, module_sca_manifest_org, module_location, @@ -155,7 +155,9 @@ def test_positive_provision_end_to_end( location=[module_location], organization=[module_sca_manifest_org], ).create() - with session: + with sat.ui_session() as session: + session.organization.select(module_sca_manifest_org.name) + session.location.select(module_location.name) session.host.create( { 'host.name': hostname, diff --git a/tests/foreman/ui/test_config_group.py b/tests/foreman/ui/test_config_group.py index 34e6b32b9c9..b27788ccd86 100644 --- a/tests/foreman/ui/test_config_group.py +++ b/tests/foreman/ui/test_config_group.py @@ -21,6 +21,7 @@ def module_puppet_class(session_puppet_enabled_sat): return session_puppet_enabled_sat.api.PuppetClass().create() +@pytest.mark.e2e @pytest.mark.tier2 @pytest.mark.upgrade def test_positive_end_to_end(session_puppet_enabled_sat, module_puppet_class): diff --git a/tests/foreman/ui/test_discoveryrule.py b/tests/foreman/ui/test_discoveryrule.py index 85958d57e53..9720470a0d2 100644 --- a/tests/foreman/ui/test_discoveryrule.py +++ b/tests/foreman/ui/test_discoveryrule.py @@ -63,6 +63,7 @@ def gen_int32(min_value=1): return gen_integer(min_value=min_value, max_value=max_value) +@pytest.mark.e2e @pytest.mark.tier2 def test_positive_crud_with_non_admin_user( module_location, manager_user, module_org, module_target_sat @@ -232,6 +233,7 @@ def test_positive_list_host_based_on_rule_search_query( assert values['properties']['properties_table']['IP Address'] == ip_address +@pytest.mark.e2e @pytest.mark.tier3 @pytest.mark.upgrade def test_positive_end_to_end(session, module_org, module_location, module_target_sat): diff --git a/tests/foreman/ui/test_operatingsystem.py b/tests/foreman/ui/test_operatingsystem.py index 71a030772c6..c5ed4a31e2e 100644 --- a/tests/foreman/ui/test_operatingsystem.py +++ b/tests/foreman/ui/test_operatingsystem.py @@ -18,6 +18,7 @@ from robottelo.utils.datafactory import gen_string +@pytest.mark.e2e @pytest.mark.tier2 def test_positive_end_to_end(session, module_org, module_location, target_sat): """Create all possible entities that required for operating system and then diff --git a/tests/foreman/ui/test_puppetclass.py b/tests/foreman/ui/test_puppetclass.py index 42ae2b359ab..0b3152330cf 100644 --- a/tests/foreman/ui/test_puppetclass.py +++ b/tests/foreman/ui/test_puppetclass.py @@ -16,6 +16,7 @@ import pytest +@pytest.mark.e2e @pytest.mark.tier2 @pytest.mark.upgrade def test_positive_end_to_end(session_puppet_enabled_sat, module_puppet_org, module_puppet_loc): diff --git a/tests/foreman/ui/test_puppetenvironment.py b/tests/foreman/ui/test_puppetenvironment.py index 6add5508549..1e7787f5505 100644 --- a/tests/foreman/ui/test_puppetenvironment.py +++ b/tests/foreman/ui/test_puppetenvironment.py @@ -18,6 +18,7 @@ from robottelo.utils.datafactory import gen_string +@pytest.mark.e2e @pytest.mark.upgrade @pytest.mark.tier2 def test_positive_end_to_end(session_puppet_enabled_sat, module_puppet_org, module_puppet_loc): diff --git a/tests/foreman/ui/test_smartclassparameter.py b/tests/foreman/ui/test_smartclassparameter.py index 7cfe14cd036..68fe2a50cfd 100644 --- a/tests/foreman/ui/test_smartclassparameter.py +++ b/tests/foreman/ui/test_smartclassparameter.py @@ -66,6 +66,7 @@ def module_domain(session_puppet_enabled_sat, module_host): return session_puppet_enabled_sat.api.Domain(id=module_host.domain.id).read() +@pytest.mark.e2e @pytest.mark.tier2 def test_positive_end_to_end(session_puppet_enabled_sat, module_puppet_classes, sc_params_list): """Perform end to end testing for smart class parameter component diff --git a/tests/foreman/ui/test_subnet.py b/tests/foreman/ui/test_subnet.py index c29bfcb850a..9eab769c82d 100644 --- a/tests/foreman/ui/test_subnet.py +++ b/tests/foreman/ui/test_subnet.py @@ -27,6 +27,7 @@ def module_dom(module_target_sat, module_org, module_location): d.delete() +@pytest.mark.e2e @pytest.mark.tier2 @pytest.mark.upgrade def test_positive_end_to_end(session, module_target_sat, module_dom):