openstack/zaqar
Code Issues Proposed changes

Merge "Using assertFalse(A) instead of assertEqual(False, A)"

Browse Source
This commit is contained in:
Jenkins 2017-06-26 05:19:28 +00:00 committed by Gerrit Code Review
commit 5e056b0dc6
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
zaqar/tests/unit/storage/base.py
View File

@ -1281,7 +1281,7 @@ class SubscriptionControllerTest(ControllerBaseTest):
subscription = self.subscription_controller.get(self.source, s_id, subscription = self.subscription_controller.get(self.source, s_id,
project=self.project) project=self.project)
self.assertEqual(False, subscription['confirmed']) self.assertFalse(subscription['confirmed'])
self.subscription_controller.confirm(self.source, s_id, self.subscription_controller.confirm(self.source, s_id,
project=self.project, project=self.project,
@ -1289,7 +1289,7 @@ class SubscriptionControllerTest(ControllerBaseTest):
subscription = self.subscription_controller.get(self.source, s_id, subscription = self.subscription_controller.get(self.source, s_id,
project=self.project) project=self.project)
self.assertEqual(True, subscription['confirmed']) self.assertTrue(subscription['confirmed'])
def test_confirm_with_nonexist_subscription(self): def test_confirm_with_nonexist_subscription(self):
s_id = 'fake-id' s_id = 'fake-id'