Pylint part 2,

Author: penguinolog

logwrap/_alogwrap.py

@@ -97,7 +97,7 @@ def wrapper(*args, **kwargs):
         return wrapper
 
 
-# pylint: disable=unexpected-keyword-arg
+# pylint: disable=unexpected-keyword-arg, no-value-for-parameter
 def async_logwrap(
     log: logging.Logger=_log_wrap_shared.logger,
     log_level: int=logging.DEBUG,
@@ -152,4 +152,4 @@ def async_logwrap(
         log_call_args_on_exc=log_call_args_on_exc,
         log_result_obj=log_result_obj
     )
-# pylint: enable=unexpected-keyword-arg
+# pylint: enable=unexpected-keyword-arg, no-value-for-parameter

logwrap/_log_wrap.py

@@ -104,7 +104,7 @@ def wrapper(*args, **kwargs):
         return wrapper
 
 
-# pylint: disable=unexpected-keyword-arg
+# pylint: disable=unexpected-keyword-arg, no-value-for-parameter
 def logwrap(
     log=_log_wrap_shared.logger,
     log_level=logging.DEBUG,
@@ -159,4 +159,4 @@ def logwrap(
         log_call_args_on_exc=log_call_args_on_exc,
         log_result_obj=log_result_obj
     )
-# pylint: enable=unexpected-keyword-arg
+# pylint: enable=unexpected-keyword-arg, no-value-for-parameter

test/test_log_wrap.py

@@ -477,6 +477,29 @@ def func(test_arg1, test_arg2):
             ),
         ))
 
+    def test_exceptions_blacklist(self, logger):
+        new_logger = mock.Mock(spec=logging.Logger, name='logger')
+        log = mock.Mock(name='log')
+        new_logger.attach_mock(log, 'log')
+
+        @logwrap.logwrap(log=new_logger, blacklisted_exceptions=[TypeError])
+        def func():
+            raise TypeError('Blacklisted')
+
+        with self.assertRaises(TypeError):
+            func()
+
+        self.assertEqual(len(logger.mock_calls), 0)
+        self.assertEqual(
+            log.mock_calls,
+            [
+                mock.call(
+                    level=logging.DEBUG,
+                    msg="Calling: \n'func'()"
+                ),
+            ]
+        )
+
 
 @mock.patch('logwrap._log_wrap_shared.logger', autospec=True)
 @unittest.skipUnless(
@@ -647,3 +670,42 @@ def func():
                 msg="Done: 'func' with result:\nNone"
             )
         ))
+
+    def test_exceptions_blacklist(self, logger):
+        new_logger = mock.Mock(spec=logging.Logger, name='logger')
+        log = mock.Mock(name='log')
+        new_logger.attach_mock(log, 'log')
+
+        namespace = {
+            'logwrap': logwrap,
+            'loop': self.loop,
+            'new_logger': new_logger,
+            'assertRaises': self.assertRaises
+        }
+
+        exec("""
+import asyncio
+
+@logwrap.async_logwrap(log=new_logger, blacklisted_exceptions=[TypeError])
+@asyncio.coroutine
+def func():
+    raise TypeError('Blacklisted')
+
+with assertRaises(TypeError):
+    loop.run_until_complete(func())
+        """,
+             namespace
+             )
+        # While we're not expanding result coroutine object from namespace,
+        # do not check execution result
+
+        self.assertEqual(len(logger.mock_calls), 0)
+        self.assertEqual(
+            log.mock_calls,
+            [
+                mock.call(
+                    level=logging.DEBUG,
+                    msg="Awaiting: \n'func'()"
+                ),
+            ]
+        )