Using assertIsNone(xxx) instead of assertEqual(None, xxx)

Following OpenStack Style Guidelines:
[1] https://docs.openstack.org/developer/hacking/#unit-tests-and-assertraises
[H203] Unit test assertions tend to give better messages for more specific
assertions. As a result, assertIsNone(xxx) is preferred over
assertEqual(None, xxx) and assertIs(xxx, None)

Change-Id: Id0e54ddeb461e56fbc8ec3a4bbc8a581f11430df
This commit is contained in:
blue55 2017-06-06 09:50:44 +08:00
parent 516bb61947
commit 5da6884dbc
1 changed files with 1 additions and 1 deletions

View File

@ -211,7 +211,7 @@ class SoftDeleteMixinTest(test_base.DbTestCase):
self.session.add(m)
self.session.commit()
self.assertEqual(0, m.deleted)
self.assertIs(None, m.deleted_at)
self.assertIsNone(m.deleted_at)
m.soft_delete(self.session)
self.assertEqual(123456, m.deleted)