tests: rename variables in revlog index parse test for clarity
authorAugie Fackler <augie@google.com>
Thu, 09 Aug 2018 13:08:29 -0400
changeset 39031 ee0720e82257
parent 39030 94cff858b4e5
child 39032 6076053589b9
tests: rename variables in revlog index parse test for clarity Now it's unambiguous which one is the expected value. c_res_{1,2} was also misleading a bit because in --pure mode we're testing the old slow Python version against the modern optimized Python version. Differential Revision: https://phab.mercurial-scm.org/D4180
tests/test-parseindex2.py
--- a/tests/test-parseindex2.py	Thu Aug 09 13:06:50 2018 -0400
+++ b/tests/test-parseindex2.py	Thu Aug 09 13:08:29 2018 -0400
@@ -184,13 +184,13 @@
         # original Python implementation of parseindex, both with and
         # without inlined data.
 
-        py_res_1 = py_parseindex(data_inlined, True)
-        c_res_1 = parse_index2(data_inlined, True)
-        self.assertEqual(py_res_1, c_res_1) # inline data
+        want = py_parseindex(data_inlined, True)
+        got = parse_index2(data_inlined, True)
+        self.assertEqual(want, got) # inline data
 
-        py_res_2 = py_parseindex(data_non_inlined, False)
-        c_res_2 = parse_index2(data_non_inlined, False)
-        self.assertEqual(py_res_2, c_res_2) # no inline data
+        want = py_parseindex(data_non_inlined, False)
+        got = parse_index2(data_non_inlined, False)
+        self.assertEqual(want, got) # no inline data
 
         ix = parsers.parse_index2(data_inlined, True)[0]
         for i, r in enumerate(ix):