continuum-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ri...@apache.org
Subject svn commit: r599785 - in /maven/continuum/branches/continuum-jpa/continuum-model-jpa/src/test/java/org/apache/maven/continuum/store: ApplicationContextAwareStoreTestCase.java jpa/JpaProjectGroupStoreTest.java jpa/JpaProjectStoreTest.java
Date Fri, 30 Nov 2007 11:31:57 GMT
Author: rinku
Date: Fri Nov 30 03:31:49 2007
New Revision: 599785

URL: http://svn.apache.org/viewvc?rev=599785&view=rev
Log:
o  more clean ups.

Modified:
    maven/continuum/branches/continuum-jpa/continuum-model-jpa/src/test/java/org/apache/maven/continuum/store/ApplicationContextAwareStoreTestCase.java
    maven/continuum/branches/continuum-jpa/continuum-model-jpa/src/test/java/org/apache/maven/continuum/store/jpa/JpaProjectGroupStoreTest.java
    maven/continuum/branches/continuum-jpa/continuum-model-jpa/src/test/java/org/apache/maven/continuum/store/jpa/JpaProjectStoreTest.java

Modified: maven/continuum/branches/continuum-jpa/continuum-model-jpa/src/test/java/org/apache/maven/continuum/store/ApplicationContextAwareStoreTestCase.java
URL: http://svn.apache.org/viewvc/maven/continuum/branches/continuum-jpa/continuum-model-jpa/src/test/java/org/apache/maven/continuum/store/ApplicationContextAwareStoreTestCase.java?rev=599785&r1=599784&r2=599785&view=diff
==============================================================================
--- maven/continuum/branches/continuum-jpa/continuum-model-jpa/src/test/java/org/apache/maven/continuum/store/ApplicationContextAwareStoreTestCase.java
(original)
+++ maven/continuum/branches/continuum-jpa/continuum-model-jpa/src/test/java/org/apache/maven/continuum/store/ApplicationContextAwareStoreTestCase.java
Fri Nov 30 03:31:49 2007
@@ -35,7 +35,7 @@
     /**
      * Continuum Store persistent unit defined in <code>persistence.xml</code>
used by tests.
      */
-    protected static final String PERSISTENT_UNIT_CONTINUUM_STORE = "continuum-store";
+    private static final String PERSISTENT_UNIT_CONTINUUM_STORE = "continuum-store";
 
     /**
      * Spring application context.
@@ -168,6 +168,15 @@
     {
         Object store = this.applicationContext.getBean( storeBeanReference );
         return (Store<T>) store;
+    }
+
+    /**
+     * Returns the name of the persistent-unit setup in <code>persistence.xml</code>.
+     */
+    @Override
+    protected String getPersistenceUnitName()
+    {
+        return PERSISTENT_UNIT_CONTINUUM_STORE;
     }
 
 }

Modified: maven/continuum/branches/continuum-jpa/continuum-model-jpa/src/test/java/org/apache/maven/continuum/store/jpa/JpaProjectGroupStoreTest.java
URL: http://svn.apache.org/viewvc/maven/continuum/branches/continuum-jpa/continuum-model-jpa/src/test/java/org/apache/maven/continuum/store/jpa/JpaProjectGroupStoreTest.java?rev=599785&r1=599784&r2=599785&view=diff
==============================================================================
--- maven/continuum/branches/continuum-jpa/continuum-model-jpa/src/test/java/org/apache/maven/continuum/store/jpa/JpaProjectGroupStoreTest.java
(original)
+++ maven/continuum/branches/continuum-jpa/continuum-model-jpa/src/test/java/org/apache/maven/continuum/store/jpa/JpaProjectGroupStoreTest.java
Fri Nov 30 03:31:49 2007
@@ -49,15 +49,6 @@
         setSqlSource( testData );
     }
 
-    /**
-     * Returns the name of the persistent-unit setup in <code>persistence.xml</code>.
-     */
-    @Override
-    protected String getPersistenceUnitName()
-    {
-        return PERSISTENT_UNIT_CONTINUUM_STORE;
-    }
-
     @Test
     public void testOpenJPASetup()
     {
@@ -94,7 +85,7 @@
     }
 
     /**
-     * Convenience method to get {@link Store} implementation tied to a {@link Project}.
+     * Convenience method to get {@link Store} implementation tied to a {@link ProjectGroup}.
      * 
      * @return
      */

Modified: maven/continuum/branches/continuum-jpa/continuum-model-jpa/src/test/java/org/apache/maven/continuum/store/jpa/JpaProjectStoreTest.java
URL: http://svn.apache.org/viewvc/maven/continuum/branches/continuum-jpa/continuum-model-jpa/src/test/java/org/apache/maven/continuum/store/jpa/JpaProjectStoreTest.java?rev=599785&r1=599784&r2=599785&view=diff
==============================================================================
--- maven/continuum/branches/continuum-jpa/continuum-model-jpa/src/test/java/org/apache/maven/continuum/store/jpa/JpaProjectStoreTest.java
(original)
+++ maven/continuum/branches/continuum-jpa/continuum-model-jpa/src/test/java/org/apache/maven/continuum/store/jpa/JpaProjectStoreTest.java
Fri Nov 30 03:31:49 2007
@@ -46,15 +46,6 @@
         setSqlSource( testData );
     }
 
-    /**
-     * Returns the name of the persistent-unit setup in <code>persistence.xml</code>.
-     */
-    @Override
-    protected String getPersistenceUnitName()
-    {
-        return PERSISTENT_UNIT_CONTINUUM_STORE;
-    }
-
     @Test
     public void testOpenJPASetup()
     {
@@ -90,7 +81,6 @@
     public void tearDown() throws Exception
     {
         super.tearDown();
-        // do nothing
     }
 
     /**



Mime
View raw message