Fix NullLogger

thombashi · Feb 14, 2020 · 63c367b · 63c367b
1 parent 6b08cca
commit 63c367b
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 10 deletions.
diff --git a/simplesqlite/_logger/_null_logger.py b/simplesqlite/_logger/_null_logger.py
@@ -1,10 +1,10 @@
 class NullLogger(object):
     level_name = None
 
-    def critical(self, *args, **kwargs):  # pragma: no cover
+    def remove(self, handler_id=None):  # pragma: no cover
         pass
 
-    def debug(self, *args, **kwargs):  # pragma: no cover
+    def add(self, **kwargs):  # pragma: no cover
         pass
 
     def disable(self, name):  # pragma: no cover
@@ -13,26 +13,29 @@ def disable(self, name):  # pragma: no cover
     def enable(self, name):  # pragma: no cover
         pass
 
-    def error(self, *args, **kwargs):  # pragma: no cover
+    def critical(self, __message, *args, **kwargs):  # pragma: no cover
         pass
 
-    def exception(self, *args, **kwargs):  # pragma: no cover
+    def debug(self, __message, *args, **kwargs):  # pragma: no cover
         pass
 
-    def info(self, *args, **kwargs):  # pragma: no cover
+    def error(self, __message, *args, **kwargs):  # pragma: no cover
         pass
 
-    def log(self, level, *args, **kwargs):  # pragma: no cover
+    def exception(self, __message, *args, **kwargs):  # pragma: no cover
         pass
 
-    def notice(self, *args, **kwargs):  # pragma: no cover
+    def info(self, __message, *args, **kwargs):  # pragma: no cover
         pass
 
-    def success(self, *args, **kwargs):  # pragma: no cover
+    def log(self, __level, __message, *args, **kwargs):  # pragma: no cover
         pass
 
-    def trace(self, *args, **kwargs):  # pragma: no cover
+    def success(self, __message, *args, **kwargs):  # pragma: no cover
         pass
 
-    def warning(self, *args, **kwargs):  # pragma: no cover
+    def trace(self, __message, *args, **kwargs):  # pragma: no cover
+        pass
+
+    def warning(self, __message, *args, **kwargs):  # pragma: no cover
         pass
diff --git a/test/test_logger.py b/test/test_logger.py
@@ -9,9 +9,17 @@
 import pytest
 
 from simplesqlite import set_logger
+from simplesqlite._logger._null_logger import NullLogger
 
 
 class Test_set_logger(object):
     @pytest.mark.parametrize(["value"], [[True], [False]])
     def test_smoke(self, value):
         set_logger(value)
+
+
+class Test_NullLogger:
+    @pytest.mark.parametrize(["value"], [[True], [False]])
+    def test_smoke(self, value, monkeypatch):
+        monkeypatch.setattr("simplesqlite._logger._logger.logger", NullLogger())
+        set_logger(value)