You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2023/01/22 16:01:49 UTC
[commons-dbcp] branch master updated: Use try-with-resources
This is an automated email from the ASF dual-hosted git repository.
ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-dbcp.git
The following commit(s) were added to refs/heads/master by this push:
new 1ba67b37 Use try-with-resources
1ba67b37 is described below
commit 1ba67b37f467499493cb28f4b2905eb7baea0ffd
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Sun Jan 22 11:01:45 2023 -0500
Use try-with-resources
---
.../dbcp2/managed/TestManagedDataSource.java | 95 ++++++++++------------
1 file changed, 43 insertions(+), 52 deletions(-)
diff --git a/src/test/java/org/apache/commons/dbcp2/managed/TestManagedDataSource.java b/src/test/java/org/apache/commons/dbcp2/managed/TestManagedDataSource.java
index 04d4e7e4..58336525 100644
--- a/src/test/java/org/apache/commons/dbcp2/managed/TestManagedDataSource.java
+++ b/src/test/java/org/apache/commons/dbcp2/managed/TestManagedDataSource.java
@@ -136,31 +136,30 @@ public class TestManagedDataSource extends TestConnectionPool {
@Test
public void testManagedConnectionEqualInnermost() throws Exception {
ds.setAccessToUnderlyingConnectionAllowed(true);
- final DelegatingConnection<?> con = (DelegatingConnection<?>) getConnection();
- final Connection inner = con.getInnermostDelegate();
- ds.setAccessToUnderlyingConnectionAllowed(false);
- final DelegatingConnection<Connection> con2 = new DelegatingConnection<>(inner);
- assertNotEquals(con2, con);
- assertTrue(con.innermostDelegateEquals(con2.getInnermostDelegate()));
- assertTrue(con2.innermostDelegateEquals(inner));
- assertNotEquals(con, con2);
+ try (final DelegatingConnection<?> con = (DelegatingConnection<?>) getConnection()) {
+ final Connection inner = con.getInnermostDelegate();
+ ds.setAccessToUnderlyingConnectionAllowed(false);
+ final DelegatingConnection<Connection> con2 = new DelegatingConnection<>(inner);
+ assertNotEquals(con2, con);
+ assertTrue(con.innermostDelegateEquals(con2.getInnermostDelegate()));
+ assertTrue(con2.innermostDelegateEquals(inner));
+ assertNotEquals(con, con2);
+ }
}
@Test
public void testManagedConnectionEqualsFail() throws Exception {
- final Connection con1 = getConnection();
- final Connection con2 = getConnection();
- assertNotEquals(con1, con2);
- con1.close();
- con2.close();
+ try (final Connection con1 = getConnection(); final Connection con2 = getConnection()) {
+ assertNotEquals(con1, con2);
+ }
}
@Test
public void testManagedConnectionEqualsNull() throws Exception {
- final Connection con1 = getConnection();
- final Connection con2 = null;
- assertNotEquals(con2, con1);
- con1.close();
+ try (final Connection con1 = getConnection()) {
+ final Connection con2 = null;
+ assertNotEquals(con2, con1);
+ }
}
/*
@@ -168,11 +167,11 @@ public class TestManagedDataSource extends TestConnectionPool {
*/
@Test
public void testManagedConnectionEqualsReflexive() throws Exception {
- final Connection con = getConnection();
- final Connection con2 = con;
- assertEquals(con2, con);
- assertEquals(con, con2);
- con.close();
+ try (final Connection con = getConnection()) {
+ final Connection con2 = con;
+ assertEquals(con2, con);
+ assertEquals(con, con2);
+ }
}
@Test
@@ -211,36 +210,31 @@ public class TestManagedDataSource extends TestConnectionPool {
ds.setAccessToUnderlyingConnectionAllowed(false);
// Grab a new connection - should get c[0]'s closed connection
// so should be delegate-equivalent
- final Connection con = newConnection();
- Assertions.assertNotEquals(c[0], con);
- Assertions.assertEquals(
- ((DelegatingConnection<?>) c[0]).getInnermostDelegateInternal(),
- ((DelegatingConnection<?>) con).getInnermostDelegateInternal());
- for (final Connection element : c) {
- element.close();
+ try (final Connection con = newConnection()) {
+ Assertions.assertNotEquals(c[0], con);
+ Assertions.assertEquals(((DelegatingConnection<?>) c[0]).getInnermostDelegateInternal(),
+ ((DelegatingConnection<?>) con).getInnermostDelegateInternal());
+ for (final Connection element : c) {
+ element.close();
+ }
+ ds.setAccessToUnderlyingConnectionAllowed(true);
}
- ds.setAccessToUnderlyingConnectionAllowed(true);
}
@Test
public void testManagedConnectionEqualsType() throws Exception {
- final Connection con1 = getConnection();
- final Integer con2 = 0;
- assertNotEquals(con2, con1);
- con1.close();
+ try (final Connection con1 = getConnection()) {
+ final Integer con2 = 0;
+ assertNotEquals(con2, con1);
+ }
}
@Test
public void testNestedConnections() throws Exception {
transactionManager.begin();
-
- final Connection c1 = newConnection();
- final Connection c2 = newConnection();
-
- transactionManager.commit();
-
- c1.close();
- c2.close();
+ try (final Connection c1 = newConnection(); final Connection c2 = newConnection()) {
+ transactionManager.commit();
+ }
}
@Test
@@ -268,16 +262,13 @@ public class TestManagedDataSource extends TestConnectionPool {
*/
@Test
public void testSharedConnection() throws Exception {
- final DelegatingConnection<?> connectionA = (DelegatingConnection<?>) newConnection();
- final DelegatingConnection<?> connectionB = (DelegatingConnection<?>) newConnection();
-
- assertNotEquals(connectionA, connectionB);
- assertNotEquals(connectionB, connectionA);
- assertFalse(connectionA.innermostDelegateEquals(connectionB.getInnermostDelegate()));
- assertFalse(connectionB.innermostDelegateEquals(connectionA.getInnermostDelegate()));
-
- connectionA.close();
- connectionB.close();
+ try (final DelegatingConnection<?> connectionA = (DelegatingConnection<?>) newConnection();
+ final DelegatingConnection<?> connectionB = (DelegatingConnection<?>) newConnection()) {
+ assertNotEquals(connectionA, connectionB);
+ assertNotEquals(connectionB, connectionA);
+ assertFalse(connectionA.innermostDelegateEquals(connectionB.getInnermostDelegate()));
+ assertFalse(connectionB.innermostDelegateEquals(connectionA.getInnermostDelegate()));
+ }
}
@Test