check-code: warning and fixes for whitespace in unified tests

contrib/check-code.py

@@ -74,6 +74,7 b' testfilters = ['
 uprefix = r"^  \$ "
 uprefixc = r"^  > "
 utestpats = [
+    (r'^(\S|  $ ).*(\S\s+|^\s+)\n', "trailing whitespace on non-output"),
     (uprefix + r'.*\|\s*sed', "use regex test output patterns instead of sed"),
     (uprefix + r'(true|exit 0)', "explicit zero exit unnecessary"),
     (uprefix + r'.*\$\?', "explicit exit code checks unnecessary"),

tests/test-convert-cvs-branch.t

@@ -42,7 +42,7 b' Branch the project'
   $ cvscall up -r BRANCH > /dev/null
   cvs update: Updating .
 
-Modify file a, then b, then a 
+Modify file a, then b, then a
 
   $ sleep 1
   $ echo "2" > a

tests/test-push-warn.t

@@ -336,7 +336,7 b' Using --new-branch to push new named bra'
   added 1 changesets with 1 changes to 1 files
 
 
-Checking prepush logic does not allow silently pushing 
+Checking prepush logic does not allow silently pushing
 multiple new heads:
 
   $ cd ..
@@ -480,7 +480,7 b' in the clone b1 is now the head of B'
   $ hg ci -Ama2
   adding a2
 
-a2 is now the new branch head of A, and a new topological head 
+a2 is now the new branch head of A, and a new topological head
 it replaces a former inner branch head, so it should at most warn about
 A, not B
 

tests/test-rebase-mq-skip.t

@@ -1,4 +1,4 b''
-This emulates the effects of an hg pull --rebase in which the remote repo 
+This emulates the effects of an hg pull --rebase in which the remote repo
 already has one local mq patch
 
   $ cat >> $HGRCPATH <<EOF