You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@openjpa.apache.org by dw...@apache.org on 2010/10/26 20:28:38 UTC
svn commit: r1027673 -
/openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/kernel/TestPessimisticLocking.java
Author: dwoods
Date: Tue Oct 26 18:28:38 2010
New Revision: 1027673
URL: http://svn.apache.org/viewvc?rev=1027673&view=rev
Log:
test should call super.tearDown() for cleanup
Modified:
openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/kernel/TestPessimisticLocking.java
Modified: openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/kernel/TestPessimisticLocking.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/kernel/TestPessimisticLocking.java?rev=1027673&r1=1027672&r2=1027673&view=diff
==============================================================================
--- openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/kernel/TestPessimisticLocking.java (original)
+++ openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/kernel/TestPessimisticLocking.java Tue Oct 26 18:28:38 2010
@@ -83,6 +83,7 @@ public class TestPessimisticLocking exte
return pm;
}
+ @Override
public void setUp() throws Exception {
super.setUp(RuntimeTest1.class, RuntimeTest2.class, RuntimeTest3.class);
@@ -101,12 +102,16 @@ public class TestPessimisticLocking exte
endEm(pm);
}
- public void tearDown() {
+ @Override
+ public void tearDown() throws Exception {
try {
- if (_factory != null)
+ if (_factory != null) {
_factory.close();
+ _factory = null;
+ }
} catch (Exception e) {
}
+ super.tearDown();
}
/**