Force option type to int when testing log level in click

lsst · Jun 23, 2021 · ecdfe42 · ecdfe42
1 parent 32438ea
commit ecdfe42
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/python/lsst/daf/butler/tests/cliLogTestBase.py b/python/lsst/daf/butler/tests/cliLogTestBase.py
@@ -63,10 +63,10 @@ def hasLsstLogHandler(logger):
 
 
 @click.command()
-@click.option("--expected-pyroot-level")
-@click.option("--expected-pybutler-level")
-@click.option("--expected-lsstroot-level")
-@click.option("--expected-lsstbutler-level")
+@click.option("--expected-pyroot-level", type=int)
+@click.option("--expected-pybutler-level", type=int)
+@click.option("--expected-lsstroot-level", type=int)
+@click.option("--expected-lsstbutler-level", type=int)
 def command_log_settings_test(expected_pyroot_level,
                               expected_pybutler_level,
                               expected_lsstroot_level,
@@ -92,7 +92,7 @@ def command_log_settings_test(expected_pyroot_level,
                                    "lsstButler")])
     for expected, actual, name in logLevels:
         if expected != actual:
-            raise(click.ClickException(f"expected {name} level to be {expected}, actual:{actual}"))
+            raise(click.ClickException(f"expected {name} level to be {expected!r}, actual:{actual!r}"))
 
 
 class CliLogTestBase():