From 92c6ec188229628d69c650667537aefd5a63f61a Mon Sep 17 00:00:00 2001
From: Tyler Goodlet <jgbt@protonmail.com>
Date: Sun, 28 Nov 2021 12:46:57 -0500
Subject: [PATCH] `get_loglevel()` always returns a str

---
 tractor/_root.py | 10 +++++++---
 tractor/log.py   |  6 +++---
 2 files changed, 10 insertions(+), 6 deletions(-)

diff --git a/tractor/_root.py b/tractor/_root.py
index 4d48aca..0d082fc 100644
--- a/tractor/_root.py
+++ b/tractor/_root.py
@@ -87,11 +87,15 @@ async def open_root_actor(
         _default_arbiter_port,
     )
 
-    loglevel = loglevel or log.get_loglevel()
-    if loglevel is not None:
+
+    if loglevel is None:
+        loglevel = log.get_loglevel()
+    else:
         log._default_loglevel = loglevel
         log.get_console_log(loglevel)
 
+    assert loglevel
+
     if debug_mode and _spawn._spawn_method == 'trio':
         _state._runtime_vars['_debug_mode'] = True
 
@@ -105,7 +109,7 @@ async def open_root_actor(
             logging.getLevelName(
                 # lul, need the upper case for the -> int map?
                 # sweet "dynamic function behaviour" stdlib...
-                log.get_loglevel().upper()
+                loglevel.upper()
             ) > logging.getLevelName('PDB')
         ):
             loglevel = 'PDB'
diff --git a/tractor/log.py b/tractor/log.py
index bcba228..8daf20d 100644
--- a/tractor/log.py
+++ b/tractor/log.py
@@ -9,8 +9,8 @@ from typing import Optional
 from ._state import ActorContextInfo
 
 
-_proj_name = 'tractor'
-_default_loglevel = 'ERROR'
+_proj_name: str = 'tractor'
+_default_loglevel: str = 'ERROR'
 
 # Super sexy formatting thanks to ``colorlog``.
 # (NOTE: we use the '{' format style)
@@ -189,5 +189,5 @@ def get_console_log(
     return log
 
 
-def get_loglevel() -> Optional[str]:
+def get_loglevel() -> str:
     return _default_loglevel