test-logtoprocess: use cat to wait for outputs

Commands started by logtoprocess are running asynchronously. To be able to
test the output, we need to block and wait for the output.

The patch uses "| cat" to wait for such "asynchronous" outputs, to make the
test more reliable. I have also written a short notice at the top, hopefully
people would be aware of the pitfall when changing the test.

tests/test-logtoprocess.t

@@ -1,70 +1,73 b''
+ATTENTION: logtoprocess runs commands asynchronously. Be sure to append "| cat"
+to hg commands, to wait for the output, if you want to test its output.
+Otherwise the test will be flaky.
+
 Test if logtoprocess correctly captures command-related log calls.
 
   $ hg init
   $ cat > $TESTTMP/foocommand.py << EOF
   > from mercurial import cmdutil
   > from time import sleep
   > cmdtable = {}
   > command = cmdutil.command(cmdtable)
   > @command('foo', [])
   > def foo(ui, repo):
   >     ui.log('foo', 'a message: %(bar)s\n', bar='spam')
   > EOF
   $ cp $HGRCPATH $HGRCPATH.bak
   $ cat >> $HGRCPATH << EOF
   > [extensions]
   > logtoprocess=
   > foocommand=$TESTTMP/foocommand.py
   > [logtoprocess]
   > command=echo 'logtoprocess command output:';
   >     echo "\$EVENT";
   >     echo "\$MSG1";
   >     echo "\$MSG2"
   > commandfinish=echo 'logtoprocess commandfinish output:';
   >     echo "\$EVENT";
   >     echo "\$MSG1";
   >     echo "\$MSG2";
   >     echo "\$MSG3"
   > foo=echo 'logtoprocess foo output:';
   >     echo "\$EVENT";
   >     echo "\$MSG1";
   >     echo "\$OPT_BAR"
   > EOF
 
 Running a command triggers both a ui.log('command') and a
 ui.log('commandfinish') call. The foo command also uses ui.log.
 
-Use head to ensure we wait for all lines to be produced, and sort to avoid
-ordering issues between the various processes we spawn:
-  $ hg foo | head -n 17 | sort
+Use sort to avoid ordering issues between the various processes we spawn:
+  $ hg foo | cat | sort
   
   
   
   0
   a message: spam
   command
   commandfinish
   foo
   foo
   foo
   foo
   foo exited 0 after * seconds (glob)
   logtoprocess command output:
   logtoprocess commandfinish output:
   logtoprocess foo output:
   spam
 
 Confirm that logging blocked time catches stdio properly:
   $ cp $HGRCPATH.bak $HGRCPATH
   $ cat >> $HGRCPATH << EOF
   > [extensions]
   > logtoprocess=
   > pager=
   > [logtoprocess]
   > uiblocked=echo "\$EVENT stdio \$OPT_STDIO_BLOCKED ms command \$OPT_COMMAND_DURATION ms"
   > [ui]
   > logblockedtimes=True
   > EOF
 
-  $ hg log
+  $ hg log | cat
   uiblocked stdio [0-9]+.[0-9]* ms command [0-9]+.[0-9]* ms (re)