Replace assertEqual(True/False, expr) with assertTrue/assertFalse

In some cases, If the result of expr is a boolen value, we shoud
use assertTrue/assertFalse to instead. Because it is clear and simple.

Change-Id: I53b345fc3915a7b0e737e9dd4d58fe09c746d61c

6 files changed, 38 insertions, 131 deletions

diff --git a/openstackclient/tests/functional/network/v2/test_address_scope.py b/openstackclient/tests/functional/network/v2/test_address_scope.py
index ebd2ba86..8a99ec5e 100644
--- a/openstackclient/tests/functional/network/v2/test_address_scope.py
+++ b/openstackclient/tests/functional/network/v2/test_address_scope.py
@@ -42,10 +42,7 @@ class AddressScopeTests(common.NetworkTests):
             cmd_output['name'],
         )
         # Check the default values
-        self.assertEqual(
-            False,
-            cmd_output['shared'],
-        )
+        self.assertFalse(cmd_output['shared'])
 
         name2 = uuid.uuid4().hex
         cmd_output = json.loads(self.openstack(
@@ -80,10 +77,7 @@ class AddressScopeTests(common.NetworkTests):
             4,
             cmd_output['ip_version'],
         )
-        self.assertEqual(
-            True,
-            cmd_output['shared'],
-        )
+        self.assertTrue(cmd_output['shared'])
 
         name2 = uuid.uuid4().hex
         cmd_output = json.loads(self.openstack(
@@ -101,10 +95,7 @@ class AddressScopeTests(common.NetworkTests):
             6,
             cmd_output['ip_version'],
         )
-        self.assertEqual(
-            False,
-            cmd_output['shared'],
-        )
+        self.assertFalse(cmd_output['shared'])
 
         # Test list
         cmd_output = json.loads(self.openstack(
@@ -149,10 +140,7 @@ class AddressScopeTests(common.NetworkTests):
             4,
             cmd_output['ip_version'],
         )
-        self.assertEqual(
-            False,
-            cmd_output['shared'],
-        )
+        self.assertFalse(cmd_output['shared'])
 
         raw_output = self.openstack(
             'address scope set ' +
@@ -174,7 +162,4 @@ class AddressScopeTests(common.NetworkTests):
             4,
             cmd_output['ip_version'],
         )
-        self.assertEqual(
-            True,
-            cmd_output['shared'],
-        )
+        self.assertTrue(cmd_output['shared'])
diff --git a/openstackclient/tests/functional/network/v2/test_network.py b/openstackclient/tests/functional/network/v2/test_network.py
index 9cef135f..1a744969 100644
--- a/openstackclient/tests/functional/network/v2/test_network.py
+++ b/openstackclient/tests/functional/network/v2/test_network.py
@@ -70,8 +70,7 @@ class NetworkTests(common.NetworkTagTests):
             '1.2.4.0/28',
             cmd_output["cidr"],
         )
-        self.assertEqual(
-            True,
+        self.assertTrue(
             cmd_output["share_address"],
         )
 
@@ -124,8 +123,7 @@ class NetworkTests(common.NetworkTagTests):
             'UP',
             cmd_output["admin_state_up"],
         )
-        self.assertEqual(
-            False,
+        self.assertFalse(
             cmd_output["shared"],
         )
         self.assertEqual(
@@ -236,21 +234,14 @@ class NetworkTests(common.NetworkTagTests):
                 'UP',
                 cmd_output["admin_state_up"],
             )
-            self.assertEqual(
-                False,
-                cmd_output["shared"],
-            )
+            self.assertFalse(cmd_output["shared"])
             self.assertEqual(
                 'Internal',
                 cmd_output["router:external"],
             )
-            self.assertEqual(
-                False,
-                cmd_output["is_default"],
-            )
-            self.assertEqual(
-                True,
-                cmd_output["port_security_enabled"],
+            self.assertFalse(cmd_output["is_default"])
+            self.assertTrue(
+                cmd_output["port_security_enabled"]
             )
         else:
             self.assertEqual(
@@ -278,27 +269,15 @@ class NetworkTests(common.NetworkTagTests):
                 'DOWN',
                 cmd_output["admin_state_up"],
             )
-            self.assertEqual(
-                True,
-                cmd_output["shared"],
-            )
-            self.assertEqual(
-                False,
-                cmd_output["is_default"],
-            )
-            self.assertEqual(
-                True,
-                cmd_output["port_security_enabled"],
-            )
+            self.assertTrue(cmd_output["shared"])
+            self.assertFalse(cmd_output["is_default"])
+            self.assertTrue(cmd_output["port_security_enabled"])
         else:
             self.assertEqual(
                 '4.5.6.0/28',
                 cmd_output["cidr"],
             )
-            self.assertEqual(
-                True,
-                cmd_output["share_address"],
-            )
+            self.assertTrue(cmd_output["share_address"])
 
         # Test list
         cmd_output = json.loads(self.openstack(
@@ -422,22 +401,15 @@ class NetworkTests(common.NetworkTagTests):
             'UP',
             cmd_output["admin_state_up"],
         )
-        self.assertEqual(
-            False,
-            cmd_output["shared"],
-        )
+        self.assertFalse(cmd_output["shared"])
         self.assertEqual(
             'Internal',
             cmd_output["router:external"],
         )
 
-        self.assertEqual(
-            False,
-            cmd_output["is_default"],
-        )
-        self.assertEqual(
-            True,
-            cmd_output["port_security_enabled"],
+        self.assertFalse(cmd_output["is_default"])
+        self.assertTrue(
+            cmd_output["port_security_enabled"]
         )
 
         raw_output = self.openstack(
@@ -463,19 +435,12 @@ class NetworkTests(common.NetworkTagTests):
             'DOWN',
             cmd_output["admin_state_up"],
         )
-        self.assertEqual(
-            True,
-            cmd_output["shared"],
-        )
+        self.assertTrue(cmd_output["shared"])
         self.assertEqual(
             'External',
             cmd_output["router:external"],
         )
-        self.assertEqual(
-            False,
-            cmd_output["is_default"],
-        )
-        self.assertEqual(
-            False,
-            cmd_output["port_security_enabled"],
+        self.assertFalse(cmd_output["is_default"])
+        self.assertFalse(
+            cmd_output["port_security_enabled"]
         )
diff --git a/openstackclient/tests/functional/network/v2/test_network_flavor.py b/openstackclient/tests/functional/network/v2/test_network_flavor.py
index ba3de2cd..cf68a096 100644
--- a/openstackclient/tests/functional/network/v2/test_network_flavor.py
+++ b/openstackclient/tests/functional/network/v2/test_network_flavor.py
@@ -87,10 +87,7 @@ class NetworkFlavorTests(common.NetworkTests):
             name1,
             cmd_output['name'],
         )
-        self.assertEqual(
-            True,
-            cmd_output['enabled'],
-        )
+        self.assertTrue(cmd_output['enabled'])
         self.assertEqual(
             'testdescription',
             cmd_output['description'],
@@ -105,10 +102,7 @@ class NetworkFlavorTests(common.NetworkTests):
             name2,
             cmd_output['name'],
         )
-        self.assertEqual(
-            False,
-            cmd_output['enabled'],
-        )
+        self.assertFalse(cmd_output['enabled'])
         self.assertEqual(
             'testdescription1',
             cmd_output['description'],
diff --git a/openstackclient/tests/functional/network/v2/test_network_flavor_profile.py b/openstackclient/tests/functional/network/v2/test_network_flavor_profile.py
index 2207c847..5b5ec926 100644
--- a/openstackclient/tests/functional/network/v2/test_network_flavor_profile.py
+++ b/openstackclient/tests/functional/network/v2/test_network_flavor_profile.py
@@ -35,10 +35,7 @@ class NetworkFlavorProfileTests(common.NetworkTests):
         ))
         ID = json_output.get('id')
         self.assertIsNotNone(ID)
-        self.assertEqual(
-            True,
-            json_output.get('enabled'),
-        )
+        self.assertTrue(json_output.get('enabled'))
         self.assertEqual(
             'fakedescription',
             json_output.get('description'),
@@ -61,10 +58,7 @@ class NetworkFlavorProfileTests(common.NetworkTests):
         ))
         ID1 = json_output.get('id')
         self.assertIsNotNone(ID1)
-        self.assertEqual(
-            True,
-            json_output.get('enabled'),
-        )
+        self.assertTrue(json_output.get('enabled'))
         self.assertEqual(
             'fakedescription',
             json_output.get('description'),
@@ -82,10 +76,7 @@ class NetworkFlavorProfileTests(common.NetworkTests):
         ))
         ID2 = json_output.get('id')
         self.assertIsNotNone(ID2)
-        self.assertEqual(
-            False,
-            json_output.get('enabled'),
-        )
+        self.assertFalse(json_output.get('enabled'))
         self.assertEqual(
             'fakedescription',
             json_output.get('description'),
@@ -120,10 +111,7 @@ class NetworkFlavorProfileTests(common.NetworkTests):
         ))
         ID = json_output_1.get('id')
         self.assertIsNotNone(ID)
-        self.assertEqual(
-            True,
-            json_output_1.get('enabled'),
-        )
+        self.assertTrue(json_output_1.get('enabled'))
         self.assertEqual(
             'fakedescription',
             json_output_1.get('description'),
@@ -138,10 +126,7 @@ class NetworkFlavorProfileTests(common.NetworkTests):
         json_output = json.loads(self.openstack(
             'network flavor profile show -f json ' + ID
         ))
-        self.assertEqual(
-            False,
-            json_output.get('enabled'),
-        )
+        self.assertFalse(json_output.get('enabled'))
         self.assertEqual(
             'fakedescription',
             json_output.get('description'),
@@ -171,10 +156,7 @@ class NetworkFlavorProfileTests(common.NetworkTests):
             ID,
             json_output["id"],
         )
-        self.assertEqual(
-            True,
-            json_output["enabled"],
-        )
+        self.assertTrue(json_output["enabled"])
         self.assertEqual(
             'fakedescription',
             json_output["description"],
diff --git a/openstackclient/tests/functional/network/v2/test_network_meter.py b/openstackclient/tests/functional/network/v2/test_network_meter.py
index 7f6da28d..0a8b89ca 100644
--- a/openstackclient/tests/functional/network/v2/test_network_meter.py
+++ b/openstackclient/tests/functional/network/v2/test_network_meter.py
@@ -48,10 +48,7 @@ class TestMeter(common.NetworkTests):
             json_output.get('name'),
         )
         # Check if default shared values
-        self.assertEqual(
-            False,
-            json_output.get('shared'),
-        )
+        self.assertFalse(json_output.get('shared'))
         self.assertEqual(
             'fakedescription',
             json_output.get('description'),
@@ -67,10 +64,7 @@ class TestMeter(common.NetworkTests):
             json_output_2.get('name'),
         )
         # Check if default shared values
-        self.assertEqual(
-            False,
-            json_output_2.get('shared'),
-        )
+        self.assertFalse(json_output_2.get('shared'))
         self.assertEqual(
             'fakedescription',
             json_output_2.get('description'),
@@ -99,10 +93,7 @@ class TestMeter(common.NetworkTests):
             'Test1',
             json_output.get('description'),
         )
-        self.assertEqual(
-            True,
-            json_output.get('shared'),
-        )
+        self.assertTrue(json_output.get('shared'))
 
         name2 = uuid.uuid4().hex
         json_output_2 = json.loads(self.openstack(
@@ -117,8 +108,7 @@ class TestMeter(common.NetworkTests):
             'Test2',
             json_output_2.get('description'),
         )
-        self.assertEqual(
-            False,
+        self.assertFalse(
             json_output_2.get('shared'),
         )
 
@@ -143,10 +133,7 @@ class TestMeter(common.NetworkTests):
         json_output = json.loads(self.openstack(
             'network meter show -f json ' + meter_id
         ))
-        self.assertEqual(
-            False,
-            json_output.get('shared'),
-        )
+        self.assertFalse(json_output.get('shared'))
         self.assertEqual(
             'fakedescription',
             json_output.get('description'),
@@ -164,10 +151,7 @@ class TestMeter(common.NetworkTests):
             meter_id,
             json_output.get('id'),
         )
-        self.assertEqual(
-            False,
-            json_output.get('shared'),
-        )
+        self.assertFalse(json_output.get('shared'))
         self.assertEqual(
             'fakedescription',
             json_output.get('description'),
diff --git a/openstackclient/tests/functional/network/v2/test_router.py b/openstackclient/tests/functional/network/v2/test_router.py
index 95c5a96f..9d5beff0 100644
--- a/openstackclient/tests/functional/network/v2/test_router.py
+++ b/openstackclient/tests/functional/network/v2/test_router.py
@@ -247,10 +247,7 @@ class RouterTests(common.NetworkTagTests):
             'router show -f json ' +
             new_name
         ))
-        self.assertEqual(
-            True,
-            cmd_output["distributed"],
-        )
+        self.assertTrue(cmd_output["distributed"])
         self.assertIsNotNone(cmd_output["external_gateway_info"])
 
         # Test unset