commit 855d1c0daae2723ea2e16693e566fd9e48383b1f
author Bertrand Lallau <bertrand.lallau@thalesgroup.com> Mon Oct 26 16:37:37 2015 +0100
committer Bertrand Lallau <bertrand.lallau@gmail.com> Mon Oct 26 22:45:25 2015 +0100
tree 4882f9469aa15f7cc3d606eee5331d7bf1f4685c
parent 3eab294d7ffcf4c0d7ffca3d7f0431382342766f

Use assertTrue/False instead of assertEqual(T/F)

The usage of assertEqual(True/False, ***) should be changed to a more
meaningful format of assertTrue/False(***).

Change-Id: I685f7bb0b4669d8813ebbb796b6014ad44d7ff0c
Closes-Bug: #1510001

functional/test_preview.py

diff --git a/functional/test_preview.py b/functional/test_preview.py
index 19da01b..b927a25 100644
--- a/functional/test_preview.py
+++ b/functional/test_preview.py
@@ -103,9 +103,9 @@
                              res['resource_type'])
 
             # common properties
-            self.assertEqual(False, res['properties']['fail'])
+            self.assertFalse(res['properties']['fail'])
             self.assertEqual(0, res['properties']['wait_secs'])
-            self.assertEqual(False, res['properties']['update_replace'])
+            self.assertFalse(res['properties']['update_replace'])
 
             if res['resource_name'] == 'one':
                 self.assertEqual('fred', res['properties']['value'])
@@ -180,9 +180,9 @@
         self.assertEqual('OS::Heat::TestResource',
                          res['resource_type'])
 
-        self.assertEqual(False, res['properties']['fail'])
+        self.assertFalse(res['properties']['fail'])
         self.assertEqual(0, res['properties']['wait_secs'])
-        self.assertEqual(False, res['properties']['update_replace'])
+        self.assertFalse(res['properties']['update_replace'])
 
         self.assertEqual('abc', res['properties']['value'])
         self.assertEqual([], res['required_by'])