patch to add support for callable statements to the jdbc driver. The patch was submi...
authorBarry Lind
Mon, 24 Jun 2002 06:16:27 +0000 (06:16 +0000)
committerBarry Lind
Mon, 24 Jun 2002 06:16:27 +0000 (06:16 +0000)
src/interfaces/jdbc/org/postgresql/errors.properties
src/interfaces/jdbc/org/postgresql/jdbc2/CallableStatement.java
src/interfaces/jdbc/org/postgresql/jdbc2/Connection.java
src/interfaces/jdbc/org/postgresql/jdbc2/PreparedStatement.java
src/interfaces/jdbc/org/postgresql/test/JDBC2Tests.java
src/interfaces/jdbc/org/postgresql/test/jdbc2/CallableStmtTest.java [new file with mode: 0644]

index c2e529ee999fffc33421f0e022f20478d9f022bf..26ff2f724b244f8e5f553e2e18c279594a11d44d 100644 (file)
@@ -83,3 +83,11 @@ postgresql.updateable.afterlastdelete:After end of result set. Can not call dele
 postgresql.updateable.notoninsertrow:Not on insert row.
 postgresql.updateable.inputstream:Input Stream is null.
 postgresql.updateable.ioerror:Input Stream Error.
+postgresql.call.noreturntype:A CallableStatement Function was declared but no call to 'registerOutParameter (1, )' was made.
+postgresql.call.noinout:PostgreSQL only supports function return value [@ 1] (no OUT or INOUT arguments)
+postgresql.call.procasfunc:This Statement [{0}] defines a procedure call (needs ?= call  to be considered a function.
+postgresql.call.malformed:Malformed stmt [{0}] usage : {1}
+postgresql.call.funcover:Cannot execute Query a call to setXXX (1, ..) was made where argument 1 is the return value of a function.
+postgresql.call.wrongget:Parameter of type {0} was registered but call to get{1} (sqltype={2}) was made.
+postgresql.call.noreturnval:A CallableStatement Function was executed with nothing returned.
+postgresql.call.wrongrtntype:A CallableStatement Function was executed and the return was of type ({0}) however type={1} was registered.
index 2f316234c2355fb4225d03f9ddea557399314f4b..4aa034832537c9e78c27d7e6a9a89ad08c6f7d86 100644 (file)
@@ -7,7 +7,7 @@ package org.postgresql.jdbc2;
 
 import java.sql.*;
 import java.math.*;
-
+import org.postgresql.util.*;
 /*
  * CallableStatement is used to execute SQL stored procedures.
  *
@@ -37,6 +37,7 @@ import java.math.*;
  *
  * @see Connection#prepareCall
  * @see ResultSet
+ * @author Paul Bethe (implementer)
  */
 
 public class CallableStatement extends org.postgresql.jdbc2.PreparedStatement implements java.sql.CallableStatement
@@ -46,9 +47,74 @@ public class CallableStatement extends org.postgresql.jdbc2.PreparedStatement im
     */
    public CallableStatement(Connection c, String q) throws SQLException
    {
-       super(c, q);
+       super(c, q); // don't parse yet..
    }
 
+
+   /**
+    * allows this class to tweak the standard JDBC call !see Usage
+    * -> and replace with the pgsql function syntax
+    * ie. select  as RESULT;
+    */
+
+   protected void parseSqlStmt () throws SQLException {
+       modifyJdbcCall ();
+       super.parseSqlStmt ();
+   }
+   /** 
+    * this method will turn a string of the form
+    * {? = call  (?, [?,..]) }
+    * into the PostgreSQL format which is 
+    * select  (?, [?, ...]) as result
+    * 
+    */
+   private void modifyJdbcCall () throws SQLException {
+       // syntax checking is not complete only a few basics :(
+       originalSql = sql; // save for error msgs..
+       int index = sql.indexOf ("="); // is implied func or proc?
+       boolean isValid = true;
+       if (index != -1) {
+           isFunction = true;
+           isValid = sql.indexOf ("?") < index; // ? before =          
+       }
+       sql = sql.trim ();
+       if (sql.startsWith ("{") && sql.endsWith ("}")) {
+           sql = sql.substring (1, sql.length() -1);
+       } else isValid = false;
+       index = sql.indexOf ("call"); 
+       if (index == -1 || !isValid)
+           throw new PSQLException ("postgresql.call.malformed", 
+                                    new Object[]{sql, JDBC_SYNTAX});
+       sql = sql.replace ('{', ' '); // replace these characters
+       sql = sql.replace ('}', ' ');
+       sql = sql.replace (';', ' ');
+       
+       // this removes the 'call' string and also puts a hidden '?'
+       // at the front of the line for functions, this will
+       // allow the registerOutParameter to work correctly
+                // because in the source sql there was one more ? for the return
+                // value that is not needed by the postgres syntax.  But to make 
+                // sure that the parameter numbers are the same as in the original
+                // sql we add a dummy parameter in this case
+       sql = (isFunction ? "?" : "") + sql.substring (index + 4);
+       
+       sql = "select " + sql + " as " + RESULT_COLUMN + ";";     
+   }
+
+   // internals 
+   static final String JDBC_SYNTAX = "{[? =] call  ([? [,?]*]) }";
+   static final String RESULT_COLUMN = "result";
+   String originalSql = "";
+   boolean isFunction;
+   // functionReturnType contains the user supplied value to check
+   // testReturn contains a modified version to make it easier to 
+   // check the getXXX methods..
+   int functionReturnType;
+   int testReturn;
+   // returnTypeSet is true when a proper call to registerOutParameter has been made
+   boolean returnTypeSet;
+   Object result;
+
    /*
     * Before executing a stored procedure call you must explicitly
     * call registerOutParameter to register the java.sql.Type of each
@@ -58,6 +124,8 @@ public class CallableStatement extends org.postgresql.jdbc2.PreparedStatement im
     * the getXXX method whose Java type XXX corresponds to the
     * parameter's registered SQL type.
     *
+    * ONLY 1 RETURN PARAMETER if {?= call ..} syntax is used
+    *
     * @param parameterIndex the first parameter is 1, the second is 2,...
     * @param sqlType SQL type code defined by java.sql.Types; for
     * parameters of type Numeric or Decimal use the version of
@@ -65,7 +133,55 @@ public class CallableStatement extends org.postgresql.jdbc2.PreparedStatement im
     * @exception SQLException if a database-access error occurs.
     */
    public void registerOutParameter(int parameterIndex, int sqlType) throws SQLException
-       {}
+       {
+           if (parameterIndex != 1)
+               throw new PSQLException ("postgresql.call.noinout");
+           if (!isFunction)
+               throw new PSQLException ("postgresql.call.procasfunc", originalSql);
+           
+           // functionReturnType contains the user supplied value to check
+           // testReturn contains a modified version to make it easier to 
+           // check the getXXX methods..
+           functionReturnType = sqlType;
+           testReturn = sqlType;
+           if (functionReturnType == Types.CHAR || 
+               functionReturnType == Types.LONGVARCHAR)
+               testReturn = Types.VARCHAR;
+           else if (functionReturnType == Types.FLOAT)
+               testReturn = Types.REAL; // changes to streamline later error checking
+           returnTypeSet = true;
+       }
+
+   /**
+    * allow calls to execute update
+    * @return 1 if succesful call otherwise 0
+    */
+   public int executeUpdate() throws SQLException
+   {
+       java.sql.ResultSet rs = super.executeQuery (compileQuery());    
+       if (isFunction) {
+           if (!rs.next ())
+               throw new PSQLException ("postgresql.call.noreturnval");
+           result = rs.getObject(1);
+           int columnType = rs.getMetaData().getColumnType(1);
+           if (columnType != functionReturnType) 
+               throw new PSQLException ("postgresql.call.wrongrtntype",
+                                        new Object[]{
+                   getSqlTypeName (columnType), getSqlTypeName (functionReturnType) }); 
+       }
+       rs.close ();
+       return 1;
+   }
+
+
+   /**
+    * allow calls to execute update
+    * @return true if succesful
+    */
+   public boolean execute() throws SQLException
+   {
+       return (executeUpdate() == 1);
+   }
 
    /*
     * You must also specify the scale for numeric/decimal types:
@@ -82,12 +198,40 @@ public class CallableStatement extends org.postgresql.jdbc2.PreparedStatement im
     */
    public void registerOutParameter(int parameterIndex, int sqlType,
                                     int scale) throws SQLException
-       {}
+       {
+           registerOutParameter (parameterIndex, sqlType); // ignore for now..
+       }
 
-   // Old api?
-   //public boolean isNull(int parameterIndex) throws SQLException {
-   //return true;
-   //}
+   /*
+    * override this method to check for set @ 1 when declared function..
+    *
+    * @param paramIndex the index into the inString
+    * @param s a string to be stored
+    * @exception SQLException if something goes wrong
+    */
+   protected void set(int paramIndex, String s) throws SQLException
+   {
+       if (paramIndex == 1 && isFunction) // need to registerOut instead
+           throw new PSQLException ("postgresql.call.funcover");
+       super.set (paramIndex, s); // else set as usual..
+   }
+
+       /*
+    * Helper - this compiles the SQL query from the various parameters
+    * This is identical to toString() except it throws an exception if a
+    * parameter is unused.
+    */
+   protected synchronized String compileQuery()
+   throws SQLException
+   {
+       if (isFunction && !returnTypeSet)
+           throw new PSQLException("postgresql.call.noreturntype");
+       if (isFunction) { // set entry 1 to dummy entry..
+           inStrings[0] = ""; // dummy entry which ensured that no one overrode
+           // and calls to setXXX (2,..) really went to first arg in a function call..
+       }
+       return super.compileQuery ();
+   }
 
    /*
     * An OUT parameter may have the value of SQL NULL; wasNull
@@ -101,14 +245,9 @@ public class CallableStatement extends org.postgresql.jdbc2.PreparedStatement im
    public boolean wasNull() throws SQLException
    {
        // check to see if the last access threw an exception
-       return false; // fake it for now
+       return (result == null);
    }
 
-   // Old api?
-   //public String getChar(int parameterIndex) throws SQLException {
-   //return null;
-   //}
-
    /*
     * Get the value of a CHAR, VARCHAR, or LONGVARCHAR parameter as a
     * Java String.
@@ -119,7 +258,8 @@ public class CallableStatement extends org.postgresql.jdbc2.PreparedStatement im
     */
    public String getString(int parameterIndex) throws SQLException
    {
-       return null;
+       checkIndex (parameterIndex, Types.VARCHAR, "String");
+       return (String)result;
    }
    //public String getVarChar(int parameterIndex) throws SQLException {
    //   return null;
@@ -138,7 +278,9 @@ public class CallableStatement extends org.postgresql.jdbc2.PreparedStatement im
     */
    public boolean getBoolean(int parameterIndex) throws SQLException
    {
-       return false;
+       checkIndex (parameterIndex, Types.BIT, "Boolean");
+       if (result == null) return false;
+       return ((Boolean)result).booleanValue ();
    }
 
    /*
@@ -150,7 +292,9 @@ public class CallableStatement extends org.postgresql.jdbc2.PreparedStatement im
     */
    public byte getByte(int parameterIndex) throws SQLException
    {
-       return 0;
+       checkIndex (parameterIndex, Types.TINYINT, "Byte");
+       if (result == null) return 0;
+       return (byte)((Integer)result).intValue ();
    }
 
    /*
@@ -162,8 +306,11 @@ public class CallableStatement extends org.postgresql.jdbc2.PreparedStatement im
     */
    public short getShort(int parameterIndex) throws SQLException
    {
-       return 0;
+       checkIndex (parameterIndex, Types.SMALLINT, "Short");
+       if (result == null) return 0;
+       return (short)((Integer)result).intValue ();
    }
+       
 
    /*
     * Get the value of an INTEGER parameter as a Java int.
@@ -174,7 +321,9 @@ public class CallableStatement extends org.postgresql.jdbc2.PreparedStatement im
     */
    public int getInt(int parameterIndex) throws SQLException
    {
-       return 0;
+       checkIndex (parameterIndex, Types.INTEGER, "Int");
+       if (result == null) return 0;
+       return ((Integer)result).intValue ();
    }
 
    /*
@@ -186,7 +335,9 @@ public class CallableStatement extends org.postgresql.jdbc2.PreparedStatement im
     */
    public long getLong(int parameterIndex) throws SQLException
    {
-       return 0;
+       checkIndex (parameterIndex, Types.BIGINT, "Long");
+       if (result == null) return 0;
+       return ((Long)result).longValue ();
    }
 
    /*
@@ -198,7 +349,9 @@ public class CallableStatement extends org.postgresql.jdbc2.PreparedStatement im
     */
    public float getFloat(int parameterIndex) throws SQLException
    {
-       return (float) 0.0;
+       checkIndex (parameterIndex, Types.REAL, "Float");
+       if (result == null) return 0;
+       return ((Float)result).floatValue ();
    }
 
    /*
@@ -210,7 +363,9 @@ public class CallableStatement extends org.postgresql.jdbc2.PreparedStatement im
     */
    public double getDouble(int parameterIndex) throws SQLException
    {
-       return 0.0;
+       checkIndex (parameterIndex, Types.DOUBLE, "Double");
+       if (result == null) return 0;
+       return ((Double)result).doubleValue ();
    }
 
    /*
@@ -227,7 +382,8 @@ public class CallableStatement extends org.postgresql.jdbc2.PreparedStatement im
    public BigDecimal getBigDecimal(int parameterIndex, int scale)
    throws SQLException
    {
-       return null;
+       checkIndex (parameterIndex, Types.NUMERIC, "BigDecimal");
+       return ((BigDecimal)result);
    }
 
    /*
@@ -240,7 +396,8 @@ public class CallableStatement extends org.postgresql.jdbc2.PreparedStatement im
     */
    public byte[] getBytes(int parameterIndex) throws SQLException
    {
-       return null;
+       checkIndex (parameterIndex, Types.VARBINARY, "Bytes");
+       return ((byte [])result);
    }
 
    // New API (JPM) (getLongVarBinary)
@@ -257,7 +414,8 @@ public class CallableStatement extends org.postgresql.jdbc2.PreparedStatement im
     */
    public java.sql.Date getDate(int parameterIndex) throws SQLException
    {
-       return null;
+       checkIndex (parameterIndex, Types.DATE, "Date");
+       return (java.sql.Date)result;
    }
 
    /*
@@ -269,7 +427,8 @@ public class CallableStatement extends org.postgresql.jdbc2.PreparedStatement im
     */
    public java.sql.Time getTime(int parameterIndex) throws SQLException
    {
-       return null;
+       checkIndex (parameterIndex, Types.TIME, "Time");
+       return (java.sql.Time)result;
    }
 
    /*
@@ -282,7 +441,8 @@ public class CallableStatement extends org.postgresql.jdbc2.PreparedStatement im
    public java.sql.Timestamp getTimestamp(int parameterIndex)
    throws SQLException
    {
-       return null;
+       checkIndex (parameterIndex, Types.TIMESTAMP, "Timestamp");
+       return (java.sql.Timestamp)result;
    }
 
    //----------------------------------------------------------------------
@@ -317,7 +477,8 @@ public class CallableStatement extends org.postgresql.jdbc2.PreparedStatement im
    public Object getObject(int parameterIndex)
    throws SQLException
    {
-       return null;
+       checkIndex (parameterIndex);        
+       return result;
    }
 
    // ** JDBC 2 Extensions **
@@ -327,9 +488,10 @@ public class CallableStatement extends org.postgresql.jdbc2.PreparedStatement im
        throw org.postgresql.Driver.notImplemented();
    }
 
-   public java.math.BigDecimal getBigDecimal(int i) throws SQLException
+   public java.math.BigDecimal getBigDecimal(int parameterIndex) throws SQLException
    {
-       throw org.postgresql.Driver.notImplemented();
+       checkIndex (parameterIndex, Types.NUMERIC, "BigDecimal");
+       return ((BigDecimal)result);
    }
 
    public Blob getBlob(int i) throws SQLException
@@ -367,10 +529,76 @@ public class CallableStatement extends org.postgresql.jdbc2.PreparedStatement im
        throw org.postgresql.Driver.notImplemented();
    }
 
+   // no custom types allowed yet..
    public void registerOutParameter(int parameterIndex, int sqlType, String typeName) throws SQLException
    {
        throw org.postgresql.Driver.notImplemented();
    }
 
+
+
+   /** helperfunction for the getXXX calls to check isFunction and index == 1
+    */
+   private void checkIndex (int parameterIndex, int type, String getName) 
+       throws SQLException {
+       checkIndex (parameterIndex);
+       if (type != this.testReturn) 
+           throw new PSQLException("postgresql.call.wrongget",
+                                   new Object[]{getSqlTypeName (testReturn),
+                                                    getName,
+                                                    getSqlTypeName (type)});
+   }
+   /** helperfunction for the getXXX calls to check isFunction and index == 1
+    * @param parameterIndex index of getXXX (index)
+    * check to make sure is a function and index == 1
+    */
+   private void checkIndex (int parameterIndex) throws SQLException {
+       if (!isFunction)
+           throw new PSQLException("postgresql.call.noreturntype");
+       if (parameterIndex != 1)
+           throw new PSQLException("postgresql.call.noinout");
+   }
+       
+   /** helper function for creating msg with type names
+    * @param sqlType a java.sql.Types.XX constant
+    * @return String which is the name of the constant..
+    */
+   private static String getSqlTypeName (int sqlType) {
+       switch (sqlType)
+           {
+           case Types.BIT:
+               return "BIT";
+           case Types.SMALLINT:
+               return "SMALLINT";
+           case Types.INTEGER:
+               return "INTEGER";
+           case Types.BIGINT:
+               return "BIGINT";
+           case Types.NUMERIC:
+               return "NUMERIC";
+           case Types.REAL:
+               return "REAL";
+           case Types.DOUBLE:
+               return "DOUBLE";
+           case Types.FLOAT:
+               return "FLOAT";
+           case Types.CHAR:
+               return "CHAR";
+           case Types.VARCHAR:
+               return "VARCHAR";
+           case Types.DATE:
+               return "DATE";
+           case Types.TIME:
+               return "TIME";
+           case Types.TIMESTAMP:
+               return "TIMESTAMP";
+           case Types.BINARY:
+               return "BINARY";
+           case Types.VARBINARY:
+               return "VARBINARY";
+           default:
+               return "UNKNOWN";
+           }
+   }
 }
 
