From 086c4f95393dc4f7fa4b275436bbc4385611cb2e Mon Sep 17 00:00:00 2001 From: Tony Xu Date: Tue, 27 Sep 2016 01:54:27 +0800 Subject: [PATCH] Modify use of assertTrue(A in B) Developers should use assertIn(A, B) instead of assertTrue(A in B ). Change-Id: I8c1165a6342459d554d21f1251c301b503a97f14 --- karbor/tests/unit/operationengine/test_manager.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/karbor/tests/unit/operationengine/test_manager.py b/karbor/tests/unit/operationengine/test_manager.py index cf7f8265..2e0e244c 100644 --- a/karbor/tests/unit/operationengine/test_manager.py +++ b/karbor/tests/unit/operationengine/test_manager.py @@ -115,8 +115,8 @@ class OperationEngineManagerTestCase(base.TestCase): trigger_id = "trigger" self.manager.resume_scheduled_operation(self.ctxt, op_id, trigger_id) - self.assertTrue(op_id in ( - self.manager._trigger_manager._trigger[trigger_id])) + self.assertIn(op_id, + self.manager._trigger_manager._trigger[trigger_id]) self.manager.resume_scheduled_operation(self.ctxt, op_id, trigger_id) self.assertTrue(1 == len(