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:32:37 UTC

svn commit: r1027674 - /openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/meta/TestMetaDataRepository.java

Author: dwoods
Date: Tue Oct 26 18:32:37 2010
New Revision: 1027674

URL: http://svn.apache.org/viewvc?rev=1027674&view=rev
Log:
use common closeEMF() in testcase

Modified:
    openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/meta/TestMetaDataRepository.java

Modified: openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/meta/TestMetaDataRepository.java
URL: http://svn.apache.org/viewvc/openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/meta/TestMetaDataRepository.java?rev=1027674&r1=1027673&r2=1027674&view=diff
==============================================================================
--- openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/meta/TestMetaDataRepository.java (original)
+++ openjpa/trunk/openjpa-persistence-jdbc/src/test/java/org/apache/openjpa/persistence/meta/TestMetaDataRepository.java Tue Oct 26 18:32:37 2010
@@ -69,9 +69,7 @@ public class TestMetaDataRepository exte
 					MdrTestEntity.class, "query");
 			assertNotNull(query);
 		} finally {
-			if (emf != null) {
-				emf.close();
-			}
+			closeEMF(emf);
 		}
 	}
 	
@@ -80,7 +78,7 @@ public class TestMetaDataRepository exte
         emf = createNamedEMF(PU_NAME, "openjpa.MetaDataRepository", "Preload=true");
         MetaDataRepository repo = emf.getConfiguration().getMetaDataRepositoryInstance();
         emf.createEntityManager();
-        emf.close();
+        closeEMF(emf);
         assertFalse("The PCRegistry should no longer reference the MetaDataRepository.", PCRegistry
             .removeRegisterClassListener(repo));
     }
@@ -98,9 +96,7 @@ public class TestMetaDataRepository exte
             
 
         } finally {
-            if (emf != null) {
-                emf.close();
-            }
+            closeEMF(emf);
         }
     }
 }