summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJames <morisja@gmail.com>2023-10-18 07:36:16 (GMT)
committerGitHub <noreply@github.com>2023-10-18 07:36:16 (GMT)
commit77dbd956090aac66e264d9d640f6adb6b0930b87 (patch)
tree29d66940586b49c487ed31c5d7177d2904b7d970
parent738574fb21967a1f313f1542dd7b70ae0dcd9705 (diff)
downloadcpython-77dbd956090aac66e264d9d640f6adb6b0930b87.zip
cpython-77dbd956090aac66e264d9d640f6adb6b0930b87.tar.gz
cpython-77dbd956090aac66e264d9d640f6adb6b0930b87.tar.bz2
gh-111019: Align expected and actual titles in test output (#111020)
Align expected and actual titles in output from assert_has_calls/assert_called_with for greater readability
-rw-r--r--Lib/test/test_unittest/testmock/testmock.py14
-rw-r--r--Lib/unittest/mock.py6
2 files changed, 10 insertions, 10 deletions
diff --git a/Lib/test/test_unittest/testmock/testmock.py b/Lib/test/test_unittest/testmock/testmock.py
index d23eb87..6af8acc 100644
--- a/Lib/test/test_unittest/testmock/testmock.py
+++ b/Lib/test/test_unittest/testmock/testmock.py
@@ -1058,7 +1058,7 @@ class MockTest(unittest.TestCase):
actual = 'not called.'
expected = "mock(1, '2', 3, bar='foo')"
- message = 'expected call not found.\nExpected: %s\nActual: %s'
+ message = 'expected call not found.\nExpected: %s\n Actual: %s'
self.assertRaisesWithMsg(
AssertionError, message % (expected, actual),
mock.assert_called_with, 1, '2', 3, bar='foo'
@@ -1073,7 +1073,7 @@ class MockTest(unittest.TestCase):
for meth in asserters:
actual = "foo(1, '2', 3, foo='foo')"
expected = "foo(1, '2', 3, bar='foo')"
- message = 'expected call not found.\nExpected: %s\nActual: %s'
+ message = 'expected call not found.\nExpected: %s\n Actual: %s'
self.assertRaisesWithMsg(
AssertionError, message % (expected, actual),
meth, 1, '2', 3, bar='foo'
@@ -1083,7 +1083,7 @@ class MockTest(unittest.TestCase):
for meth in asserters:
actual = "foo(1, '2', 3, foo='foo')"
expected = "foo(bar='foo')"
- message = 'expected call not found.\nExpected: %s\nActual: %s'
+ message = 'expected call not found.\nExpected: %s\n Actual: %s'
self.assertRaisesWithMsg(
AssertionError, message % (expected, actual),
meth, bar='foo'
@@ -1093,7 +1093,7 @@ class MockTest(unittest.TestCase):
for meth in asserters:
actual = "foo(1, '2', 3, foo='foo')"
expected = "foo(1, 2, 3)"
- message = 'expected call not found.\nExpected: %s\nActual: %s'
+ message = 'expected call not found.\nExpected: %s\n Actual: %s'
self.assertRaisesWithMsg(
AssertionError, message % (expected, actual),
meth, 1, 2, 3
@@ -1103,7 +1103,7 @@ class MockTest(unittest.TestCase):
for meth in asserters:
actual = "foo(1, '2', 3, foo='foo')"
expected = "foo()"
- message = 'expected call not found.\nExpected: %s\nActual: %s'
+ message = 'expected call not found.\nExpected: %s\n Actual: %s'
self.assertRaisesWithMsg(
AssertionError, message % (expected, actual), meth
)
@@ -1552,7 +1552,7 @@ class MockTest(unittest.TestCase):
'^{}$'.format(
re.escape('Calls not found.\n'
'Expected: [call()]\n'
- 'Actual: [call(1)]'))) as cm:
+ ' Actual: [call(1)]'))) as cm:
mock.assert_has_calls([call()])
self.assertIsNone(cm.exception.__cause__)
@@ -1564,7 +1564,7 @@ class MockTest(unittest.TestCase):
'Error processing expected calls.\n'
"Errors: [None, TypeError('too many positional arguments')]\n"
"Expected: [call(), call(1, 2)]\n"
- 'Actual: [call(1)]'))) as cm:
+ ' Actual: [call(1)]'))) as cm:
mock.assert_has_calls([call(), call(1, 2)])
self.assertIsInstance(cm.exception.__cause__, TypeError)
diff --git a/Lib/unittest/mock.py b/Lib/unittest/mock.py
index e8c8360..c6b46ee 100644
--- a/Lib/unittest/mock.py
+++ b/Lib/unittest/mock.py
@@ -857,7 +857,7 @@ class NonCallableMock(Base):
def _format_mock_failure_message(self, args, kwargs, action='call'):
- message = 'expected %s not found.\nExpected: %s\nActual: %s'
+ message = 'expected %s not found.\nExpected: %s\n Actual: %s'
expected_string = self._format_mock_call_signature(args, kwargs)
call_args = self.call_args
actual_string = self._format_mock_call_signature(*call_args)
@@ -960,7 +960,7 @@ class NonCallableMock(Base):
if self.call_args is None:
expected = self._format_mock_call_signature(args, kwargs)
actual = 'not called.'
- error_message = ('expected call not found.\nExpected: %s\nActual: %s'
+ error_message = ('expected call not found.\nExpected: %s\n Actual: %s'
% (expected, actual))
raise AssertionError(error_message)
@@ -1011,7 +1011,7 @@ class NonCallableMock(Base):
raise AssertionError(
f'{problem}\n'
f'Expected: {_CallList(calls)}'
- f'{self._calls_repr(prefix="Actual").rstrip(".")}'
+ f'{self._calls_repr(prefix=" Actual").rstrip(".")}'
) from cause
return