From 51a6a8b687c0508707c62bbb1adfd5744d119cce Mon Sep 17 00:00:00 2001 From: howardlee Date: Mon, 7 Nov 2016 16:50:39 +0800 Subject: [PATCH] Use assertIs(Not)None to check for None [H203] Use assertIs(Not)None to check for None (off by default) Unit test assertions tend to give better messages for more specific assertions. As a result, assertIsNone(...) is preferred over assertEqual(None, ...) and assertIs(None, ...), and assertIsNotNone(...) is preferred over assertNotEqual(None, ...) and assertIsNot(None, ...). Off by default. Trivial fix Change-Id: I5b6ac7d99f0689843eb98cb3e9b9b10531322640 --- taskflow/tests/unit/persistence/base.py | 2 +- taskflow/tests/unit/test_failure.py | 10 +++++----- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/taskflow/tests/unit/persistence/base.py b/taskflow/tests/unit/persistence/base.py index a52db5088..d7e10bc06 100644 --- a/taskflow/tests/unit/persistence/base.py +++ b/taskflow/tests/unit/persistence/base.py @@ -311,7 +311,7 @@ class PersistenceTestMixin(object): lb2 = conn.get_logbook(lb_id) fd2 = lb2.find(fd.uuid) td2 = fd2.find(td.uuid) - self.assertIsNot(td2, None) + self.assertIsNotNone(td2) self.assertEqual('detail-1', td2.name) self.assertEqual('4.2', td2.version) self.assertEqual(states.EXECUTE, td2.intention) diff --git a/taskflow/tests/unit/test_failure.py b/taskflow/tests/unit/test_failure.py index 536ddbc79..99c2e70e5 100644 --- a/taskflow/tests/unit/test_failure.py +++ b/taskflow/tests/unit/test_failure.py @@ -70,7 +70,7 @@ class GeneralFailureObjTestsMixin(object): self.assertIs(self.fail_obj.check(RuntimeError), RuntimeError) def test_check_type_not_there(self): - self.assertIs(self.fail_obj.check(ValueError), None) + self.assertIsNone(self.fail_obj.check(ValueError)) class CaptureFailureTestCase(test.TestCase, GeneralFailureObjTestsMixin): @@ -102,10 +102,10 @@ class ReCreatedFailureTestCase(test.TestCase, GeneralFailureObjTestsMixin): exc_type_names=list(fail_obj)) def test_value_lost(self): - self.assertIs(self.fail_obj.exception, None) + self.assertIsNone(self.fail_obj.exception) def test_no_exc_info(self): - self.assertIs(self.fail_obj.exc_info, None) + self.assertIsNone(self.fail_obj.exc_info) def test_pformat_traceback(self): text = self.fail_obj.pformat(traceback=True) @@ -205,7 +205,7 @@ class FailureObjectTestCase(test.TestCase): self.assertEqual(expected, str(exc)) def test_empty_does_not_reraise(self): - self.assertIs(failure.Failure.reraise_if_any([]), None) + self.assertIsNone(failure.Failure.reraise_if_any([])) def test_reraises_one(self): fls = [_captured_failure('Woot!')] @@ -269,7 +269,7 @@ class FailureObjectTestCase(test.TestCase): def test_compares_to_none(self): captured = _captured_failure('Woot!') - self.assertNotEqual(captured, None) + self.assertIsNotNone(captured) self.assertFalse(captured.matches(None)) def test_pformat_traceback(self):