index 86b1a4fc79c8d0a7120ea8b78cebbb78f6921ed5..9ab3cced8edb3138bd01e0a1ca553a2a02878a01 100644 (file)
@@ -17,7 +17,7 @@ import org.postgresql.largeobject.*;
 import org.postgresql.util.*;
 
 /*
- * $Id: Connection.java,v 1.19 2002/06/11 02:55:16 barry Exp $
+ * $Id: Connection.java,v 1.20 2002/06/24 06:16:27 barry Exp $
  *
  * A Connection represents a session with a specific database. Within the
  * context of a Connection, SQL statements are executed and results are
@@ -135,11 +135,10 @@ public class Connection extends org.postgresql.Connection implements java.sql.Co
 
         public java.sql.CallableStatement prepareCall(String sql, int resultSetType, int resultSetConcurrency) throws SQLException
         {
-                throw new PSQLException("postgresql.con.call");
-                //CallableStatement s = new CallableStatement(this,sql);
-                //s.setResultSetType(resultSetType);
-                //s.setResultSetConcurrency(resultSetConcurrency);
-                //return s;
+           CallableStatement s = new CallableStatement(this,sql);
+           s.setResultSetType(resultSetType);
+           s.setResultSetConcurrency(resultSetConcurrency);
+           return s;
         }
 
         /*
index d0c07718bdbcf33d9bbf2ebace4c1f99e510c17c..eac3acf3acad8f706a20e8a2eb05892baf67ae49 100644 (file)
@@ -58,13 +58,16 @@ public class PreparedStatement extends Statement implements java.sql.PreparedSta
    {
        super(connection);
 
+       this.sql = sql;
+       this.connection = connection;
+       parseSqlStmt (); // this allows Callable stmt to override
+   }
+
+   protected void parseSqlStmt () throws SQLException {
        Vector v = new Vector();
        boolean inQuotes = false;
        int lastParmEnd = 0, i;
 
-       this.sql = sql;
-       this.connection = connection;
-
        for (i = 0; i < sql.length(); ++i)
        {
            int c = sql.charAt(i);
@@ -118,7 +121,7 @@ public class PreparedStatement extends Statement implements java.sql.PreparedSta
     * This is identical to toString() except it throws an exception if a
     * parameter is unused.
     */
