changeset 38991:22216c4607bb

tests: move chunks of test-parseindex2.py to use unittest properly This doesn't touch the version-detection tests yet, because those are more involved. Differential Revision: https://phab.mercurial-scm.org/D4178
author Augie Fackler <augie@google.com>
date Thu, 09 Aug 2018 13:05:40 -0400
parents 087a755310c3
children 94cff858b4e5
files tests/test-parseindex2.py
diffstat 1 files changed, 6 insertions(+), 13 deletions(-) [+]
line wrap: on
line diff
--- a/tests/test-parseindex2.py	Thu Aug 09 12:59:23 2018 -0400
+++ b/tests/test-parseindex2.py	Thu Aug 09 13:05:40 2018 -0400
@@ -176,12 +176,8 @@
 
     def testbadargs(self):
         # Check that parse_index2() raises TypeError on bad arguments.
-        try:
+        with self.assertRaises(TypeError):
             parse_index2(0, True)
-        except TypeError:
-            pass
-        else:
-            print("Expected to get TypeError.")
 
     def testparseindexfile(self):
         # Check parsers.parse_index2() on an index file against the
@@ -194,20 +190,17 @@
         py_res_2 = py_parseindex(data_non_inlined, False)
         c_res_2 = parse_index2(data_non_inlined, False)
 
-        if py_res_1 != c_res_1:
-            print("Parse index result (with inlined data) differs!")
-
-        if py_res_2 != c_res_2:
-            print("Parse index result (no inlined data) differs!")
+        self.assertEqual(py_res_1, c_res_1) # inline data
+        self.assertEqual(py_res_2, c_res_2) # no inline data
 
         ix = parsers.parse_index2(data_inlined, True)[0]
         for i, r in enumerate(ix):
             if r[7] == nullid:
                 i = -1
             try:
-                if ix[r[7]] != i:
-                    print('Reverse lookup inconsistent for %r'
-                        % r[7].encode('hex'))
+                self.assertEqual(
+                    ix[r[7]], i,
+                    'Reverse lookup inconsistent for %r' % r[7].encode('hex'))
             except TypeError:
                 # pure version doesn't support this
                 break