Test qtop breaking when series ends with guarded patches.

tests/test-mq-guards

@@ -1,154 +1,156 b''
 #!/bin/sh
 
 echo "[extensions]" >> $HGRCPATH
 echo "mq=" >> $HGRCPATH
 
 hg init
 hg qinit
 
 echo x > x
 hg ci -Ama
 
 hg qnew a.patch
 echo a > a
 hg add a
 hg qrefresh
 
 hg qnew b.patch
 echo b > b
 hg add b
 hg qrefresh
 
 hg qnew c.patch
 echo c > c
 hg add c
 hg qrefresh
 
 hg qpop -a
 
 echo % should fail
 hg qguard does-not-exist.patch +bleh
 
 echo % should fail
 hg qguard +fail
 
 hg qpush
 echo % should guard a.patch
 hg qguard +a
 echo % should print +a
 hg qguard
 hg qpop
 
 hg qguard a.patch
 echo % should push b.patch
 hg qpush
 
 hg qpop
 hg qselect a
 echo % should push a.patch
 hg qpush
 
 hg qguard c.patch -a
 echo % should print -a
 hg qguard c.patch
 
 echo % should skip c.patch
 hg qpush -a
+echo % should display b.patch
+hg qtop
 
 hg qguard -n c.patch
 echo % should push c.patch
 hg qpush -a
 
 hg qpop -a
 hg qselect -n
 echo % should push all
 hg qpush -a
 
 hg qpop -a
 hg qguard a.patch +1
 hg qguard b.patch +2
 hg qselect 1
 echo % should push a.patch, not b.patch
 hg qpush
 hg qpush
 hg qpop -a
 
 hg qselect 2
 echo % should push b.patch
 hg qpush
 hg qpush -a
 # Used to be an issue with holes in the patch sequence
 # So, put one hole on the base and ask for topmost patch.
 hg qtop
 hg qpop -a
 
 hg qselect 1 2
 echo % should push a.patch, b.patch
 hg qpush
 hg qpush
 hg qpop -a
 
 hg qguard a.patch +1 +2 -3
 hg qselect 1 2 3
 echo % list patches and guards
 hg qguard -l
 echo % list series
 hg qseries -v
 echo % list guards
 hg qselect
 echo % should push b.patch
 hg qpush
 
 hg qpush -a
 hg qselect -n --reapply
 echo % guards in series file: +1 +2 -3
 hg qselect -s
 echo % should show c.patch
 hg qapplied
 
 hg qrename a.patch new.patch
 echo % should show :
 echo % new.patch: +1 +2 -3
 echo % b.patch: +2
 echo % c.patch: unguarded
 hg qguard -l
 
 hg qnew d.patch
 hg qpop
 echo % should show new.patch and b.patch as Guarded, c.patch as Applied
 echo % and d.patch as Unapplied
 hg qseries -v
 
 hg qguard d.patch +2
 echo % new.patch, b.patch: Guarded. c.patch: Applied. d.patch: Guarded.
 hg qseries -v
 
 qappunappv()
 (
     for command in qapplied "qapplied -v" qunapplied "qunapplied -v"; do
         echo % hg $command
         hg $command
     done
 )
 
 hg qpop -a
 hg qguard -l
 qappunappv
 hg qselect 1
 qappunappv
 hg qpush -a
 qappunappv
 hg qselect 2
 qappunappv
 
 for patch in `hg qseries`; do
     echo % hg qapplied $patch
     hg qapplied $patch
     echo % hg qunapplied $patch
     hg qunapplied $patch
 done
 
 echo % hg qseries -m: only b.patch should be shown
 echo the guards file was not ignored in the past
 hg qdelete -k b.patch
 hg qseries -m

tests/test-mq-guards.out

@@ -1,182 +1,184 b''
 adding x
 Patch queue now empty
 % should fail
 abort: no patch named does-not-exist.patch
 % should fail
 abort: no patches applied
 applying a.patch
 Now at: a.patch
 % should guard a.patch
 % should print +a
 a.patch: +a
 Patch queue now empty
 a.patch: +a
 % should push b.patch
 applying b.patch
 Now at: b.patch
 Patch queue now empty
 number of unguarded, unapplied patches has changed from 2 to 3
 % should push a.patch
 applying a.patch
 Now at: a.patch
 % should print -a
 c.patch: -a
 % should skip c.patch
 applying b.patch
 skipping c.patch - guarded by '-a'
 Now at: b.patch
