diff -r b21e28790c03 Doc/library/unittest.mock.rst --- a/Doc/library/unittest.mock.rst Fri Sep 28 15:19:45 2012 +0100 +++ b/Doc/library/unittest.mock.rst Fri Sep 28 16:07:23 2012 +0100 @@ -276,7 +276,7 @@ >>> mock.assert_called_once_with('foo', bar='baz') Traceback (most recent call last): ... - AssertionError: Expected to be called once. Called 2 times. + AssertionError: Expected 'mock' to be called once. Called 2 times. .. method:: assert_any_call(*args, **kwargs) @@ -2020,7 +2020,7 @@ >>> mock.assert_called_once_with(1, 2, 3) Traceback (most recent call last): ... - AssertionError: Expected to be called once. Called 2 times. + AssertionError: Expected 'mock' to be called once. Called 2 times. Because mocks auto-create attributes on demand, and allow you to call them with arbitrary arguments, if you misspell one of these assert methods then diff -r b21e28790c03 Lib/unittest/mock.py --- a/Lib/unittest/mock.py Fri Sep 28 15:19:45 2012 +0100 +++ b/Lib/unittest/mock.py Fri Sep 28 16:07:23 2012 +0100 @@ -731,8 +731,8 @@ arguments.""" self = _mock_self if not self.call_count == 1: - msg = ("Expected to be called once. Called %s times." % - self.call_count) + msg = ("Expected '%s' to be called once. Called %s times." % + (self._mock_name or 'mock', self.call_count)) raise AssertionError(msg) return self.assert_called_with(*args, **kwargs) diff -r b21e28790c03 Lib/unittest/test/testmock/testmock.py --- a/Lib/unittest/test/testmock/testmock.py Fri Sep 28 15:19:45 2012 +0100 +++ b/Lib/unittest/test/testmock/testmock.py Fri Sep 28 16:07:23 2012 +0100 @@ -463,6 +463,13 @@ mock.assert_called_with) + def test_assert_called_once_with_message(self): + mock = Mock(name='geoffrey') + self.assertRaisesRegex(AssertionError, + r"Expected 'geoffrey' to be called once\.", + mock.assert_called_once_with) + + def test__name__(self): mock = Mock() self.assertRaises(AttributeError, lambda: mock.__name__)