You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@continuum.apache.org by ri...@apache.org on 2007/11/30 12:31:57 UTC
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
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
}
/**