-   private synchronized String compileQuery()
+   protected synchronized String compileQuery()
    throws SQLException
    {
        sbuf.setLength(0);
@@ -818,7 +821,7 @@ public class PreparedStatement extends Statement implements java.sql.PreparedSta
     * @param s a string to be stored
     * @exception SQLException if something goes wrong
     */
-   private void set(int paramIndex, String s) throws SQLException
+   protected void set(int paramIndex, String s) throws SQLException
    {
        if (paramIndex < 1 || paramIndex > inStrings.length)
            throw new PSQLException("postgresql.prep.range");
index bacad690281d023e63f167965832d62f80cdbd7b..06c594ec551dbd0ebc2f7b5d99b2b86b099c4922 100644 (file)
@@ -226,8 +226,9 @@ public class JDBC2Tests extends TestSuite
 
        // Fastpath/LargeObject
        suite.addTestSuite(BlobTest.class);
-    suite.addTestSuite( UpdateableResultTest.class );
+       suite.addTestSuite( UpdateableResultTest.class );
 
+       suite.addTestSuite( CallableStmtTest.class );
        // That's all folks
        return suite;
    }
diff --git a/src/interfaces/jdbc/org/postgresql/test/jdbc2/CallableStmtTest.java b/src/interfaces/jdbc/org/postgresql/test/jdbc2/CallableStmtTest.java
new file mode 100644 (file)
index 0000000..5c83d8c
--- /dev/null
@@ -0,0 +1,115 @@
+package org.postgresql.test.jdbc2;
+
+import org.postgresql.test.JDBC2Tests;
+import junit.framework.TestCase;
+import java.io.*;
+import java.sql.*;
+
+/*
+ * CallableStatement tests.
+ * @author Paul Bethe
+ */
+public class CallableStmtTest extends TestCase
+{
+   private Connection con;
+
+   public CallableStmtTest (String name)
+   {
+       super(name);
+   }
+
+   protected void setUp() throws Exception
+   {
+       con = JDBC2Tests.openDB();
+       Statement stmt = con.createStatement ();
+       stmt.execute ("CREATE OR REPLACE FUNCTION testspg__getString (varchar) " + 
+                     "RETURNS varchar AS ' DECLARE inString alias for $1; begin "+
+                     "return ''bob''; end; ' LANGUAGE 'plpgsql';");
+       stmt.execute ("CREATE OR REPLACE FUNCTION testspg__getDouble (float) " + 
+                     "RETURNS float AS ' DECLARE inString alias for $1; begin " +
+                     "return 42.42; end; ' LANGUAGE 'plpgsql';");
+       stmt.execute ("CREATE OR REPLACE FUNCTION testspg__getInt (int) RETURNS int " +
+                     " AS 'DECLARE inString alias for $1; begin " + 
+                     "return 42; end;' LANGUAGE 'plpgsql';"); 
+       stmt.execute ("CREATE OR REPLACE FUNCTION testspg__getNumeric (numeric) " + 
+                     "RETURNS numeric AS ' DECLARE inString alias for $1; " + 
+                     "begin    return 42; end; ' LANGUAGE 'plpgsql';");
+       stmt.close ();
+   }
+
+   protected void tearDown() throws Exception
+   {
+       Statement stmt = con.createStatement ();
+       stmt.execute ("drop FUNCTION testspg__getString (varchar);");
+       stmt.execute ("drop FUNCTION testspg__getDouble (float);");
+       stmt.execute ("drop FUNCTION testspg__getInt (int);");
+       stmt.execute ("drop FUNCTION testspg__getNumeric (numeric);");
+       JDBC2Tests.closeDB(con);
+   }
+
+
+   final String func = "{ ? = call ";
+   final String pkgName = "testspg__";
+   //  protected void runTest () throws Throwable {
+   //testGetString ();
+   //}
+
+   public void testGetDouble () throws Throwable {
+       // System.out.println ("Testing CallableStmt Types.DOUBLE");
+       CallableStatement call = con.prepareCall (func + pkgName + "getDouble (?) }");
+       call.setDouble (2, (double)3.04);
+       call.registerOutParameter (1, Types.DOUBLE);
+       call.execute ();
+       double result = call.getDouble (1);
+       assertTrue ("correct return from getString ()", result == 42.42);
+   }
+
+   public void testGetInt () throws Throwable {    
+       // System.out.println ("Testing CallableStmt Types.INTEGER");
+       CallableStatement call = con.prepareCall (func + pkgName + "getInt (?) }");
+       call.setInt (2, 4);
+       call.registerOutParameter (1, Types.INTEGER);
+       call.execute ();
+       int result = call.getInt (1);
+       assertTrue ("correct return from getString ()", result == 42);
+   }
+
+   public void testGetNumeric () throws Throwable {    
+       // System.out.println ("Testing CallableStmt Types.NUMERIC");
+       CallableStatement call = con.prepareCall (func + pkgName + "getNumeric (?) }");
+       call.setBigDecimal (2, new java.math.BigDecimal(4));
+       call.registerOutParameter (1, Types.NUMERIC);
+       call.execute ();
+       java.math.BigDecimal result = call.getBigDecimal (1);
+       assertTrue ("correct return from getString ()", 
+                   result.equals (new java.math.BigDecimal(42)));
+   }
+
+   public void testGetString () throws Throwable { 
+       // System.out.println ("Testing CallableStmt Types.VARCHAR");
+       CallableStatement call = con.prepareCall (func + pkgName + "getString (?) }");
+       call.setString (2, "foo");
+       call.registerOutParameter (1, Types.VARCHAR);
+       call.execute ();
+       String result = call.getString (1);
+       assertTrue ("correct return from getString ()", result.equals ("bob"));
+
+   }
+
+   public void testBadStmt () throws Throwable {
+       tryOneBadStmt ("{ ?= " + pkgName + "getString (?) }");
+       tryOneBadStmt ("{ ?= call getString (?) ");
+       tryOneBadStmt ("{ = ? call getString (?); }");
+   }
+
+   protected void tryOneBadStmt (String sql) throws Throwable {
+       boolean wasCaught = false;
+       try {
+           CallableStatement call = con.prepareCall (sql);
+       } catch (SQLException e) {
+           wasCaught = true; // good -> this statement was missing something
+       }
+       assertTrue ("bad statment ('"+sql+"')was not caught", wasCaught);
+   }
+           
+}