Fix handling of parametrized tests

- adapt test to catch the bug
- see #38

Author: mrbean-bremen

pytest_order/sorter.py

@@ -192,13 +192,18 @@ def items_from_label(
         with suppress(KeyError):
             node_ids = self.node_id_last[last_comp]
             for node_id in node_ids:
-                if (node_id.endswith(label) or node_id.endswith("]") and
-                        node_id.rpartition("[")[0].endswith(label)):
+                if node_id.endswith(label):
                     id_start = node_id[:-label_len]
-                    if is_cls_mark and id_start.count("::") == 2:
-                        continue
-                    if item_id.startswith(id_start):
-                        items.append(self.node_ids[node_id])
+                elif (node_id.endswith("]") and
+                      node_id.rpartition("[")[0].endswith(label)):
+                    id_start = node_id.rpartition("[")[0][:-label_len]
+                else:
+                    continue
+                if is_cls_mark and id_start.count("::") == 2:
+                    continue
+                if item_id.startswith(id_start):
+                    items.append(self.node_ids[node_id])
+
         return items
 
     def items_from_class_label(self, label: str, item: Item) -> List[Item]:

tests/test_relative_ordering.py

@@ -501,7 +501,7 @@ def test_dependency_on_parametrized_test(item_names_for):
         def test_1():
             pass
 
-        @pytest.mark.parametrize("arg", [1, 2, 3, 4])
+        @pytest.mark.parametrize("arg", ["aaaaa", "bbbbb", "ccccc", "ddddd"])
         def test_2(arg):
             pass
 
@@ -511,5 +511,6 @@ def test_3():
         """
     )
     assert item_names_for(test_content) == [
-        "test_3", "test_2[1]", "test_2[2]", "test_2[3]", "test_2[4]", "test_1"
+        "test_3", "test_2[aaaaa]", "test_2[bbbbb]",
+        "test_2[ccccc]", "test_2[ddddd]", "test_1"
     ]