fix allure-framework#586

Savior-Wu · web-flow · commit a009faac03c7 · 2021-08-06T09:41:41.000+08:00
Fix issue allure-framework#586: The dynamic suite and default suite are kept both in report Current: a case only has one parentSuite/suite/subSuite, and the dynamic has higher priority than the default
diff --git a/allure-pytest/src/listener.py b/allure-pytest/src/listener.py
@@ -114,7 +114,12 @@ def pytest_runtest_teardown(self, item):
         test_result = self.allure_logger.get_test(uuid)
         test_result.labels.extend([Label(name=name, value=value) for name, value in allure_labels(item)])
         test_result.labels.extend([Label(name=LabelType.TAG, value=value) for value in pytest_markers(item)])
-        test_result.labels.extend([Label(name=name, value=value) for name, value in allure_suite_labels(item)])
+        default_suite = [Label(name=name, value=value) for name, value in allure_suite_labels(item)]
+        for default in default_suite[::-1]:
+            for label in test_result.labels:
+                if label.name == default.name:
+                    default_suite.remove(default)
+        test_result.labels.extend(default_suite)
         test_result.labels.append(Label(name=LabelType.HOST, value=self._host))
         test_result.labels.append(Label(name=LabelType.THREAD, value=self._thread))
         test_result.labels.append(Label(name=LabelType.FRAMEWORK, value='pytest'))
