Remove infinite-loop hazards in ecpg test suite.
authorTom Lane <tgl@sss.pgh.pa.us>
Thu, 24 Jan 2019 21:46:55 +0000 (16:46 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Thu, 24 Jan 2019 21:47:06 +0000 (16:47 -0500)
commitd5a1fde397269bd60b99e70a4eb2934b75fd31ec
tree16b5d39d7efe4be09c2baa281d39028bc1907b1b
parentbbd5c207b960dd072445a3547f16abea0c27f726
Remove infinite-loop hazards in ecpg test suite.

A report from Andrew Dunstan showed that an ecpglib breakage that
causes repeated query failures could lead to infinite loops in some
ecpg test scripts, because they contain "while(1)" loops with no
exit condition other than successful test completion.  That might
be all right for manual testing, but it seems entirely unacceptable
for automated test environments such as our buildfarm.  We don't
want buildfarm owners to have to intervene manually when a test
goes wrong.

To fix, just change all those while(1) loops to exit after at most
100 iterations (which is more than any of them expect to iterate).
This seems sufficient since we'd see discrepancies in the test output
if any loop executed the wrong number of times.

I tested this by dint of intentionally breaking ecpg_do_prologue
to always fail, and verifying that the tests still got to completion.

Back-patch to all supported branches, since the whole point of this
exercise is to protect the buildfarm against future mistakes.

Discussion: https://postgr.es/m/18693.1548302004@sss.pgh.pa.us
18 files changed:
src/interfaces/ecpg/test/compat_informix/test_informix.pgc
src/interfaces/ecpg/test/compat_oracle/char_array.pgc
src/interfaces/ecpg/test/expected/compat_informix-test_informix.c
src/interfaces/ecpg/test/expected/compat_oracle-char_array.c
src/interfaces/ecpg/test/expected/pgtypeslib-nan_test.c
src/interfaces/ecpg/test/expected/preproc-autoprep.c
src/interfaces/ecpg/test/expected/preproc-outofscope.c
src/interfaces/ecpg/test/expected/preproc-variable.c
src/interfaces/ecpg/test/expected/preproc-whenever_do_continue.c
src/interfaces/ecpg/test/expected/sql-fetch.c
src/interfaces/ecpg/test/expected/sql-quote.c
src/interfaces/ecpg/test/pgtypeslib/nan_test.pgc
src/interfaces/ecpg/test/preproc/autoprep.pgc
src/interfaces/ecpg/test/preproc/outofscope.pgc
src/interfaces/ecpg/test/preproc/variable.pgc
src/interfaces/ecpg/test/preproc/whenever_do_continue.pgc
src/interfaces/ecpg/test/sql/fetch.pgc
src/interfaces/ecpg/test/sql/quote.pgc