From cb2b0e45d4384a45e0852e8e7eba958a70949abb Mon Sep 17 00:00:00 2001 From: Benjamin Peterson Date: Mon, 23 Mar 2009 22:29:45 +0000 Subject: comply with the evilJavaNamingScheme for attribute names It seems my love of PEP 8 overrode the need for consistentcy --- Doc/library/unittest.rst | 4 ++-- Lib/test/test_unittest.py | 4 ++-- Lib/unittest.py | 14 +++++++------- 3 files changed, 11 insertions(+), 11 deletions(-) diff --git a/Doc/library/unittest.rst b/Doc/library/unittest.rst index 22b2870..69227cf 100644 --- a/Doc/library/unittest.rst +++ b/Doc/library/unittest.rst @@ -950,7 +950,7 @@ tools which support interactive reporting while tests are being run. :func:`expectedFailure` decorator. The default implementation appends a tuple ``(test, formatted_err)`` to the - instance's ``expected_failures`` attribute, where *formatted_err* is a + instance's ``expectedFailures`` attribute, where *formatted_err* is a formatted traceback derived from *err*. @@ -960,7 +960,7 @@ tools which support interactive reporting while tests are being run. decorator, but succeeded. The default implementation appends the test to the instance's - ``unexpected_successes`` attribute. + ``unexpectedSuccesses`` attribute. .. _testloader-objects: diff --git a/Lib/test/test_unittest.py b/Lib/test/test_unittest.py index 6a32dbf..cd8f967 100644 --- a/Lib/test/test_unittest.py +++ b/Lib/test/test_unittest.py @@ -2362,7 +2362,7 @@ class Test_TestSkipping(TestCase): test.run(result) self.assertEqual(events, ['startTest', 'addExpectedFailure', 'stopTest']) - self.assertEqual(result.expected_failures[0][0], test) + self.assertEqual(result.expectedFailures[0][0], test) self.assertTrue(result.wasSuccessful()) def test_unexpected_success(self): @@ -2377,7 +2377,7 @@ class Test_TestSkipping(TestCase): self.assertEqual(events, ['startTest', 'addUnexpectedSuccess', 'stopTest']) self.assertFalse(result.failures) - self.assertEqual(result.unexpected_successes, [test]) + self.assertEqual(result.unexpectedSuccesses, [test]) self.assertTrue(result.wasSuccessful()) diff --git a/Lib/unittest.py b/Lib/unittest.py index 8263887..74585ee 100644 --- a/Lib/unittest.py +++ b/Lib/unittest.py @@ -176,8 +176,8 @@ class TestResult(object): self.errors = [] self.testsRun = 0 self.skipped = [] - self.expected_failures = [] - self.unexpected_successes = [] + self.expectedFailures = [] + self.unexpectedSuccesses = [] self.shouldStop = False def startTest(self, test): @@ -209,12 +209,12 @@ class TestResult(object): def addExpectedFailure(self, test, err): """Called when an expected failure/error occured.""" - self.expected_failures.append( + self.expectedFailures.append( (test, self._exc_info_to_string(err, test))) def addUnexpectedSuccess(self, test): """Called when a test was expected to fail, but succeed.""" - self.unexpected_successes.append(test) + self.unexpectedSuccesses.append(test) def wasSuccessful(self): "Tells whether or not this result was a success" @@ -923,10 +923,10 @@ class TextTestRunner(object): self.stream.writeln("Ran %d test%s in %.3fs" % (run, run != 1 and "s" or "", timeTaken)) self.stream.writeln() - results = map(len, (result.expected_failures, - result.unexpected_successes, + results = map(len, (result.expectedFailures, + result.unexpectedSuccesses, result.skipped)) - expected_fails, unexpected_successes, skipped = results + expectedFails, unexpectedSuccesses, skipped = results infos = [] if not result.wasSuccessful(): self.stream.write("FAILED") -- cgit v0.12