<div dir="ltr">Looks good (sorry no crew keys with me).</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Feb 8, 2013 at 12:56 PM, Durham Goode <span dir="ltr"><<a href="mailto:durham@fb.com" target="_blank">durham@fb.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"># HG changeset patch<br>
# User Durham Goode <<a href="mailto:durham@fb.com">durham@fb.com</a>><br>
# Date 1360324302 28800<br>
# Node ID 02dd78c2b8d74dc8a97d1aa17238cebf37ccd708<br>
# Parent  7ce1c3b93414e08a9be1d83f17e7d87d2a24ab3b<br>
tests: fix tests that use integer bookmarks<br>
<br>
Integer bookmarks are now not allowed (since they alias with revision numbers),<br>
so a hglib test was failing.<br>
<br>
diff --git a/tests/test-bookmarks.py b/tests/test-bookmarks.py<br>
--- a/tests/test-bookmarks.py<br>
+++ b/tests/test-bookmarks.py<br>
@@ -10,14 +10,14 @@<br>
         self.append('a', 'a')<br>
         rev1, node1 = self.client.commit('second')<br>
<br>
-        self.client.bookmark('0', rev0)<br>
+        self.client.bookmark('zero', rev0)<br>
         self.assertEquals(self.client.bookmarks(),<br>
-                          ([('0', rev0, node0[:12])], -1))<br>
+                          ([('zero', rev0, node0[:12])], -1))<br>
<br>
-        self.client.bookmark('1', rev1)<br>
+        self.client.bookmark('one', rev1)<br>
         self.assertEquals(self.client.bookmarks(),<br>
-                          ([('0', rev0, node0[:12]),<br>
-                            ('1', rev1, node1[:12])], 1))<br>
+                          ([('one', rev1, node1[:12]),<br>
+                            ('zero', rev0, node0[:12])], 0))<br>
<br>
     #def test_spaces(self):<br>
     #    self.client.bookmark('s pace', self.rev0)<br>
_______________________________________________<br>
Mercurial-devel mailing list<br>
<a href="mailto:Mercurial-devel@selenic.com">Mercurial-devel@selenic.com</a><br>
<a href="http://selenic.com/mailman/listinfo/mercurial-devel" target="_blank">http://selenic.com/mailman/listinfo/mercurial-devel</a><br>
</blockquote></div><br></div>