From 0a26ee63ad78c5bcf0611fe7f81cc3ab275f664e Mon Sep 17 00:00:00 2001 From: Jesse Bannon Date: Fri, 13 Mar 2026 13:05:38 -0700 Subject: [PATCH] suppress logs for inspect --- src/ytdl_sub/main.py | 7 ++++++- tests/integration/cli/test_inspect.py | 5 +++++ 2 files changed, 11 insertions(+), 1 deletion(-) diff --git a/src/ytdl_sub/main.py b/src/ytdl_sub/main.py index 44258a5c..ad1c54f3 100644 --- a/src/ytdl_sub/main.py +++ b/src/ytdl_sub/main.py @@ -1,7 +1,7 @@ import sys from ytdl_sub.cli.parsers.main import parser -from ytdl_sub.utils.logger import Logger +from ytdl_sub.utils.logger import Logger, LoggerLevels def _main() -> int: @@ -10,6 +10,11 @@ def _main() -> int: args, _ = parser.parse_known_args() Logger.set_log_level(log_level_name=args.ytdl_sub_log_level) + # Suppress all logs during inspection since the output of the subcommand itself + # is all that is necessary + if args.subparser == "inspect": + Logger.set_log_level(log_level_name=LoggerLevels.QUIET.name) + # pylint: disable=import-outside-toplevel import ytdl_sub.cli.entrypoint diff --git a/tests/integration/cli/test_inspect.py b/tests/integration/cli/test_inspect.py index e7136317..f593baca 100644 --- a/tests/integration/cli/test_inspect.py +++ b/tests/integration/cli/test_inspect.py @@ -6,6 +6,7 @@ from conftest import mock_run_from_cli from unit.config.test_subscription_resolution import compare_resolved_yaml from ytdl_sub.config.validators.variable_validation import ResolutionLevel +from ytdl_sub.utils.system import IS_WINDOWS class TestInspect: @@ -22,6 +23,10 @@ class TestInspect: config_provided: bool, inspect_level: str, ): + # TODO: fix mock_run_from_cli in windows to handle file paths correctly + if IS_WINDOWS: + return + # Shares same test fixture as `test_subscription_resolution.py` args = f"--config {default_config_path} " if config_provided else "" args += f"inspect {tv_show_subscriptions_path} --match 'NOVA PBS' "