test: reset logger after tests (#936)

Co-authored-by: alambare-csgroup <aubin.lambare@csgroup.eu>

tests/test_cli.py

@@ -16,6 +16,7 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
+import logging
 import os
 import random
 import re
@@ -40,7 +41,6 @@
     download,
     eodag,
     search_crunch,
-    setup_logging,
 )
 from tests.units import test_core
 from tests.utils import mock, no_blanks, write_eodag_conf_with_fake_credentials
@@ -80,11 +80,13 @@ def setUp(self):
 
     def tearDown(self):
         super(TestEodagCli, self).tearDown()
-        # Default logging: no logging but still displays progress bars
-        setup_logging(1)
         # stop Mock and remove tmp config dir
         self.expanduser_mock.stop()
         self.tmp_home_dir.cleanup()
+        # reset logging
+        logger = logging.getLogger("eodag")
+        logger.handlers = []
+        logger.level = 0
 
     def test_eodag_without_args(self):
         """Calling eodag without arguments should print help message"""

tests/units/test_apis_plugins.py

@@ -692,6 +692,10 @@ def test_plugins_apis_cds_logging(self):
         client = self.api_plugin._get_cds_client(**auth_dict)
         self.assertEqual(client.logger.level, logging.DEBUG)
 
+        logger = logging.getLogger("eodag")
+        logger.handlers = []
+        logger.level = 0
+
     def test_plugins_apis_cds_query_dates_missing(self):
         """CdsApi.query must use default dates if missing"""
         # given start & stop

tests/units/test_core.py

@@ -19,6 +19,7 @@
 import copy
 import glob
 import json
+import logging
 import os
 import shutil
 import unittest
@@ -76,6 +77,10 @@ def tearDownClass(cls):
         # stop Mock and remove tmp config dir
         cls.expanduser_mock.stop()
         cls.tmp_home_dir.cleanup()
+        # reset logging
+        logger = logging.getLogger("eodag")
+        logger.handlers = []
+        logger.level = 0
 
 
 class TestCore(TestCoreBase):
@@ -1512,6 +1517,9 @@ def test__search_by_id(
 
         found = self.dag._search_by_id(uid="foo", productType="bar", provider="baz")
 
+        from eodag.utils.logging import get_logging_verbose
+
+        _ = get_logging_verbose()
         # get_search_plugins
         mock_get_search_plugins.assert_called_once_with(
             self.dag._plugins_manager, product_type="bar", provider="baz"

tests/units/test_utils.py

@@ -17,6 +17,7 @@
 # limitations under the License.
 
 import copy
+import logging
 import os
 import sys
 import unittest
@@ -41,6 +42,17 @@
 
 
 class TestUtils(unittest.TestCase):
+    def setUp(self) -> None:
+        super(TestUtils, self).setUp()
+        setup_logging(verbose=1)
+
+    def tearDown(self) -> None:
+        super(TestUtils, self).tearDown()
+        # reset logging
+        logger = logging.getLogger("eodag")
+        logger.handlers = []
+        logger.level = 0
+
     def test_utils_get_timestamp(self):
         """get_timestamp must return a UNIX timestamp"""
         # Date to timestamp to date, this assumes the date is in UTC