From 78ff546fa4dc2744e45eb91bf5b353a6a446b10d Mon Sep 17 00:00:00 2001
From: Pierre-Yves Chibon <pingou@pingoured.fr>
Date: Feb 13 2019 20:53:21 +0000
Subject: Leave the application's log handler alone


Not having handlers used to be necessary when we were sending error
emails by default but this is no longer the case and this debugging
information is sometime most useful when trying to debug tests.

Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>

---

diff --git a/tests/__init__.py b/tests/__init__.py
index 6a61cf2..565790c 100644
--- a/tests/__init__.py
+++ b/tests/__init__.py
@@ -403,8 +403,6 @@ class SimplePagureTest(unittest.TestCase):
         imp.reload(pagure.lib.tasks_services)
 
         self._app = pagure.flask_app.create_app({'DB_URL': self.dbpath})
-        # Remove the log handlers for the tests
-        self._app.logger.handlers = []
 
         self.app = self._app.test_client()
         self.gr_patcher = mock.patch('pagure.lib.tasks.get_result')