Fix log-level option

Author: ltm

icloudpd/logger.py

@@ -2,7 +2,7 @@
 
 import sys
 import logging
-from logging import DEBUG, INFO
+from logging import INFO
 
 
 class IPDLogger(logging.Logger):
@@ -32,11 +32,10 @@ def tqdm_write(self, message, loglevel=INFO):
             self.tqdm.write(message)
 
 
-def setup_logger(loglevel=DEBUG):
+def setup_logger():
     """Set up logger and add stdout handler"""
     logging.setLoggerClass(IPDLogger)
     logger = logging.getLogger("icloudpd")
-    logger.setLevel(loglevel)
     has_stdout_handler = False
     for handler in logger.handlers:
         if handler.name == "stdoutLogger":

tests/test_cli.py

@@ -3,13 +3,18 @@
 import os
 import shutil
 from vcr import VCR
+import pytest
 from click.testing import CliRunner
 from icloudpd.base import main
 
 vcr = VCR(decode_compressed_response=True)
 
 
 class CliTestCase(TestCase):
+    @pytest.fixture(autouse=True)
+    def inject_fixtures(self, caplog):
+        self._caplog = caplog
+
     def test_cli(self):
         runner = CliRunner()
         result = runner.invoke(main, ["--help"])
@@ -37,8 +42,11 @@ def test_log_levels(self):
                     "tests/fixtures/Photos",
                 ],
             )
+            self.assertNotIn("DEBUG", self._caplog.text)
+            self.assertIn("INFO", self._caplog.text)
             assert result.exit_code == 0
         with vcr.use_cassette("tests/vcr_cassettes/listing_photos.yml"):
+            self._caplog.clear()
             result = runner.invoke(
                 main,
                 [
@@ -54,6 +62,8 @@ def test_log_levels(self):
                     "tests/fixtures/Photos",
                 ],
             )
+            self.assertNotIn("DEBUG", self._caplog.text)
+            self.assertNotIn("INFO", self._caplog.text)
             assert result.exit_code == 0
 
     def test_tqdm(self):