Merge pull request #2587 from ClickHouse/jdbc_parameter_metadata

[JDBC] added out of index test for parameter metadata

Author: chernser

jdbc-v2/src/main/java/com/clickhouse/jdbc/metadata/ParameterMetaDataImpl.java

@@ -127,6 +127,8 @@ public String getParameterClassName(int param) throws SQLException {
     @Override
     public int getParameterMode(int param) throws SQLException {
         checkParamIndex(param);
+        // only in parameter mode IN is supported by prepared statement
+        // other modes are designed for callable statements
         return parameterModeIn;
     }
 }

jdbc-v2/src/test/java/com/clickhouse/jdbc/metadata/ParameterMetaDataImplTest.java

@@ -5,7 +5,9 @@
 import java.sql.SQLException;
 import java.sql.Types;
 
-import static org.testng.Assert.*;
+import static org.testng.Assert.assertEquals;
+import static org.testng.Assert.assertFalse;
+import static org.testng.Assert.assertThrows;
 
 
 public class ParameterMetaDataImplTest {
@@ -81,4 +83,17 @@ public void testGetParameterMode() throws SQLException {
         assertThrows(() -> metaData.getParameterMode(0));
         assertThrows(() -> metaData.getParameterMode(2));
     }
+
+    @Test(groups = {"integration"})
+    public void testColumnOutOfIndex() throws SQLException {
+        ParameterMetaDataImpl metaData = new ParameterMetaDataImpl(1);
+        int indexAfterLastColumn = 2;
+        assertThrows(SQLException.class, () -> metaData.getParameterMode(indexAfterLastColumn));
+        assertThrows(SQLException.class, () -> metaData.getParameterType(indexAfterLastColumn));
+        assertThrows(SQLException.class, () -> metaData.getScale(indexAfterLastColumn));
+        assertThrows(SQLException.class, () -> metaData.getPrecision(indexAfterLastColumn));
+        assertThrows(SQLException.class, () -> metaData.getParameterClassName(indexAfterLastColumn));
+        assertThrows(SQLException.class, () -> metaData.isNullable(indexAfterLastColumn));
+        assertThrows(SQLException.class, () -> metaData.isSigned(indexAfterLastColumn));
+    }
 }