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();
     }
 
     /**