Address PR comments

Signed-off-by: Theofilos Manitaras <teojgor@gmail.com>

Author: teojgo

docs/manpage.rst

@@ -1060,7 +1060,7 @@ The way the tests are generated and how they interact with the test filtering op
       To achieve the previous behaviour, you should use ``--distribute=idle*``.
 
    .. versionchanged:: 4.9
-      ``--distribute=NODESTATE`` now allows you to specify alternative valid states using the ``|`` character.
+      ``--distribute=NODESTATE`` now allows you to specify multiple valid states using the ``|`` character.
 
 .. option:: -P, --parameterize=[TEST.]VAR=VAL0,VAL1,...
 

reframe/core/schedulers/__init__.py

@@ -162,7 +162,7 @@ def filter_nodes_by_state(nodelist, state):
         If ``avail``, only the available nodes will be returned.
         All other values are interpreted as a state string.
         The pipe character ``|`` can be used as to specify multiple
-        alternative node states and has the highest priority.
+        alternative node states.
         State match is exclusive unless the ``*`` is added at the end of the
         state string.
         When defining multiple states using ``|``, ``*`` has to be added at

unittests/test_schedulers.py

@@ -1247,18 +1247,30 @@ def test_flex_alloc_not_enough_nodes_constraint_expr(make_flexible_job):
         prepare_job(job)
 
 
-def test_flex_alloc_allocated_or_idle_partition(make_flexible_job):
+def test_flex_alloc_alloc_state_OR(make_flexible_job):
     job = make_flexible_job('allocated|idle')
     job.options = ['--partition=p3']
     prepare_job(job)
     assert job.num_tasks == 12
 
-
-def test_flex_alloc_maint_star_or_allocated(make_flexible_job):
     job = make_flexible_job('maint*|idle')
     prepare_job(job)
     assert job.num_tasks == 16
 
+    job = make_flexible_job('maint|avail')
+    job.options = ['--partition=p1']
+    prepare_job(job)
+    assert job.num_tasks == 12
+
+    job = make_flexible_job('all|idle')
+    prepare_job(job)
+    assert job.num_tasks == 16
+
+    job = make_flexible_job('allocated|idle|maint')
+    job.options = ['--partition=p1']
+    prepare_job(job)
+    assert job.num_tasks == 12
+
 
 @pytest.fixture
 def slurm_node_allocated():