Configure PythonHere logging defaults

Author: b3b

pythonhere/main.py

@@ -12,12 +12,14 @@
     startup_script_exception = None  # pylint: disable=invalid-name
 
 import asyncio
+import logging
 import os
 import sys
 import threading
 from pathlib import Path
 from typing import Any
 
+import asyncssh
 from enum_here import ScreenName, ServerState
 from exception_manager_here import install_exception_handler, show_exception_popup
 from kivy.app import App
@@ -31,6 +33,12 @@
 monkeypatch_kivy()
 
 
+def configure_logging():
+    """Configure logging for the app runtime."""
+    logging.getLogger().setLevel(logging.INFO)
+    asyncssh.set_log_level("WARNING")
+
+
 class PythonHereApp(App):
     """PythonHere main app."""
 
@@ -217,6 +225,7 @@ def chdir(self, path: str):
 
 async def main():
     """Run PythonHere."""
+    configure_logging()
     app = PythonHereApp()
     await app.run_app()
 

tests/test_main.py

@@ -1,12 +1,13 @@
 import asyncio
+import logging
 from pathlib import Path
 from types import SimpleNamespace
 from unittest.mock import PropertyMock
 
 import pytest
 from asyncssh import PermissionDenied
 from enum_here import ScreenName, ServerState
-from main import PythonHereApp
+from main import PythonHereApp, configure_logging, main
 from ui_here.server_screen_here import ServerScreenManager
 from version_here import __version__
 
@@ -15,6 +16,36 @@ def test_dev_version_is_set():
     assert __version__ == "0.0.0"
 
 
+def test_configure_logging_sets_default_info_and_asyncssh_warning(mocker):
+    set_log_level = mocker.patch("main.asyncssh.set_log_level")
+    root_logger = logging.getLogger()
+
+    original_level = root_logger.level
+    try:
+        root_logger.setLevel(logging.NOTSET)
+
+        configure_logging()
+
+        assert root_logger.level == logging.INFO
+        set_log_level.assert_called_once_with("WARNING")
+    finally:
+        root_logger.setLevel(original_level)
+
+
+@pytest.mark.asyncio
+async def test_main_configures_logging_before_running_app(mocker):
+    configure = mocker.patch("main.configure_logging")
+    app = mocker.Mock()
+    app.run_app = mocker.AsyncMock()
+    app_class = mocker.patch("main.PythonHereApp", return_value=app)
+
+    await main()
+
+    configure.assert_called_once_with()
+    app_class.assert_called_once_with()
+    app.run_app.assert_awaited_once_with()
+
+
 def test_server_screen_update_states(mocker):
     screen = SimpleNamespace(current=None)
     app = SimpleNamespace(