完善测试

git-svn-id: http://code.alibabatech.com/svn/druid/trunk@821 b9813039-fb51-4c41-a8b9-e21c2acb5095

src/main/java/com/alibaba/druid/mock/MockConnection.java

@@ -152,6 +152,10 @@ public class MockConnection implements Connection {
 
     @Override
     public void setAutoCommit(boolean autoCommit) throws SQLException {
+        if (closed) {
+            throw new SQLException();
+        }
+
         this.autoCommit = autoCommit;
     }
 
@@ -162,11 +166,17 @@ public class MockConnection implements Connection {
 
     @Override
     public void commit() throws SQLException {
+        if (closed) {
+            throw new SQLException();
+        }
 
     }
 
     @Override
     public void rollback() throws SQLException {
+        if (closed) {
+            throw new SQLException();
+        }
 
     }
 
@@ -280,22 +290,32 @@ public class MockConnection implements Connection {
 
     @Override
     public Savepoint setSavepoint() throws SQLException {
+        if (closed) {
+            throw new SQLException();
+        }
         return null;
     }
 
     @Override
     public Savepoint setSavepoint(String name) throws SQLException {
+        if (closed) {
+            throw new SQLException();
+        }
         return null;
     }
 
     @Override
     public void rollback(Savepoint savepoint) throws SQLException {
-
+        if (closed) {
+            throw new SQLException();
+        }
     }
 
     @Override
     public void releaseSavepoint(Savepoint savepoint) throws SQLException {
-
+        if (closed) {
+            throw new SQLException();
+        }
     }
 
     @Override

src/main/java/com/alibaba/druid/pool/PoolableConnection.java

@@ -845,7 +845,7 @@ public class PoolableConnection implements PooledConnection, Connection {
         holder.getStatementEventListeners().remove(listener);
     }
 
-    protected void checkOpen() throws SQLException {
+    public void checkOpen() throws SQLException {
         if (holder == null) {
             throw new SQLException("connection is closed");
         }

src/main/java/com/alibaba/druid/pool/PoolablePreparedStatement.java

@@ -699,9 +699,9 @@ public class PoolablePreparedStatement extends PoolableStatement implements Prep
 
         protected final MethodType methodType;
 
-        public PreparedStatementKey(String sql, String catalog, MethodType methodType){
+        public PreparedStatementKey(String sql, String catalog, MethodType methodType) throws SQLException{
             if (sql == null) {
-                throw new IllegalArgumentException("sql is null");
+                throw new SQLException("sql is null");
             }
 
             this.sql = sql;