Skip to content
This repository has been archived by the owner on Apr 7, 2022. It is now read-only.

Commit

Permalink
Merge pull request #9976 from tpapaioa/test_paginator_config_pages_db
Browse files Browse the repository at this point in the history
[1LP][RFR] Uncollect Server > Database tests in test_paginator_config_pages
  • Loading branch information
mshriver authored Mar 13, 2020
2 parents d8b47e2 + 11db730 commit 1ee3ee3
Showing 1 changed file with 14 additions and 7 deletions.
21 changes: 14 additions & 7 deletions cfme/tests/configure/test_paginator.py
Original file line number Diff line number Diff line change
Expand Up @@ -7,8 +7,16 @@
from cfme.configure.configuration.region_settings import RedHatUpdates
from cfme.utils.appliance.implementations.ui import navigate_to

general_db = [
('servers', None, 'DatabaseSummary', False),
('servers', None, 'DatabaseTables', True),
('servers', None, 'DatabaseIndexes', True),
('servers', None, 'DatabaseSettings', True),
('servers', None, 'DatabaseClientConnections', True),
('servers', None, 'DatabaseUtilization', False),
]

general_list_pages = [
general_non_db = [
('servers', None, 'Details', False),
('servers', None, 'Authentication', False),
('servers', None, 'Workers', False),
Expand All @@ -22,12 +30,6 @@
('servers', None, 'Utilization', False),
('servers', None, 'Timelines', False),
('servers', None, 'ServerDiagnosticsCollectLogs', False),
('servers', None, 'DatabaseSummary', False),
('servers', None, 'DatabaseTables', True),
('servers', None, 'DatabaseIndexes', True),
('servers', None, 'DatabaseSettings', True),
('servers', None, 'DatabaseClientConnections', True),
('servers', None, 'DatabaseUtilization', False),
('regions', None, 'Details', False),
('regions', None, 'ImportTags', False),
('regions', None, 'Import', False),
Expand Down Expand Up @@ -58,6 +60,8 @@
('tenants', None, 'All', True),
]

general_list_pages = general_db + general_non_db

details_pages = [
('users', None, 'Details', False),
('groups', None, 'Details', False),
Expand Down Expand Up @@ -97,6 +101,9 @@ def schedule(appliance):
@pytest.mark.parametrize('place_info', general_list_pages,
ids=['{}_{}'.format(set_type[0], set_type[2].lower())
for set_type in general_list_pages])
@pytest.mark.uncollectif(lambda appliance, place_info:
appliance.version >= '5.11' and place_info in general_db,
reason="This page is disabled in 5.11 and later.")
def test_paginator_config_pages(appliance, place_info):
"""
Check paginator is visible for config pages
Expand Down

0 comments on commit 1ee3ee3

Please sign in to comment.