From 2413284e1710c8a989130ff182ebea6f5789ffcf Mon Sep 17 00:00:00 2001 From: Alex Richards Date: Mon, 5 Jan 2026 21:17:56 +0000 Subject: [PATCH] Remove config logging from init and suppress pydub warning --- src/djtools/__init__.py | 7 +++++++ src/djtools/configs/config_formatter.py | 11 ----------- tests/configs/test_base_config_formatter.py | 17 ----------------- 3 files changed, 7 insertions(+), 28 deletions(-) diff --git a/src/djtools/__init__.py b/src/djtools/__init__.py index 0582d8f2..6ad73cff 100644 --- a/src/djtools/__init__.py +++ b/src/djtools/__init__.py @@ -9,6 +9,12 @@ is uploaded to the Beatcloud. """ +import warnings + +warnings.filterwarnings("ignore", message="Couldn't find ffmpeg or avconv") +warnings.filterwarnings("ignore", message="Couldn't find ffprobe or avprobe") + +# pylint: disable=wrong-import-position from .configs import build_config from .collection import ( COLLECTION_OPERATIONS, @@ -73,6 +79,7 @@ def main(): logger, log_file = initialize_logger() config = build_config() logger.setLevel(config.log_level.value) + logger.info(repr(config)) # Run "collection", "spotify", "sync", and "utils" package operations if # any of the flags to do so are present in the config. diff --git a/src/djtools/configs/config_formatter.py b/src/djtools/configs/config_formatter.py index 9e22b26e..6e6b5cc9 100644 --- a/src/djtools/configs/config_formatter.py +++ b/src/djtools/configs/config_formatter.py @@ -2,25 +2,14 @@ BaseModel that defines a __repr__ method to be used by all the config objects used in this library.""" -import logging from typing import Any from pydantic import BaseModel -logger = logging.getLogger(__name__) - - class BaseConfigFormatter(BaseModel, extra="forbid"): """Pydantic BaseModel with a __repr__ method for pretty printing.""" - def __init__(self, *args, **kwargs): - """Constructor.""" - super().__init__(*args, **kwargs) - - if self.__class__.__name__ == "BaseConfig": - logger.info(repr(self)) - def _format_value(self, value: Any, indent_level: int) -> str: spaces = "\t" * indent_level diff --git a/tests/configs/test_base_config_formatter.py b/tests/configs/test_base_config_formatter.py index c6f5b774..a3fb39b5 100644 --- a/tests/configs/test_base_config_formatter.py +++ b/tests/configs/test_base_config_formatter.py @@ -1,7 +1,5 @@ """Testing for the config_formatter module.""" -from unittest import mock - from pydantic import ValidationError from djtools.configs.config_formatter import BaseConfigFormatter @@ -21,21 +19,6 @@ class SampleConfig(BaseConfigFormatter): assert repr(config) == expected_repr -def test_base_config_formatter_logging(): - """Define a class named "BaseConfig" to trigger the logging behavior.""" - with mock.patch("djtools.configs.config_formatter.logger") as mock_logger: - - class BaseConfig(BaseConfigFormatter): - """Dummy class.""" - - field: str - - _ = BaseConfig(field="test") - mock_logger.info.assert_called_once_with( - "BaseConfig(\n\tfield='test'\n)" - ) - - def test_base_config_formatter_nested(): """Define nested configurations."""