Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[6.13.z] Mark UI tests as end-to-end for Rocket Team Components #15337

Merged
merged 1 commit into from
Jun 6, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion tests/foreman/cli/test_leapp_client.py
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@

:Team: Rocket

:CaseImportance: High
:CaseImportance: Critical

:CaseAutomation: Automated

Expand Down Expand Up @@ -202,6 +202,7 @@ def precondition_check_upgrade_and_install_leapp_tool(custom_leapp_host):
custom_leapp_host.power_control(state='reboot', ensure=True)


@pytest.mark.e2e
@pytest.mark.parametrize(
'upgrade_path',
[
Expand Down
1 change: 1 addition & 0 deletions tests/foreman/ui/test_computeprofiles.py
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@
import pytest


@pytest.mark.e2e
@pytest.mark.tier2
@pytest.mark.upgrade
def test_positive_end_to_end(session, module_location, module_org):
Expand Down
5 changes: 3 additions & 2 deletions tests/foreman/ui/test_computeresource.py
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ def rhev_data():


@pytest.mark.tier2
def test_positive_end_to_end(session, rhev_data, module_org, module_location):
def test_positive_end_to_end(session, rhev_data, module_target_sat):
"""Perform end to end testing for compute resource RHEV.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@shweta83 the test is using entities instead of module_target_sat.api. Also, it'd be better if we use module_target_sat.ui_session() here. Same suggestion for other tests in the module. It's probably the same in master branch, so I'll leave it to you to decide if it should be done in separate PR.


:id: 3c079675-e5d3-490e-9b7e-1c2950f9965d
Expand Down Expand Up @@ -356,8 +356,9 @@ def test_positive_update_organization(session, rhev_data, module_location):
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
Expand Down
4 changes: 2 additions & 2 deletions tests/foreman/ui/test_computeresource_azurerm.py
Original file line number Diff line number Diff line change
Expand Up @@ -74,10 +74,10 @@ 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,
Expand Down Expand Up @@ -151,13 +151,13 @@ 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,
Expand Down
1 change: 1 addition & 0 deletions tests/foreman/ui/test_computeresource_ec2.py
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,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")
Expand Down
7 changes: 4 additions & 3 deletions tests/foreman/ui/test_computeresource_gce.py
Original file line number Diff line number Diff line change
Expand Up @@ -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.

Expand Down Expand Up @@ -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,
Expand Down Expand Up @@ -225,13 +226,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,
Expand Down
6 changes: 4 additions & 2 deletions tests/foreman/ui/test_computeresource_libvirt.py
Original file line number Diff line number Diff line change
Expand Up @@ -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,
Expand Down Expand Up @@ -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,
Expand Down
1 change: 1 addition & 0 deletions tests/foreman/ui/test_config_group.py
Original file line number Diff line number Diff line change
Expand Up @@ -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):
Expand Down
2 changes: 2 additions & 0 deletions tests/foreman/ui/test_discoveryrule.py
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down Expand Up @@ -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):
Expand Down
1 change: 1 addition & 0 deletions tests/foreman/ui/test_operatingsystem.py
Original file line number Diff line number Diff line change
Expand Up @@ -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):
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
def test_positive_end_to_end(session, module_org, module_location, target_sat):
def test_positive_end_to_end(session, module_org, module_location, module_target_sat):

"""Create all possible entities that required for operating system and then
Expand Down
1 change: 1 addition & 0 deletions tests/foreman/ui/test_puppetclass.py
Original file line number Diff line number Diff line change
Expand Up @@ -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):
Expand Down
1 change: 1 addition & 0 deletions tests/foreman/ui/test_puppetenvironment.py
Original file line number Diff line number Diff line change
Expand Up @@ -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):
Expand Down
1 change: 1 addition & 0 deletions tests/foreman/ui/test_smartclassparameter.py
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down
1 change: 1 addition & 0 deletions tests/foreman/ui/test_subnet.py
Original file line number Diff line number Diff line change
Expand Up @@ -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):
Expand Down
Loading