Integrate pytest-rerunfailures (closes # 121 via #272)

Author: sseliverstov

allure-pytest/src/listener.py

@@ -58,16 +58,11 @@ def stop_fixture(self, parent_uuid, uuid, name, exc_type, exc_val, exc_tb):
                                               statusDetails=get_status_details(exc_type, exc_val, exc_tb))
 
     @pytest.hookimpl(hookwrapper=True)
-    def pytest_runtest_protocol(self, item):
+    def pytest_runtest_setup(self, item):
         uuid = self._cache.set(item.nodeid)
         test_result = TestResult(name=item.name, uuid=uuid)
         self.allure_logger.schedule_test(uuid, test_result)
-        yield
-        uuid = self._cache.pop(item.nodeid)
-        self.allure_logger.close_test(uuid)
 
-    @pytest.hookimpl(hookwrapper=True)
-    def pytest_runtest_setup(self, item):
         yield
         uuid = self._cache.get(item.nodeid)
         test_result = self.allure_logger.get_test(uuid)
@@ -186,6 +181,9 @@ def pytest_runtest_makereport(self, item, call):
                 test_result.status = status
                 test_result.statusDetails = status_details
 
+            uuid = self._cache.pop(item.nodeid)
+            self.allure_logger.close_test(uuid)
+
     @allure_commons.hookimpl
     def attach_data(self, body, name, attachment_type, extension):
         self.allure_logger.attach_data(uuid4(), body, name=name, attachment_type=attachment_type, extension=extension)

allure-pytest/test/integrations/__init__.py

@@ -0,0 +1,3 @@
+
+def pytest_ignore_collect(path, config):
+    return config.pluginmanager.hasplugin('xdist')

allure-pytest/test/integrations/rerunfailures/__init__.py

@@ -0,0 +1,26 @@
+import threading
+import pytest
+
+lord_of_the_flies = threading.local()
+
+
+@pytest.mark.flaky(reruns=5)
+def test_rerunfialures_failed_to_passed():
+    """
+    >>> allure_report = getfixture('allure_report')
+
+    >>> assert_that(allure_report,
+    ...             has_test_case('test_rerunfialures_failed_to_passed',
+    ...                           with_status('failed')
+    ...             )
+    ... )
+
+    >>> assert_that(allure_report,
+    ...             has_test_case('test_rerunfialures_failed_to_passed',
+    ...                           with_status('passed')
+    ...             )
+    ... )
+    """
+    saved = getattr(lord_of_the_flies, 'good', False)
+    lord_of_the_flies.good = saved or 'maybe next time'
+    assert saved

allure-pytest/test/integrations/rerunfailures/conftest.py

@@ -14,6 +14,7 @@ setenv = ALLURE_INDENT_OUTPUT=yep
 
 deps =
     pyhamcrest
+    pytest-rerunfailures
     {distshare}/allure-python-commons-2*.zip
     {distshare}/allure-python-commons-test-2*.zip