feat(logging): add get_logger helper
This commit is contained in:
@@ -104,3 +104,8 @@ def format_tool_log(
|
||||
line += f"\n {error_message}"
|
||||
|
||||
return line
|
||||
|
||||
|
||||
def get_logger(name: str) -> logging.Logger:
|
||||
"""Get a child logger under the grist_mcp namespace."""
|
||||
return logging.getLogger(f"grist_mcp.{name}")
|
||||
|
||||
@@ -150,3 +150,21 @@ class TestSetupLogging:
|
||||
|
||||
logger = logging.getLogger("grist_mcp")
|
||||
assert logger.level == logging.INFO
|
||||
|
||||
|
||||
class TestGetLogger:
|
||||
def test_returns_child_logger(self):
|
||||
from grist_mcp.logging import get_logger
|
||||
|
||||
logger = get_logger("server")
|
||||
assert logger.name == "grist_mcp.server"
|
||||
|
||||
def test_inherits_parent_level(self, monkeypatch):
|
||||
monkeypatch.setenv("LOG_LEVEL", "WARNING")
|
||||
|
||||
from grist_mcp.logging import setup_logging, get_logger
|
||||
setup_logging()
|
||||
|
||||
logger = get_logger("test")
|
||||
# Child inherits from parent when level is NOTSET
|
||||
assert logger.getEffectiveLevel() == logging.WARNING
|
||||
|
||||
Reference in New Issue
Block a user