Re: Bug with callable statement and output parameters

On Thu, 27 Apr 2006, Dave Cramer wrote:

> Patch attached for review and test
>

This patch does not handle the situation where the number of registered
parameters does not match the number of returned parameters as the
attached tests show.

Also the regression tests you sent me separately are no good.  They do not
pass because they mix creating and destroying the test function in the
test method and the tearDown method.  This should go into setUp and
tearDown.  Also you can't call executeQuery on a CallableStatement because
it will complain with "No results were returned by the query."

The whole Jdbc3CallableStatementTest sucks.  It should have all of its
function/table creation destruction moved into setUp/tearDown.  Also the
coding:

try {
     // something
} catch(Exception e) {
     fail(e.getMessage();
}

Is a waste of code.  Not only does it not do anything, but it also
obscures the source of the exception.

Kris Jurka