unregister pytest plugin (fixes #202 via #203)

Author: sseliverstov

allure-pytest/src/plugin.py

@@ -88,18 +88,30 @@ def link_pattern(string):
                                          str.format().""")
 
 
+def cleanup_factory(plugin):
+    def clean_up():
+        name = allure_commons.plugin_manager.get_name(plugin)
+        allure_commons.plugin_manager.unregister(name=name)
+    return clean_up
+
+
 def pytest_configure(config):
     report_dir = config.option.allure_report_dir
+
     test_helper = AllureTestHelper(config)
+    # TODO: Why helper is present anyway?
     allure_commons.plugin_manager.register(test_helper)
+    config.add_cleanup(cleanup_factory(test_helper))
 
     if report_dir:
         test_listener = AllureListener(config)
         config.pluginmanager.register(test_listener)
         allure_commons.plugin_manager.register(test_listener)
+        config.add_cleanup(cleanup_factory(test_listener))
 
         file_logger = AllureFileLogger(report_dir)
         allure_commons.plugin_manager.register(file_logger)
+        config.add_cleanup(cleanup_factory(file_logger))
 
 
 def pytest_collection_modifyitems(items, config):

allure-pytest/test/plugin/register_unregister/master_test.py

@@ -0,0 +1,19 @@
+import os
+import pytest
+
+
+def test_register_unregister(tmpdir):
+    """
+    >>> allure_report = getfixture('allure_report')
+    >>> assert_that(allure_report,
+    ...             has_test_case('test_register_unregister',
+    ...                           with_status('passed')
+    ...             )
+    ... )
+    """
+    current_dir = os.path.dirname(os.path.realpath(__file__))
+    target = os.path.join(current_dir, 'sample.py')
+    allure_option = '--alluredir={dir}'.format(dir=tmpdir.mkdir('trash').strpath)
+    assert pytest.main([allure_option, target]) == 0
+    assert pytest.main([allure_option, target]) == 0
+

allure-pytest/test/plugin/register_unregister/sample.py

@@ -0,0 +1,5 @@
+import allure
+
+
+def test_sample():
+    allure.attach('Peace!!!')