Fix metadata thrift recordings (#1236)

## Summary
- Re-recorded WireMock stubs for 7 metadata tests that had invalid 403
"RBAC: access denied" recordings (GetTypeInfo, GetProcedures,
GetProcedureColumns, GetFunctions, GetExportedKeys, GetVersionColumns,
GetAttributes)
- Added new Thrift recordings for GetIndexInfo test
- Fixed flaky `testSetAndGetSchema` and `testSetAndGetCatalog` in
ConnectionIntegrationTests by adding `safeClose()` helper that skips
`connection.close()` in THRIFT + REPLAY mode

## Root Cause

**8 Metadata test failures:** The original Thrift recordings were
captured with an expired/unauthorized token, resulting in 403 responses
being recorded as WireMock stubs. When replayed in CI, the 403 caused
`getValidJDBCConnection()` to throw, and the `setUp()` method's catch
block silently swallowed the error, leaving `connection=null` which
NPE'd in each test.

**Flaky ConnectionIntegrationTests:** `testSetAndGetSchema` and
`testSetAndGetCatalog` execute statements (SET CATALOG, USE SCHEMA)
which generate `CloseOperation` Thrift calls during `conn.close()`.
WireMock uses exact binary body matching (`binaryEqualTo`) for these
stubs, but stub registration order from disk is non-deterministic. When
stubs load in the wrong order, WireMock returns 404 for CloseOperation
calls. This is the same known issue that all other fake service test
classes already work around by skipping `connection.close()` in THRIFT +
REPLAY mode.

## Test plan
- [x] All 7 re-recorded metadata tests verified to pass in THRIFT_SERVER
+ REPLAY mode
- [x] Full MetadataIntegrationTests suite (46 tests) passes with 0
failures
- [x] ConnectionIntegrationTests (17 tests) passes with 0 failures, 3
consecutive runs
- [x] All 5 test classes (150 tests total) pass 3/3 consecutive runs in
both THRIFT and SQL_EXEC replay modes
- [x] No regressions in other existing fake service test classes

🤖 Generated with [Claude Code](https://claude.com/claude-code)

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

@@ -3,10 +3,13 @@
 import static com.databricks.jdbc.integration.IntegrationTestUtil.*;
 import static org.junit.jupiter.api.Assertions.*;
 
+import com.databricks.jdbc.api.impl.DatabricksConnection;
+import com.databricks.jdbc.common.DatabricksClientType;
 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 com.databricks.jdbc.integration.fakeservice.FakeServiceExtension;
 import java.sql.*;
 import java.util.Properties;
 import org.junit.jupiter.api.Test;
@@ -94,7 +97,7 @@ void testSetAndGetCatalog() throws SQLException {
     conn.setCatalog(testCatalog);
     assertEquals(testCatalog, conn.getCatalog(), "getCatalog() should return what was set");
 
-    conn.close();
+    safeClose(conn);
   }
 
   // --- Connection properties and management tests ---
@@ -154,7 +157,7 @@ void testSetAndGetSchema() throws SQLException {
     conn.setSchema(testSchema);
     assertEquals(testSchema, conn.getSchema(), "getSchema() should return what was set");
 
-    conn.close();
+    safeClose(conn);
   }
 
   @Test
@@ -244,6 +247,23 @@ void testGetTransactionIsolation() throws SQLException {
     conn.close();
   }
 
+  /**
+   * Closes the connection, but skips the close in Thrift REPLAY mode due to WireMock stub matching
+   * issues with CloseOperation/CloseSession binary bodies.
+   */
+  private void safeClose(Connection conn) throws SQLException {
+    if (conn != null) {
+      if (((DatabricksConnection) conn).getConnectionContext().getClientType()
+              == DatabricksClientType.THRIFT
+          && getFakeServiceMode() == FakeServiceExtension.FakeServiceMode.REPLAY) {
+        // Hacky fix
+        // Wiremock has error in stub matching for close operation in THRIFT + REPLAY mode
+      } else {
+        conn.close();
+      }
+    }
+  }
+
   private Properties createConnectionProperties(Properties extraProps) {
     Properties connProps = new Properties();
     connProps.putAll(extraProps);