Fix connection leaks in hibernate-testing connection provider delegates

Author: beikov

hibernate-core/src/main/java/org/hibernate/engine/jdbc/connections/internal/DriverManagerConnectionProviderImpl.java

@@ -471,7 +471,9 @@ protected void closeConnection(Connection conn, Throwable t) {
 				}
 			}
 			finally {
-				allConnections.remove( conn );
+				if ( !allConnections.remove( conn ) ) {
+					ConnectionInfoLogger.INSTANCE.debug( "Connection remove failed." );
+				}
 			}
 		}
 

hibernate-core/src/test/java/org/hibernate/orm/test/resource/ConnectionProviderDelegate.java

@@ -122,7 +122,7 @@ public Connection getConnection() throws SQLException {
 
 	@Override
 	public void closeConnection(Connection connection) throws SQLException {
-		connectionProvider.closeConnection( connection );
+		connectionProvider.closeConnection( ((ConnectionWrapper) connection).delegate );
 	}
 
 	@Override

hibernate-testing/src/main/java/org/hibernate/testing/jta/JtaAwareConnectionProviderImpl.java

@@ -218,7 +218,7 @@ private void delist(Connection connection) {
 			System.err.println( "!!!Error trying to reset synchronization registry!!!" );
 		}
 		try {
-			delegate.closeConnection( connection );
+			delegate.closeConnection( ((ConnectionWrapper) connection).delegate );
 		}
 		catch (SQLException e) {
 			System.err.println( "!!!Error trying to close JDBC connection from delist callbacks!!!" );