mq: do not call [0] on revset stable

mq: do not call [0] on revset

The __getitem__ method have been removed. The "first" method is to be used
instead. Test have been extended to test this code path.

--- a/hgext/mq.py	Sat Nov 01 22:58:30 2014 +0000
+++ b/hgext/mq.py	Sat Nov 01 22:59:37 2014 +0000
@@ -2001,7 +2001,7 @@
                 base = repo.changelog.node(rev.first())
                 if base in [n.node for n in self.applied]:
                     raise util.Abort(_('revision %d is already managed')
-                                     % rev[0])
+                                     % rev.first())
                 if heads != [self.applied[-1].node]:
                     raise util.Abort(_('revision %d is not the parent of '
                                        'the queue') % rev.first())

--- a/tests/test-mq.t	Sat Nov 01 22:58:30 2014 +0000
+++ b/tests/test-mq.t	Sat Nov 01 22:59:37 2014 +0000
@@ -1596,4 +1596,11 @@
   $ hg ci -m a
   $ hg qimport -r 0::
 
+reimport things
+
+  $ hg qimport -r 1::
+  abort: revision 2 is already managed
+  [255]
+
+
   $ cd ..