From 749ea50598823cea4faac313ba81c6fda0f16070 Mon Sep 17 00:00:00 2001 From: "Swapnil Kulkarni (coolsvap)" Date: Thu, 7 Jan 2016 12:13:10 +0530 Subject: [PATCH] Use assertTrue/False instead of assertEqual(T/F) The usage of assertEqual(True/False, ***) should be changed to a meaningful format of assertTrue/False(***). Change-Id: I978e6c171c2a9adeb0f9b8edb15ce81c4d7c4c7d Closes-Bug:#1512207 --- oslo_cache/tests/test_cache_backend_mongo.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/oslo_cache/tests/test_cache_backend_mongo.py b/oslo_cache/tests/test_cache_backend_mongo.py index fcedd081..a8092a7a 100644 --- a/oslo_cache/tests/test_cache_backend_mongo.py +++ b/oslo_cache/tests/test_cache_backend_mongo.py @@ -394,9 +394,9 @@ class MongoCache(test_cache.BaseTestCase): self.assertEqual('cache', region.backend.api.cache_collection) self.assertEqual('test_user', region.backend.api.username) self.assertEqual('test_password', region.backend.api.password) - self.assertEqual(True, region.backend.api.use_replica) + self.assertTrue(region.backend.api.use_replica) self.assertEqual('my_replica', region.backend.api.replicaset_name) - self.assertEqual(False, region.backend.api.conn_kwargs['ssl']) + self.assertFalse(region.backend.api.conn_kwargs['ssl']) self.assertEqual(60, region.backend.api.ttl_seconds) def test_multiple_region_cache_configuration(self): @@ -711,8 +711,8 @@ class MongoCache(test_cache.BaseTestCase): # There is no proxy so can access MongoCacheBackend directly api_methargs = region.backend.api.meth_kwargs self.assertEqual(30000, api_methargs['wtimeout']) - self.assertEqual(True, api_methargs['j']) - self.assertEqual(True, api_methargs['continue_on_error']) + self.assertTrue(api_methargs['j']) + self.assertTrue(api_methargs['continue_on_error']) self.assertEqual(60, api_methargs['secondary_acceptable_latency_ms']) random_key = uuid.uuid4().hex