Move apply_log_level() to log_handler.py

Moved the apply_log_level() function from ttmp32gme.py to log_handler.py for better code organization. Updated imports in both ttmp32gme.py and test_log_level_change.py accordingly.

Co-authored-by: thawn <1308449+thawn@users.noreply.github.com>

Author: Copilot

src/ttmp32gme/log_handler.py

@@ -5,6 +5,39 @@
 from threading import Lock
 from typing import List
 
+# Get logger for this module
+logger = logging.getLogger(__name__)
+
+
+def apply_log_level(level_str: str) -> None:
+    """Apply log level to all relevant loggers.
+
+    Args:
+        level_str: Log level string (DEBUG, INFO, WARNING, ERROR, CRITICAL)
+    """
+    level = getattr(logging, level_str, logging.WARNING)
+
+    # Set root logger level
+    logging.getLogger().setLevel(level)
+
+    # Set werkzeug logger level
+    # When not in DEBUG/INFO mode, suppress werkzeug's INFO logs to reduce clutter
+    werkzeug_logger = logging.getLogger("werkzeug")
+    if level_str in ["DEBUG", "INFO"]:
+        werkzeug_logger.setLevel(level)
+    else:
+        # Suppress werkzeug's INFO logs (like request logs) when not in verbose mode
+        werkzeug_logger.setLevel(logging.WARNING)
+
+    # Set waitress logger level (for production server)
+    waitress_logger = logging.getLogger("waitress")
+    if level_str in ["DEBUG", "INFO"]:
+        waitress_logger.setLevel(level)
+    else:
+        waitress_logger.setLevel(logging.WARNING)
+
+    logger.info(f"Log level changed to {level_str}")
+
 
 class MemoryLogHandler(logging.Handler):
     """Custom log handler that stores recent log records in memory."""

src/ttmp32gme/ttmp32gme.py

@@ -36,7 +36,7 @@
     DBHandler,
     LibraryActionModel,
 )
-from ttmp32gme.log_handler import MemoryLogHandler
+from ttmp32gme.log_handler import MemoryLogHandler, apply_log_level
 from ttmp32gme.print_handler import (
     create_print_layout,
     format_print_button,
@@ -115,36 +115,6 @@ def get_db():
     return db_handler
 
 
-def apply_log_level(level_str: str) -> None:
-    """Apply log level to all relevant loggers.
-
-    Args:
-        level_str: Log level string (DEBUG, INFO, WARNING, ERROR, CRITICAL)
-    """
-    level = getattr(logging, level_str, logging.WARNING)
-
-    # Set root logger level
-    logging.getLogger().setLevel(level)
-
-    # Set werkzeug logger level
-    # When not in DEBUG/INFO mode, suppress werkzeug's INFO logs to reduce clutter
-    werkzeug_logger = logging.getLogger("werkzeug")
-    if level_str in ["DEBUG", "INFO"]:
-        werkzeug_logger.setLevel(level)
-    else:
-        # Suppress werkzeug's INFO logs (like request logs) when not in verbose mode
-        werkzeug_logger.setLevel(logging.WARNING)
-
-    # Set waitress logger level (for production server)
-    waitress_logger = logging.getLogger("waitress")
-    if level_str in ["DEBUG", "INFO"]:
-        waitress_logger.setLevel(level)
-    else:
-        waitress_logger.setLevel(logging.WARNING)
-
-    logger.info(f"Log level changed to {level_str}")
-
-
 def fetch_config() -> Dict[str, Any]:
     """Fetch configuration from database."""
     db = get_db()

tests/unit/test_log_level_change.py

@@ -10,7 +10,7 @@
 
 def test_apply_log_level_sets_root_logger():
     """Test that apply_log_level sets the root logger level."""
-    from ttmp32gme.ttmp32gme import apply_log_level
+    from ttmp32gme.log_handler import apply_log_level
 
     root_logger = logging.getLogger()
     original_level = root_logger.level
@@ -35,7 +35,7 @@ def test_apply_log_level_sets_root_logger():
 
 def test_apply_log_level_sets_werkzeug_logger():
     """Test that apply_log_level sets werkzeug logger appropriately."""
-    from ttmp32gme.ttmp32gme import apply_log_level
+    from ttmp32gme.log_handler import apply_log_level
 
     werkzeug_logger = logging.getLogger("werkzeug")
     root_logger = logging.getLogger()
@@ -67,7 +67,7 @@ def test_apply_log_level_sets_werkzeug_logger():
 
 def test_apply_log_level_sets_waitress_logger():
     """Test that apply_log_level sets waitress logger appropriately."""
-    from ttmp32gme.ttmp32gme import apply_log_level
+    from ttmp32gme.log_handler import apply_log_level
 
     waitress_logger = logging.getLogger("waitress")
     root_logger = logging.getLogger()
@@ -99,7 +99,7 @@ def test_apply_log_level_sets_waitress_logger():
 
 def test_apply_log_level_logs_info_message(caplog):
     """Test that apply_log_level logs an info message."""
-    from ttmp32gme.ttmp32gme import apply_log_level
+    from ttmp32gme.log_handler import apply_log_level
 
     root_logger = logging.getLogger()
     original_level = root_logger.level