+% should display b.patch
+b.patch
 % should push c.patch
 applying c.patch
 Now at: c.patch
 Patch queue now empty
 guards deactivated
 number of unguarded, unapplied patches has changed from 3 to 2
 % should push all
 applying b.patch
 applying c.patch
 Now at: c.patch
 Patch queue now empty
 number of unguarded, unapplied patches has changed from 1 to 2
 % should push a.patch, not b.patch
 applying a.patch
 Now at: a.patch
 applying c.patch
 Now at: c.patch
 Patch queue now empty
 % should push b.patch
 applying b.patch
 Now at: b.patch
 applying c.patch
 Now at: c.patch
 c.patch
 Patch queue now empty
 number of unguarded, unapplied patches has changed from 2 to 3
 % should push a.patch, b.patch
 applying a.patch
 Now at: a.patch
 applying b.patch
 Now at: b.patch
 Patch queue now empty
 number of unguarded, unapplied patches has changed from 3 to 2
 % list patches and guards
 a.patch: +1 +2 -3
 b.patch: +2
 c.patch: unguarded
 % list series
 0 G a.patch
 1 U b.patch
 2 U c.patch
 % list guards
 1
 2
 3
 % should push b.patch
 applying b.patch
 Now at: b.patch
 applying c.patch
 Now at: c.patch
 guards deactivated
 popping guarded patches
 Patch queue now empty
 reapplying unguarded patches
 applying c.patch
 Now at: c.patch
 % guards in series file: +1 +2 -3
 +1
 +2
 -3
 % should show c.patch
 c.patch
 % should show :
 % new.patch: +1 +2 -3
 % b.patch: +2
 % c.patch: unguarded
 new.patch: +1 +2 -3
 b.patch: +2
 c.patch: unguarded
 Now at: c.patch
 % should show new.patch and b.patch as Guarded, c.patch as Applied
 % and d.patch as Unapplied
 0 G new.patch
 1 G b.patch
 2 A c.patch
 3 U d.patch
 % new.patch, b.patch: Guarded. c.patch: Applied. d.patch: Guarded.
 0 G new.patch
 1 G b.patch
 2 A c.patch
 3 G d.patch
 Patch queue now empty
 new.patch: +1 +2 -3
 b.patch: +2
 c.patch: unguarded
 d.patch: +2
 % hg qapplied
 % hg qapplied -v
 % hg qunapplied
 c.patch
 % hg qunapplied -v
 0 G new.patch
 1 G b.patch
 2 U c.patch
 3 G d.patch
 number of unguarded, unapplied patches has changed from 1 to 2
 % hg qapplied
 % hg qapplied -v
 % hg qunapplied
 new.patch
 c.patch
 % hg qunapplied -v
 0 U new.patch
 1 G b.patch
 2 U c.patch
 3 G d.patch
 applying new.patch
 skipping b.patch - guarded by ['+2']
 applying c.patch
 skipping d.patch - guarded by ['+2']
 Now at: c.patch
 % hg qapplied
 new.patch
 c.patch
 % hg qapplied -v
 0 A new.patch
 1 G b.patch
 2 A c.patch
 % hg qunapplied
 % hg qunapplied -v
 3 G d.patch
 number of unguarded, unapplied patches has changed from 0 to 1
 number of guarded, applied patches has changed from 1 to 0
 % hg qapplied
 new.patch
 c.patch
 % hg qapplied -v
 0 A new.patch
 1 U b.patch
 2 A c.patch
 % hg qunapplied
 d.patch
 % hg qunapplied -v
 3 U d.patch
 % hg qapplied new.patch
 new.patch
 % hg qunapplied new.patch
 b.patch
 d.patch
 % hg qapplied b.patch
 new.patch
 % hg qunapplied b.patch
 d.patch
 % hg qapplied c.patch
 new.patch
 c.patch
 % hg qunapplied c.patch
 d.patch
 % hg qapplied d.patch
 new.patch
 c.patch
 % hg qunapplied d.patch
 % hg qseries -m: only b.patch should be shown
 the guards file was not ignored in the past
 b.patch