Browse Source

[1790] Use appropriate boolean assert calls

Mukund Sivaraman 13 years ago
parent
commit
b685e7c4e4
1 changed files with 28 additions and 28 deletions
  1. 28 28
      src/bin/xfrin/tests/xfrin_test.py

+ 28 - 28
src/bin/xfrin/tests/xfrin_test.py

@@ -2869,21 +2869,21 @@ class TestXfrinProcess(unittest.TestCase):
         self.assertEqual(len(transfers), self.__created_connections)
         self.assertEqual(len(transfers), self.__created_connections)
         self.assertEqual([published], self.__published)
         self.assertEqual([published], self.__published)
         if published == XFRIN_OK:
         if published == XFRIN_OK:
-            self.assertEqual(True, self._module_cc.get_called)
-            self.assertEqual(True, self._module_cc.get_called_correctly)
+            self.assertTrue(self._module_cc.get_called)
+            self.assertTrue(self._module_cc.get_called_correctly)
         else:
         else:
-            self.assertEqual(False, self._module_cc.get_called)
-            self.assertEqual(False, self._module_cc.get_called_correctly)
+            self.assertFalse(self._module_cc.get_called)
+            self.assertFalse(self._module_cc.get_called_correctly)
 
 
     def test_ixfr_ok(self):
     def test_ixfr_ok(self):
         """
         """
         Everything OK the first time, over IXFR.
         Everything OK the first time, over IXFR.
         """
         """
         self.__do_test([XFRIN_OK], [RRType.IXFR()], RRType.IXFR())
         self.__do_test([XFRIN_OK], [RRType.IXFR()], RRType.IXFR())
-        self.assertEqual(False, self._send_cc_session.send_called)
-        self.assertEqual(False, self._send_cc_session.send_called_correctly)
-        self.assertEqual(False, self._send_cc_session.recv_called)
-        self.assertEqual(False, self._send_cc_session.recv_called_correctly)
+        self.assertFalse(self._send_cc_session.send_called)
+        self.assertFalse(self._send_cc_session.send_called_correctly)
+        self.assertFalse(self._send_cc_session.recv_called)
+        self.assertFalse(self._send_cc_session.recv_called_correctly)
 
 
     def test_axfr_ok(self):
     def test_axfr_ok(self):
         """
         """
@@ -2926,10 +2926,10 @@ class TestXfrinProcess(unittest.TestCase):
                                               'file': 'data/inmem-xfrin.sqlite3'}],
                                               'file': 'data/inmem-xfrin.sqlite3'}],
                                    'type': 'memory', 'class': 'IN'}]
                                    'type': 'memory', 'class': 'IN'}]
         self.__do_test([XFRIN_OK], [RRType.IXFR()], RRType.IXFR())
         self.__do_test([XFRIN_OK], [RRType.IXFR()], RRType.IXFR())
-        self.assertEqual(True, self._send_cc_session.send_called)
-        self.assertEqual(True, self._send_cc_session.send_called_correctly)
-        self.assertEqual(True, self._send_cc_session.recv_called)
-        self.assertEqual(True, self._send_cc_session.recv_called_correctly)
+        self.assertTrue(self._send_cc_session.send_called)
+        self.assertTrue(self._send_cc_session.send_called_correctly)
+        self.assertTrue(self._send_cc_session.recv_called)
+        self.assertTrue(self._send_cc_session.recv_called_correctly)
 
 
     def test_inmem_not_memory(self):
     def test_inmem_not_memory(self):
         """
         """
@@ -2940,10 +2940,10 @@ class TestXfrinProcess(unittest.TestCase):
                                               'file': 'data/inmem-xfrin.sqlite3'}],
                                               'file': 'data/inmem-xfrin.sqlite3'}],
                                    'type': 'punched-card', 'class': 'IN'}]
                                    'type': 'punched-card', 'class': 'IN'}]
         self.__do_test([XFRIN_OK], [RRType.IXFR()], RRType.IXFR())
         self.__do_test([XFRIN_OK], [RRType.IXFR()], RRType.IXFR())
