Add metadata, connection, statement and PreparedStatement integration tests (#1213)

## Summary

- Add 26 new fake service integration tests across 4 existing test
classes
- Tests cover metadata retrieval, connection management, statement
lifecycle, and PreparedStatement enhancements
- All tests use the existing WireMock-based record-replay infrastructure
in SEA mode
- Recorded against production and verified in replay mode (no token
needed for CI)

## Changes by test class

| Test Class | New Tests | Coverage Area |
|-----------|-----------|--------------|
| MetadataIntegrationTests | +10 | ResultSetMetaData (columnLabel,
typeName, precision, scale, displaySize, className, boolean properties,
signed, multiple data types), DatabaseMetaData.getTypeInfo(),
ParameterMetaData.getParameterCount() |
| ConnectionIntegrationTests | +7 | isClosed(), isValid(), getCatalog(),
getSchema(), getMetaData(), getWarnings()/clearWarnings(),
getClientInfo() |
| ExecutionIntegrationTests | +5 | Statement lifecycle:
close()/isClosed(), getConnection(), getMaxRows()/setMaxRows(),
getQueryTimeout()/setQueryTimeout(), getWarnings()/clearWarnings() |
| PreparedStatementIntegrationTests | +4 | clearParameters(), setNull()
with INT type, setObject() without SQL type, getMetaData() for SELECT |

## New JDBC methods covered

### ResultSetMetaData
- getColumnLabel(), getColumnTypeName(), getPrecision(), getScale()
- getColumnDisplaySize(), getColumnClassName()
- isAutoIncrement(), isSearchable(), isCurrency(), isReadOnly(),
isSigned()

### DatabaseMetaData
- getTypeInfo() - full type catalog

### ParameterMetaData
- getParameterCount()

### Connection
- isClosed(), isValid(int), getCatalog(), getSchema()
- getMetaData(), getWarnings(), clearWarnings(), getClientInfo()

### Statement
- close(), isClosed(), getConnection()
- getMaxRows(), setMaxRows(), getQueryTimeout(), setQueryTimeout()
- getWarnings(), clearWarnings()

### PreparedStatement
- clearParameters(), setNull() with INT, setObject() without type
- getMetaData() before execution

## Test plan

- [x] All 56 tests pass in REPLAY mode (no production access needed)
- [x] All tests compile successfully
- [x] WireMock recordings committed under src/test/resources/sqlexecapi/
and cloudfetchapi/
- [ ] Verify CI passes

## Updated Coverage

Including PR #1212 

┌─────────────────────────────────────┬───────────────────┬──────────────────┐
│ │ Before (both PRs) │ After (both PRs) │

├─────────────────────────────────────┼───────────────────┼──────────────────┤
│ JDBC methods with integration tests │ ~44 / 328 │ ~96 / 328 │

├─────────────────────────────────────┼───────────────────┼──────────────────┤
│ Coverage % │ 13.4% │ ~29.3% │

└─────────────────────────────────────┴───────────────────┴──────────────────┘

NO_CHANGELOG=true

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

Author: gopalldb

src/test/java/com/databricks/jdbc/integration/fakeservice/tests/ConnectionIntegrationTests.java

@@ -2,15 +2,12 @@
 
 import static com.databricks.jdbc.integration.IntegrationTestUtil.*;
 import static org.junit.jupiter.api.Assertions.*;
-import static org.junit.jupiter.api.Assertions.assertThrows;
 
 import com.databricks.jdbc.common.DatabricksJdbcUrlParams;
 import com.databricks.jdbc.exception.DatabricksSQLException;
 import com.databricks.jdbc.integration.fakeservice.AbstractFakeServiceIntegrationTests;
 import com.databricks.jdbc.integration.fakeservice.FakeServiceConfigLoader;
-import java.sql.Connection;
-import java.sql.DriverManager;
-import java.sql.SQLException;
+import java.sql.*;
 import java.util.Properties;
 import org.junit.jupiter.api.Test;
 
@@ -83,7 +80,7 @@ void testPATinOAuthTokenPassThrough() throws Exception {
     conn.close();
   }
 
-  // --- Connection management tests ---
+  // --- Connection properties and management tests ---
 
   @Test
   void testIsClosed_NewConnection() throws SQLException {
@@ -127,6 +124,41 @@ void testGetSchema_ReturnsNonNull() throws SQLException {
     conn.close();
   }
 
+  @Test
+  void testGetMetaData_ReturnsNonNull() throws SQLException {
+    Connection conn = getValidJDBCConnection();
+
+    DatabaseMetaData metaData = conn.getMetaData();
+    assertNotNull(metaData, "getMetaData() should return non-null");
+    assertNotNull(metaData.getDriverName(), "Driver name should not be null");
+    assertFalse(metaData.getDriverName().isEmpty(), "Driver name should not be empty");
+
+    conn.close();
+  }
+
+  @Test
+  void testGetWarnings_AndClearWarnings() throws SQLException {
+    Connection conn = getValidJDBCConnection();
+
+    // New connection may or may not have warnings, but getWarnings() should not throw
+    SQLWarning warnings = conn.getWarnings();
+    // clearWarnings should not throw
+    conn.clearWarnings();
+    assertNull(conn.getWarnings(), "Warnings should be null after clearWarnings()");
+
+    conn.close();
+  }
+
+  @Test
+  void testGetClientInfo_ReturnsProperties() throws SQLException {
+    Connection conn = getValidJDBCConnection();
+
+    Properties clientInfo = conn.getClientInfo();
+    assertNotNull(clientInfo, "getClientInfo() should return non-null Properties");
+
+    conn.close();
+  }
+
   private Properties createConnectionProperties(Properties extraProps) {
     Properties connProps = new Properties();
     connProps.putAll(extraProps);

src/test/java/com/databricks/jdbc/integration/fakeservice/tests/ExecutionIntegrationTests.java

@@ -278,6 +278,81 @@ void testExecuteAsyncStatement() throws Exception {
         rs2.unwrap(IDatabricksResultSet.class).getStatementStatus().getState());
   }
 
+  // --- Statement lifecycle and property tests ---
+
+  @Test
+  void testStatement_CloseAndIsClosed() throws SQLException {
+    Statement stmt = connection.createStatement();
+    assertFalse(stmt.isClosed(), "New statement should not be closed");
+
+    stmt.close();
+    assertTrue(stmt.isClosed(), "Statement should be closed after close()");
+  }
+
+  @Test
+  void testStatement_GetConnection() throws SQLException {
+    Statement stmt = connection.createStatement();
+    Connection stmtConn = stmt.getConnection();
+    assertNotNull(stmtConn, "getConnection() should return non-null");
+    assertSame(connection, stmtConn, "getConnection() should return the parent connection");
+
+    stmt.close();
+  }
+
+  @Test
+  void testStatement_MaxRows() throws SQLException {
+    Statement stmt = connection.createStatement();
+
+    // Default maxRows should be 0 (no limit)
+    assertEquals(0, stmt.getMaxRows(), "Default maxRows should be 0");
+
+    stmt.setMaxRows(100);
+    assertEquals(100, stmt.getMaxRows(), "maxRows should be 100 after setMaxRows(100)");
+
+    // setMaxRows(0) means no limit
+    stmt.setMaxRows(0);
+    assertEquals(0, stmt.getMaxRows(), "maxRows should be 0 after setMaxRows(0)");
+
+    // Negative value should throw
+    assertThrows(
+        SQLException.class, () -> stmt.setMaxRows(-1), "setMaxRows(-1) should throw SQLException");
+
+    stmt.close();
+  }
+
+  @Test
+  void testStatement_QueryTimeout() throws SQLException {
+    Statement stmt = connection.createStatement();
+
+    // Default timeout should be 0 (no timeout)
+    assertEquals(0, stmt.getQueryTimeout(), "Default queryTimeout should be 0");
+
+    stmt.setQueryTimeout(30);
+    assertEquals(30, stmt.getQueryTimeout(), "queryTimeout should be 30 after setQueryTimeout(30)");
+
+    // Negative value should throw
+    assertThrows(
+        SQLException.class,
+        () -> stmt.setQueryTimeout(-1),
+        "setQueryTimeout(-1) should throw SQLException");
+
+    stmt.close();
+  }
+
+  @Test
+  void testStatement_Warnings() throws SQLException {
+    Statement stmt = connection.createStatement();
+
+    // getWarnings should not throw
+    SQLWarning warnings = stmt.getWarnings();
+
+    // clearWarnings should not throw
+    stmt.clearWarnings();
+    assertNull(stmt.getWarnings(), "Warnings should be null after clearWarnings()");
+
+    stmt.close();
+  }
+
   // --- Execution result handling tests (getResultSet, getUpdateCount, getMoreResults) ---
 
   @Test
@@ -538,7 +613,5 @@ void testPreparedStatementBatch_ClearBatch() throws SQLException {
     assertEquals(10, rs.getInt("id"));
     assertEquals("fresh1", rs.getString("col1"));
     assertFalse(rs.next(), "Should only have 1 row");
-
-    deleteTable(connection, tableName);
   }
 }