changeset d62ef9d3c1c3 in trytond:default
details: https://hg.tryton.org/trytond?cmd=changeset;node=d62ef9d3c1c3
description:
        Replace assertTrue and assert False by assertEqual

        When we must test that the actual value is True or False, we must use
        assertEqual.

        issue9093
        review256651002
diffstat:

 trytond/tests/test_tools.py |  67 ++++++++++++++++++++++++--------------------
 1 files changed, 36 insertions(+), 31 deletions(-)

diffs (132 lines):

diff -r bb494c884ebd -r d62ef9d3c1c3 trytond/tests/test_tools.py
--- a/trytond/tests/test_tools.py       Fri Mar 06 23:14:24 2020 +0100
+++ b/trytond/tests/test_tools.py       Fri Mar 06 23:20:35 2020 +0100
@@ -294,9 +294,9 @@
         self.assertEqual(domain_inversion(domain, 'x'), [['x', '=', 3]])
 
         domain = []
-        self.assertTrue(domain_inversion(domain, 'x'))
-        self.assertTrue(domain_inversion(domain, 'x', {'x': 5}))
-        self.assertTrue(domain_inversion(domain, 'z', {'x': 7}))
+        self.assertEqual(domain_inversion(domain, 'x'), True)
+        self.assertEqual(domain_inversion(domain, 'x', {'x': 5}), True)
+        self.assertEqual(domain_inversion(domain, 'z', {'x': 7}), True)
 
         domain = [['x.id', '>', 5]]
         self.assertEqual(domain_inversion(domain, 'x'), [['x.id', '>', 5]])
@@ -304,13 +304,13 @@
     def test_and_inversion(self):
         domain = [['x', '=', 3], ['y', '>', 5]]
         self.assertEqual(domain_inversion(domain, 'x'), [['x', '=', 3]])
-        self.assertFalse(domain_inversion(domain, 'x', {'y': 4}))
+        self.assertEqual(domain_inversion(domain, 'x', {'y': 4}), False)
         self.assertEqual(
             domain_inversion(domain, 'x', {'y': 6}), [['x', '=', 3]])
 
         domain = [['x', '=', 3], ['y', '=', 5]]
-        self.assertTrue(domain_inversion(domain, 'z'))
-        self.assertTrue(domain_inversion(domain, 'z', {'x': 2, 'y': 7}))
+        self.assertEqual(domain_inversion(domain, 'z'), True)
+        self.assertEqual(domain_inversion(domain, 'z', {'x': 2, 'y': 7}), True)
         self.assertEqual(
             domain_inversion(domain, 'x', {'y': None}), [['x', '=', 3]])
 
@@ -327,23 +327,24 @@
         self.assertEqual(
             domain_inversion(domain, 'x', {'y': 4, 'z': 'ab'}),
             [['x', '=', 3]])
-        self.assertTrue(domain_inversion(domain, 'x', {'y': 7}))
-        self.assertTrue(domain_inversion(domain, 'x', {'y': 7, 'z': 'b'}))
-        self.assertTrue(domain_inversion(domain, 'x', {'z': 'abc'}))
-        self.assertTrue(domain_inversion(domain, 'x', {'y': 4, 'z': 'abc'}))
+        self.assertEqual(domain_inversion(domain, 'x', {'y': 7}), True)
+        self.assertEqual(
+            domain_inversion(domain, 'x', {'y': 7, 'z': 'b'}), True)
+        self.assertEqual(domain_inversion(domain, 'x', {'z': 'abc'}), True)
+        self.assertEqual(
+            domain_inversion(domain, 'x', {'y': 4, 'z': 'abc'}), True)
 
         domain = ['OR', ['x', '=', 3], ['y', '=', 5]]
         self.assertEqual(
             domain_inversion(domain, 'x', {'y': None}), [['x', '=', 3]])
 
         domain = ['OR', ['x', '=', 3], ['y', '>', 5]]
-        self.assertTrue(domain_inversion(domain, 'z'))
+        self.assertEqual(domain_inversion(domain, 'z'), True)
 
         domain = ['OR', ['x.id', '>', 5], ['y', '<', 3]]
         self.assertEqual(domain_inversion(domain, 'y'), True)
