diff --git a/tests/conftest.py b/tests/conftest.py index 42e37ca1df..7d3f651141 100644 --- a/tests/conftest.py +++ b/tests/conftest.py @@ -117,6 +117,10 @@ def node(host, request): 'dev': 99 } + sanitized_group_names = group_names + if 'all' in sanitized_group_names: + sanitized_group_names.remove('all') + # capture the initial/default state test_is_applicable = False for marker in request.node.iter_markers(): @@ -133,7 +137,7 @@ def node(host, request): if request.node.get_closest_marker('rbdmirror_secondary') and not ceph_rbd_mirror_remote_user: # noqa E501 pytest.skip('Not a valid test for a non-secondary rbd-mirror node') - if request.node.get_closest_marker('ceph_crash') and group_names in [['nfss'], ['iscsigws'], ['clients'], ['monitoring']]: + if request.node.get_closest_marker('ceph_crash') and sanitized_group_names in [['nfss'], ['iscsigws'], ['clients'], ['monitoring']]: pytest.skip('Not a valid test for nfs, client or iscsigw nodes') if request.node.get_closest_marker("no_docker") and docker: @@ -148,7 +152,7 @@ def node(host, request): pytest.skip( "Not a valid test with dashboard disabled") - if request.node.get_closest_marker("dashboard") and group_names == ['clients']: + if request.node.get_closest_marker("dashboard") and sanitized_group_names == ['clients']: pytest.skip('Not a valid test for client node') data = dict(