[PATCH 2 of 3 hglib] client: change return value of commit() to (rev, node)

[Idan Kamara]

# HG changeset patch
# User Idan Kamara <idankk86 at gmail.com>
# Date 1312995548 -10800
# Node ID 21190e06aa2d4744356f42b3f072d039a5d3bc50
# Parent  3a1faa3b1fc92bc3031421da109c6016d0bcbfe0
client: change return value of commit() to (rev, node)

this is the original output hg emits with commit --debug

diff -r 3a1faa3b1fc9 -r 21190e06aa2d hglib/client.py
--- a/hglib/client.py	Wed Aug 10 19:59:03 2011 +0300
+++ b/hglib/client.py	Wed Aug 10 19:59:08 2011 +0300
@@ -186,7 +186,7 @@
 
         out = self.rawcommand(args)
         rev, node = out.splitlines()[-1].rsplit(':')
-        return self.log(node)[0]
+        return int(rev.split()[-1]), node
 
     def config(self, refresh=False):
         if not self._config or refresh:
diff -r 3a1faa3b1fc9 -r 21190e06aa2d tests/test-branch.py
--- a/tests/test-branch.py	Wed Aug 10 19:59:03 2011 +0300
+++ b/tests/test-branch.py	Wed Aug 10 19:59:08 2011 +0300
@@ -8,7 +8,9 @@
     def test_basic(self):
         self.assertEquals(self.client.branch('foo'), 'foo')
         self.append('a', 'a')
-        rev = self.client.commit('first', addremove=True)
+        rev, node = self.client.commit('first', addremove=True)
+
+        rev = self.client.log(node)[0]
 
         self.assertEquals(rev.branch, 'foo')
         self.assertEquals(self.client.branches(),
diff -r 3a1faa3b1fc9 -r 21190e06aa2d tests/test-branches.py
--- a/tests/test-branches.py	Wed Aug 10 19:59:03 2011 +0300
+++ b/tests/test-branches.py	Wed Aug 10 19:59:08 2011 +0300
@@ -13,7 +13,8 @@
         branches = self.client.branches()
 
         expected = []
-        for r in (rev1, rev0):
+        for r, n in (rev1, rev0):
+            r = self.client.log(r)[0]
             expected.append((r.branch, int(r.rev), r.node[:12]))
 
         self.assertEquals(branches, expected)
diff -r 3a1faa3b1fc9 -r 21190e06aa2d tests/test-log.py
--- a/tests/test-log.py	Wed Aug 10 19:59:03 2011 +0300
+++ b/tests/test-log.py	Wed Aug 10 19:59:08 2011 +0300
@@ -4,14 +4,14 @@
 class test_log(common.basetest):
     def test_basic(self):
         self.append('a', 'a')
-        rev0 = self.client.commit('first', addremove=True)
+        rev0, node0 = self.client.commit('first', addremove=True)
         self.append('a', 'a')
-        rev1 = self.client.commit('second')
+        rev1, node1 = self.client.commit('second')
 
         revs = self.client.log()
         revs.reverse()
 
         self.assertTrue(len(revs) == 2)
-        self.assertEquals(revs[1], rev1)
+        self.assertEquals(revs[1].node, node1)
 
         self.assertEquals(revs[0], self.client.log('0')[0])
diff -r 3a1faa3b1fc9 -r 21190e06aa2d tests/test-outgoing-incoming.py
--- a/tests/test-outgoing-incoming.py	Wed Aug 10 19:59:03 2011 +0300
+++ b/tests/test-outgoing-incoming.py	Wed Aug 10 19:59:08 2011 +0300
@@ -15,10 +15,10 @@
         self.assertEquals(self.client.outgoing(path='bar'), bar.incoming())
 
         self.append('a', 'a')
-        rev = self.client.commit('third')
+        rev, node = self.client.commit('third')
         out = self.client.outgoing(path='bar')
 
         self.assertEquals(len(out), 1)
-        self.assertEquals(out[0], rev)
+        self.assertEquals(out[0].node, node)
 
         self.assertEquals(out, bar.incoming())