-        self.assertEqual(
-            domain_inversion(domain, 'y', {'z': 4}), True)
-        self.assertTrue(domain_inversion(domain, 'y', {'x': 3}))
+        self.assertEqual(domain_inversion(domain, 'y', {'z': 4}), True)
+        self.assertEqual(domain_inversion(domain, 'y', {'x': 3}), True)
 
         domain = ['OR', ['length', '>', 5], ['language.code', '=', 'de_DE']]
         self.assertEqual(
@@ -353,19 +354,20 @@
     def test_orand_inversion(self):
         domain = ['OR', [['x', '=', 3], ['y', '>', 5], ['z', '=', 'abc']],
             [['x', '=', 4]], [['y', '>', 6]]]
-        self.assertTrue(domain_inversion(domain, 'x'))
+        self.assertEqual(domain_inversion(domain, 'x'), True)
+        self.assertEqual(domain_inversion(domain, 'x', {'y': 4}), True)
         self.assertEqual(
-            domain_inversion(domain, 'x', {'y': 4}), True)
-        self.assertTrue(domain_inversion(domain, 'x', {'z': 'abc', 'y': 7}))
-        self.assertTrue(domain_inversion(domain, 'x', {'y': 7}))
-        self.assertTrue(domain_inversion(domain, 'x', {'z': 'ab'}))
+            domain_inversion(domain, 'x', {'z': 'abc', 'y': 7}), True)
+        self.assertEqual(domain_inversion(domain, 'x', {'y': 7}), True)
+        self.assertEqual(domain_inversion(domain, 'x', {'z': 'ab'}), True)
 
     def test_andor_inversion(self):
         domain = [['OR', ['x', '=', 4], ['y', '>', 6]], ['z', '=', 3]]
         self.assertEqual(domain_inversion(domain, 'z'), [['z', '=', 3]])
         self.assertEqual(
             domain_inversion(domain, 'z', {'x': 5}), [['z', '=', 3]])
-        self.assertFalse(domain_inversion(domain, 'z', {'x': 5, 'y': 5}))
+        self.assertEqual(
+            domain_inversion(domain, 'z', {'x': 5, 'y': 5}), False)
         self.assertEqual(
             domain_inversion(domain, 'z', {'x': 5, 'y': 7}), [['z', '=', 3]])
 
@@ -374,7 +376,7 @@
         self.assertEqual(domain_inversion(domain, 'z'), [['z', '=', 3]])
         self.assertEqual(
             domain_inversion(domain, 'z', {'x': 5}), [['z', '=', 3]])
-        self.assertFalse(domain_inversion(domain, 'z', {'y': 5}))
+        self.assertEqual(domain_inversion(domain, 'z', {'y': 5}), False)
         self.assertEqual(
             domain_inversion(domain, 'z', {'x': 4, 'y': 7}), [['z', '=', 3]])
 
@@ -387,16 +389,19 @@
         domain = ['OR', ['OR', ['x', '=', 3], ['y', '>', 5]],
             ['OR', ['x', '=', 2], ['z', '=', 'abc']],
             ['OR', ['y', '=', 8], ['z', '=', 'y']]]
-        self.assertTrue(domain_inversion(domain, 'x'))
-        self.assertTrue(domain_inversion(domain, 'x', {'y': 4}))
-        self.assertTrue(domain_inversion(domain, 'x', {'z': 'ab'}))
-        self.assertTrue(domain_inversion(domain, 'x', {'y': 7}))
-        self.assertTrue(domain_inversion(domain, 'x', {'z': 'abc'}))
-        self.assertTrue(domain_inversion(domain, 'x', {'z': 'y'}))
-        self.assertTrue(domain_inversion(domain, 'x', {'y': 8}))
-        self.assertTrue(domain_inversion(domain, 'x', {'y': 8, 'z': 'b'}))
-        self.assertTrue(domain_inversion(domain, 'x', {'y': 4, 'z': 'y'}))
-        self.assertTrue(domain_inversion(domain, 'x', {'y': 7, 'z': 'abc'}))
+        self.assertEqual(domain_inversion(domain, 'x'), True)
+        self.assertEqual(domain_inversion(domain, 'x', {'y': 4}), True)
+        self.assertEqual(domain_inversion(domain, 'x', {'z': 'ab'}), True)
+        self.assertEqual(domain_inversion(domain, 'x', {'y': 7}), True)
+        self.assertEqual(domain_inversion(domain, 'x', {'z': 'abc'}), True)
+        self.assertEqual(domain_inversion(domain, 'x', {'z': 'y'}), True)
+        self.assertEqual(domain_inversion(domain, 'x', {'y': 8}), True)
+        self.assertEqual(
+            domain_inversion(domain, 'x', {'y': 8, 'z': 'b'}), True)
+        self.assertEqual(
+            domain_inversion(domain, 'x', {'y': 4, 'z': 'y'}), True)
+        self.assertEqual(
+            domain_inversion(domain, 'x', {'y': 7, 'z': 'abc'}), True)
         self.assertEqual(
             domain_inversion(domain, 'x', {'y': 4, 'z': 'b'}),
             ['OR', [['x', '=', 3]], [['x', '=', 2]]])

Reply via email to