From: Stefano Lattarini Date: Fri, 26 Aug 2011 09:11:36 +0000 (+0200) Subject: tests: fix spurious failure for awk implementation of TAP driver X-Git-Tag: ng-0.5a~89^2~95^2 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=85aa4fee2425be149d54a634f7664d70184c5b41;p=thirdparty%2Fautomake.git tests: fix spurious failure for awk implementation of TAP driver * tests/tap-realtime.test: The awk+shell implementation of the TAP driver must "read ahead" of one line in order to catch the exit status of the test script it runs. So, in the TAP-producer script here, be sure to echo one "dummy" line after each TAP result line in order not to cause false positives. --- diff --git a/ChangeLog b/ChangeLog index e407d8989..666445abf 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,12 @@ +2011-08-26 Stefano Lattarini + + tests: fix spurious failure for awk implementation of TAP driver + * tests/tap-realtime.test: The awk+shell implementation of the TAP + driver must "read ahead" of one line in order to catch the exit + status of the test script it runs. So, in the TAP-producer script + here, be sure to echo one "dummy" line after each TAP result line + in order not to cause false positives. + 2011-08-25 Stefano Lattarini tap/awk: allow escaping of TAP directives diff --git a/tests/tap-realtime.test b/tests/tap-realtime.test index 4a38f1422..61476c38b 100755 --- a/tests/tap-realtime.test +++ b/tests/tap-realtime.test @@ -47,15 +47,23 @@ echo 1..3 # FIXME: creative quoting to placate maintainer-check sleep="sleep "3 +# The awk+shell implementation of the TAP driver must "read ahead" of one +# line in order to catch the exit status of the test script it runs. So +# be sure to echo one "dummy" line after each result line in order not to +# cause false positives. + echo ok 1 - foo +echo DUMMY $sleep test -f ok-1 || { echo 'Bail out!'; exit 1; } echo ok 2 - bar +echo DUMMY $sleep test -f ok-2 || { echo 'Bail out!'; exit 1; } echo ok 3 - baz +echo DUMMY $sleep test -f ok-3 || { echo 'Bail out!'; exit 1; }