You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by ma...@apache.org on 2013/12/03 20:31:09 UTC
svn commit: r1547542 -
/commons/proper/dbcp/trunk/src/test/org/apache/commons/dbcp2/managed/TestManagedDataSourceInTx.java
Author: markt
Date: Tue Dec 3 19:31:08 2013
New Revision: 1547542
URL: http://svn.apache.org/r1547542
Log:
Remove pointless (and in some cases incorrect) comments
Remove duplicate test
Modified:
commons/proper/dbcp/trunk/src/test/org/apache/commons/dbcp2/managed/TestManagedDataSourceInTx.java
Modified: commons/proper/dbcp/trunk/src/test/org/apache/commons/dbcp2/managed/TestManagedDataSourceInTx.java
URL: http://svn.apache.org/viewvc/commons/proper/dbcp/trunk/src/test/org/apache/commons/dbcp2/managed/TestManagedDataSourceInTx.java?rev=1547542&r1=1547541&r2=1547542&view=diff
==============================================================================
--- commons/proper/dbcp/trunk/src/test/org/apache/commons/dbcp2/managed/TestManagedDataSourceInTx.java (original)
+++ commons/proper/dbcp/trunk/src/test/org/apache/commons/dbcp2/managed/TestManagedDataSourceInTx.java Tue Dec 3 19:31:08 2013
@@ -224,17 +224,10 @@ public class TestManagedDataSourceInTx e
Connection connection = newConnection();
- // conection should be open
assertFalse("Connection should be open", connection.isClosed());
- // close the connection
connection.close();
- // conection should be open
- assertTrue("Connection should be closed", connection.isClosed());
-
-
- // conection should be open
assertTrue("Connection should be closed", connection.isClosed());
}