-        self.assertEqual(False, self._send_cc_session.send_called)
-        self.assertEqual(False, self._send_cc_session.send_called_correctly)
-        self.assertEqual(False, self._send_cc_session.recv_called)
-        self.assertEqual(False, self._send_cc_session.recv_called_correctly)
+        self.assertFalse(self._send_cc_session.send_called)
+        self.assertFalse(self._send_cc_session.send_called_correctly)
+        self.assertFalse(self._send_cc_session.recv_called)
+        self.assertFalse(self._send_cc_session.recv_called_correctly)
 
 
     def test_inmem_not_sqlite3(self):
     def test_inmem_not_sqlite3(self):
         """
         """
@@ -2955,10 +2955,10 @@ class TestXfrinProcess(unittest.TestCase):
                                               'file': 'data/inmem-xfrin.sqlite3'}],
                                               'file': 'data/inmem-xfrin.sqlite3'}],
                                    'type': 'memory', 'class': 'IN'}]
                                    'type': 'memory', 'class': 'IN'}]
         self.__do_test([XFRIN_OK], [RRType.IXFR()], RRType.IXFR())
         self.__do_test([XFRIN_OK], [RRType.IXFR()], RRType.IXFR())
-        self.assertEqual(False, self._send_cc_session.send_called)
-        self.assertEqual(False, self._send_cc_session.send_called_correctly)
-        self.assertEqual(False, self._send_cc_session.recv_called)
-        self.assertEqual(False, self._send_cc_session.recv_called_correctly)
+        self.assertFalse(self._send_cc_session.send_called)
+        self.assertFalse(self._send_cc_session.send_called_correctly)
+        self.assertFalse(self._send_cc_session.recv_called)
+        self.assertFalse(self._send_cc_session.recv_called_correctly)
 
 
     def test_inmem_not_of_same_class(self):
     def test_inmem_not_of_same_class(self):
         """
         """
@@ -2970,10 +2970,10 @@ class TestXfrinProcess(unittest.TestCase):
                                               'file': 'data/inmem-xfrin.sqlite3'}],
                                               'file': 'data/inmem-xfrin.sqlite3'}],
                                    'type': 'memory', 'class': 'XX'}]
                                    'type': 'memory', 'class': 'XX'}]
         self.__do_test([XFRIN_OK], [RRType.IXFR()], RRType.IXFR())
         self.__do_test([XFRIN_OK], [RRType.IXFR()], RRType.IXFR())
-        self.assertEqual(False, self._send_cc_session.send_called)
-        self.assertEqual(False, self._send_cc_session.send_called_correctly)
-        self.assertEqual(False, self._send_cc_session.recv_called)
-        self.assertEqual(False, self._send_cc_session.recv_called_correctly)
+        self.assertFalse(self._send_cc_session.send_called)
+        self.assertFalse(self._send_cc_session.send_called_correctly)
+        self.assertFalse(self._send_cc_session.recv_called)
+        self.assertFalse(self._send_cc_session.recv_called_correctly)
 
 
     def test_inmem_not_present(self):
     def test_inmem_not_present(self):
         """
         """
@@ -2984,10 +2984,10 @@ class TestXfrinProcess(unittest.TestCase):
                                               'file': 'data/inmem-xfrin.sqlite3'}],
                                               'file': 'data/inmem-xfrin.sqlite3'}],
                                    'type': 'memory', 'class': 'IN'}]
                                    'type': 'memory', 'class': 'IN'}]
         self.__do_test([XFRIN_OK], [RRType.IXFR()], RRType.IXFR())
         self.__do_test([XFRIN_OK], [RRType.IXFR()], RRType.IXFR())
-        self.assertEqual(False, self._send_cc_session.send_called)
-        self.assertEqual(False, self._send_cc_session.send_called_correctly)
-        self.assertEqual(False, self._send_cc_session.recv_called)
-        self.assertEqual(False, self._send_cc_session.recv_called_correctly)
+        self.assertFalse(self._send_cc_session.send_called)
+        self.assertFalse(self._send_cc_session.send_called_correctly)
+        self.assertFalse(self._send_cc_session.recv_called)
+        self.assertFalse(self._send_cc_session.recv_called_correctly)
 
 
 class TestFormatting(unittest.TestCase):
 class TestFormatting(unittest.TestCase):
     # If the formatting functions are moved to a more general library
     # If the formatting functions are moved to a more general library