openstack/deb-designate
Code Issues Proposed changes

Replace assertTrue(a in b) with assertIn(a, b)

Browse Source 
Since python2.6 is no more supported, replacing
assertTrue(a in b) with assertIn(a, b) can be done.
This will give better message during the test failure
as a bonus.

Change-Id: I14b1801dd710eaf6d8433a6193eeaacd5d1c5a90
This commit is contained in:
Pradeep Kumar Singh 2015-07-29 09:43:26 +09:00
parent 65694bc2f0
commit 53f4eb62ae
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
designate/tests/unit/test_objects/test_base.py
View File

@ -598,7 +598,7 @@ class DictObjectMixinTest(oslotest.base.BaseTestCase):
def test_contains(self):
obj = TestObjectDict(name=1)
self.assertTrue('name' in obj)
self.assertIn('name', obj)
def test_get(self):
obj = TestObjectDict(name=1)
@ -799,9 +799,9 @@ class ListObjectMixinTest(oslotest.base.BaseTestCase):
# Create a ListObject
obj = TestObjectList(objects=[obj_one, obj_two])
self.assertTrue(obj_one in obj)
self.assertTrue(obj_two in obj)
self.assertFalse(obj_three in obj)
self.assertIn(obj_one, obj)
self.assertIn(obj_two, obj)
self.assertNotIn(obj_three, obj)
def test_extend(self):
# Create a few objects