Simplify logic to determine logging level

pypa · Sep 6, 2023 · 895cf0a · 895cf0a
1 parent baaa574
commit 895cf0a
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 11 deletions.
diff --git a/src/pipx/main.py b/src/pipx/main.py
@@ -816,16 +816,7 @@ def setup_logging(verbose: bool) -> None:
     pipx.constants.pipx_log_file = setup_log_file()
 
     # Determine logging level
-    if verbose >= 1:
-        level_number = logging.DEBUG
-    elif verbose == -1:
-        level_number = logging.WARNING
-    elif verbose == -2:
-        level_number = logging.ERROR
-    elif verbose <= -3:
-        level_number = logging.CRITICAL
-    else:
-        level_number = logging.INFO
+    level_number = max(0, 2 - verbose) * 10
 
     level = logging.getLevelName(level_number)
 

diff --git a/tests/test_install.py b/tests/test_install.py
@@ -125,8 +125,8 @@ def test_include_deps(pipx_temp_env, capsys):
 @pytest.mark.parametrize(
     "package_name, package_spec",
     [
-        ("tox-ini-fmt", PKG["tox-ini-fmt"]["spec"]),
         ("zest-releaser", PKG["zest-releaser"]["spec"]),
+        ("tox-ini-fmt", PKG["tox-ini-fmt"]["spec"]),
     ],
 )
 def test_name_tricky_characters(