I wrote:
> BTW, this exposes a pretty nasty omission in pg_regress: it fails to
> say anything about a nonzero exit code from a psql child process
> that's running a test.   Seems like wait_for_tests() ought to complain
> about that.  Any objections?

So I coded this up, and fortunately thought to try it with ecpg's tests
before committing:

test preproc/define       ... ok
test preproc/init         ... ok
test preproc/type         ... ok
test preproc/variable     ... ok
test preproc/whenever     ... FAILED: test process exited with exit code 1
test sql/array            ... ok
test sql/binary           ... ok
test sql/code100          ... ok
test sql/copystdout       ... ok

Apparently the exit(1) is intentional in that test.

We could possibly extend the syntax of regression schedule files to have
a way to say what's the expected exit status, but that seems like more
work than it's worth.  Would it be all right to just remove the test of
"on error stop" mode?

                        regards, tom lane

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to