v0.8.1. All tests are passing. So this closes #36 and closes #31

pout/__init__.py

@@ -61,7 +61,7 @@
 )
 
 
-__version__ = '0.8.0'
+__version__ = '0.8.1'
 
 
 # This is the standard logger for debugging pout itself, if it hasn't been
@@ -71,7 +71,6 @@
 # http://stackoverflow.com/questions/6333916/python-logging-ensure-a-handler-is-added-only-once
 if len(logger.handlers) == 0:
     logger.setLevel(logging.WARNING)
-    #logger.setLevel(logging.ERROR)
     logger.addHandler(logging.NullHandler())
 
 
@@ -123,13 +122,6 @@ def v(*args, **kwargs):
     if not args:
         raise ValueError("you didn't pass any arguments to print out")
 
-    #frame = inspect.currentframe()
-#     frames = inspect.stack()
-#     frame = frames[0]
-#     modname = inspect.getmodule(frame[0]).__name__
-#     funcname = frame[0].f_code.co_name
-#     print("{}:{} {} {}.{}".format(frame[1], frame[2], frame[4], modname, funcname))
-
     with Reflect.context(args, **kwargs) as r:
         instance = V_CLASS(r, stream, **kwargs)
         instance()

pout/environ.py

@@ -9,6 +9,8 @@
 ENCODING = os.environ.get("POUT_ENCODING", "UTF-8")
 ENCODING_REPLACE = os.environ.get("POUT_ENCODING_REPLACE", "pout.replace")
 
+OBJECT_DEPTH = int(os.environ.get("POUT_OBJECT_DEPTH", 4))
+
 
 def handle_decode_replace(cls, e):
     """this handles replacing bad characters when printing out

pout/interface.py

@@ -116,8 +116,6 @@ def _str(self, name, val):
 
             except (TypeError, KeyError, AttributeError) as e:
                 logger.warning(e, exc_info=True)
-
-            else:
                 s = "{} = {}".format(name, self._str_val(val))
 
         else:
@@ -149,7 +147,6 @@ def value(self):
 
 
 class ValueInterface(BaseInterface):
-
     def name_value(self):
         call_info = self.reflect.info
         args = ["{}\n\n".format(self._str(v['name'], v['val'])) for v in call_info['args']]
@@ -160,17 +157,6 @@ def value(self):
         args = ["{}\n\n".format(self._str(None, v['val'])) for v in call_info['args']]
         return self._printstr(args)
 
-#     def __repr__(self):
-#         call_info = self.reflect.info
-#         #pout2.v(call_info)
-#         args = ["{}\n\n".format(self._str(v['name'], v['val'])) for v in call_info['args']]
-#         return self._printstr(args, call_info)
-
-#     def value(self):
-#         """Returns only the value of the passed in args, no context information"""
-#         call_info = self.reflect.info
-#         args = ["{}\n\n".format(self._str(None, v['val'])) for v in call_info['args']]
-#         return self._printstr(args)
 
 
 class HereInterface(BaseInterface):

pout/value.py

@@ -6,13 +6,17 @@
 import os
 import traceback
 from collections import KeysView
+import logging
 
 from .compat import *
 from . import environ
 from .path import Path
 from .utils import String
 
 
+logger = logging.getLogger(__name__)
+
+
 class Inspect(object):
 
     @property
@@ -481,8 +485,11 @@ def _getattr(self, val, key, default_val):
         """wrapper around global getattr(...) method that suppresses any exception raised"""
         try:
             ret = getattr(val, key, default_val)
-        except Exception:
+
+        except Exception as e:
+            logger.exception(e)
             ret = default_val
+
         return ret
 
     def _get_name(self, val, src_file, default='Unknown'):
@@ -565,7 +572,7 @@ def __repr__(self):
             s += repr(Value(val.__pout__()))
 
         else:
-            if depth < 4:
+            if depth < environ.OBJECT_DEPTH:
                 s += "\n<"
                 s_body = ''
 

pout_test/pout_test.py

@@ -171,35 +171,6 @@ class Module(object): pass
         del self.issue_module
         del self.issue_fields
 
-    def test__get_name(self):
-        """makes sure if __getattr__ raises other errors than AttributeError then
-        pout will still print correctly"""
-        class FooGetName(object):
-            def __init__(self):
-                self.fields = {}
-            def __getattr__(self, key):
-                # This will raise a KeyError when key doesn't exist
-                return self.fields[key]
-
-        with testdata.capture() as c:
-            fgn = FooGetName()
-            pout.v(fgn)
-        for s in ["pout_test.FooGetName", "id:", "path:", "Ancestry:", "__str__:", "fields = "]:
-            self.assertTrue(s in c, s)
-
-    def test_find_call_depth(self):
-        s = "foo"
-        class PoutChild(pout.Pout):
-            def v(self, *args):
-                self._printstr("PoutChild")
-                super(PoutChild, self).v(*args)
-
-        pout.pout_class = PoutChild
-        with testdata.capture() as c:
-            pout.v(s)
-        self.assertTrue('s (3) = "foo"' in c)
-        pout.pout_class = pout.Pout
-
     def test__get_arg_info(self):
         foo = 1
         with testdata.capture() as c:
@@ -391,6 +362,22 @@ def test_ss_return(self):
 
 
 class VTest(unittest.TestCase):
+    def test_get_name(self):
+        """makes sure if __getattr__ raises other errors than AttributeError then
+        pout will still print correctly"""
+        class FooGetName(object):
+            def __init__(self):
+                self.fields = {}
+            def __getattr__(self, key):
+                # This will raise a KeyError when key doesn't exist
+                return self.fields[key]
+
+        with testdata.capture() as c:
+            fgn = FooGetName()
+            pout.v(fgn)
+        for s in ["pout_test.FooGetName", "id:", "path:", "Ancestry:", "__str__:", "fields = "]:
+            self.assertTrue(s in c, s)
+
     def test_vs(self):
         with testdata.capture() as c:
             d = {'foo': 1, 'bar': 2}