From 7552de7866dcd270a0f353b007b4aceaa7f3ff3e Mon Sep 17 00:00:00 2001 From: Nick Pope Date: Mon, 21 Oct 2019 09:55:05 +0100 Subject: Used more specific unittest assertions in tests. * assertIsNone()/assertIsNotNone() instead of comparing to None. * assertLess() for < comparisons. * assertIs() for 'is' expressions. * assertIsInstance() for isinstance() expressions. * rounding of assertAlmostEqual() for round() expressions. * assertIs(..., True/False) instead of comparing to True/False. * assertIs()/assertIsNot() for ==/!= comparisons. * assertNotEqual() for == comparisons. * assertTrue()/assertFalse() instead of comparing to True/False. --- tests/file_uploads/tests.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) (limited to 'tests/file_uploads') diff --git a/tests/file_uploads/tests.py b/tests/file_uploads/tests.py index 7b31d369b2..db9b915d35 100644 --- a/tests/file_uploads/tests.py +++ b/tests/file_uploads/tests.py @@ -479,8 +479,9 @@ class FileUploadTests(TestCase): try: self.client.post('/upload_errors/', post_data) except reference_error.__class__ as err: - self.assertFalse( - str(err) == str(reference_error), + self.assertNotEqual( + str(err), + str(reference_error), "Caught a repeated exception that'll cause an infinite loop in file uploads." ) except Exception as err: -- cgit v1.3