Hi, > changeset: 71465:be558ad15789 > user: Eli Bendersky <eli...@gmail.com> > summary: > Issue #11049: adding some tests to test.support > Based on original patch by Giampaolo Rodola with contributions from R. David > Murray > > files: > Lib/test/support.py | 21 +- > Lib/test/test_support.py | 178 +++++++++++++++++++++++++++ > 2 files changed, 189 insertions(+), 10 deletions(-) > > diff --git a/Lib/test/support.py b/Lib/test/support.py > --- a/Lib/test/support.py > +++ b/Lib/test/support.py > @@ -170,7 +170,7 @@ > attribute = getattr(obj, name) > except AttributeError: > raise unittest.SkipTest("module %s has no attribute %s" % ( > - obj.__name__, name)) > + repr(obj), name))
I would use %r instead of %s for both fields here. Non-ASCII characters and unseen whitespace are at least two reasons to overuse %r in debug/error messages instead of %s. Regards _______________________________________________ Python-Dev mailing list Python-Dev@python.org http://mail.python.org/mailman/listinfo/python-dev Unsubscribe: http://mail.python.org/mailman/options/python-dev/archive%40mail-archive.com