[Python-checkins] r78229 - in python/trunk/Lib: test/test_unittest.py unittest/case.py
michael.foord
python-checkins at python.org
Thu Feb 18 22:37:08 CET 2010
Author: michael.foord
Date: Thu Feb 18 22:37:07 2010
New Revision: 78229
Log:
Fix unittest.TestCase.assertDictContainsSubset so it can't die with unicode issues when constructing failure messages. Issue 7956
Modified:
python/trunk/Lib/test/test_unittest.py
python/trunk/Lib/unittest/case.py
Modified: python/trunk/Lib/test/test_unittest.py
==============================================================================
--- python/trunk/Lib/test/test_unittest.py (original)
+++ python/trunk/Lib/test/test_unittest.py Thu Feb 18 22:37:07 2010
@@ -2573,13 +2573,10 @@
with self.assertRaises(self.failureException):
self.assertDictContainsSubset({'a': 1, 'c': 1}, {'a': 1})
- @unittest.expectedFailure
- def test_crazy(self):
one = ''.join(chr(i) for i in range(255))
- two = u'\uFFFD'
- first = {'foo': one}
- second = {'foo': two}
- self.assertDictContainsSubset(first, second)
+ # this used to cause a UnicodeDecodeError constructing the failure msg
+ with self.assertRaises(self.failureException):
+ self.assertDictContainsSubset({'foo': one}, {'foo': u'\uFFFD'})
def testAssertEqual(self):
equal_pairs = [
Modified: python/trunk/Lib/unittest/case.py
==============================================================================
--- python/trunk/Lib/unittest/case.py (original)
+++ python/trunk/Lib/unittest/case.py Thu Feb 18 22:37:07 2010
@@ -163,7 +163,7 @@
try:
testMethod = getattr(self, methodName)
except AttributeError:
- raise ValueError("no such test method in %s: %s" % \
+ raise ValueError("no such test method in %s: %s" %
(self.__class__, methodName))
self._testMethodDoc = testMethod.__doc__
self._cleanups = []
@@ -697,7 +697,7 @@
"""Just like self.assertTrue(a is b), but with a nicer default message."""
if expr1 is not expr2:
standardMsg = '%s is not %s' % (safe_repr(expr1),
- safe_repr(expr2))
+ safe_repr(expr2))
self.fail(self._formatMessage(msg, standardMsg))
def assertIsNot(self, expr1, expr2, msg=None):
@@ -725,7 +725,8 @@
missing.append(key)
elif value != actual[key]:
mismatched.append('%s, expected: %s, actual: %s' %
- (key, value, actual[key]))
+ (safe_repr(key), safe_repr(value),
+ safe_repr(actual[key])))
if not (missing or mismatched):
return
More information about the Python-checkins
mailing list