You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jdo-commits@db.apache.org by mb...@apache.org on 2006/03/22 20:22:33 UTC
svn commit: r387917 -
/db/jdo/trunk/tck20/src/java/org/apache/jdo/tck/util/BatchTestRunner.java
Author: mbo
Date: Wed Mar 22 11:22:29 2006
New Revision: 387917
URL: http://svn.apache.org/viewcvs?rev=387917&view=rev
Log:
JDO-335: call closePMF in finally block
Modified:
db/jdo/trunk/tck20/src/java/org/apache/jdo/tck/util/BatchTestRunner.java
Modified: db/jdo/trunk/tck20/src/java/org/apache/jdo/tck/util/BatchTestRunner.java
URL: http://svn.apache.org/viewcvs/db/jdo/trunk/tck20/src/java/org/apache/jdo/tck/util/BatchTestRunner.java?rev=387917&r1=387916&r2=387917&view=diff
==============================================================================
--- db/jdo/trunk/tck20/src/java/org/apache/jdo/tck/util/BatchTestRunner.java (original)
+++ db/jdo/trunk/tck20/src/java/org/apache/jdo/tck/util/BatchTestRunner.java Wed Mar 22 11:22:29 2006
@@ -90,7 +90,7 @@
run(new TestSuite(clazz));
}
- /** Runs the specified test. */
+ /** Runs the specified test or test suite */
public static TestResult run(Test test) {
return new BatchTestRunner().doRun(test);
}
@@ -100,11 +100,15 @@
new BatchTestRunner().doRun(suite, true);
}
- /** Runs the specified test and close the pmf. */
+ /** Runs the specified test or test suite and closes the pmf. */
public TestResult doRun(Test test) {
- TestResult result = doRun(test, false);
- JDO_Test.dumpSupportedOptions(LOG_DIRECTORY + "configuration");
- JDO_Test.closePMF();
+ TestResult result = null;
+ try {
+ result = doRun(test, false);
+ JDO_Test.dumpSupportedOptions(LOG_DIRECTORY + "configuration");
+ } finally {
+ JDO_Test.closePMF();
+ }
return result;
}