You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tapestry.apache.org by ka...@apache.org on 2013/04/07 09:04:11 UTC

[2/2] git commit: FIXED - TAP5-2102: Allow supplying EntityManager properties via TapestryPersistenceUnitInfo - interestingly, the tests collide in a suite since app5 sets a custom PersistenceProviderResolver. Also seems that the tests are run in the wro

FIXED - TAP5-2102: Allow supplying EntityManager properties via
TapestryPersistenceUnitInfo
- interestingly, the tests collide in a suite since app5 sets a custom
PersistenceProviderResolver. Also seems that the tests are run in the
wrong order (first functional, then unit tests). Too cumbersome to fix
the DummyPersistenceProvider, just add Eclipse as an available
PersistenceProvider, it's already in the classpath


Project: http://git-wip-us.apache.org/repos/asf/tapestry-5/repo
Commit: http://git-wip-us.apache.org/repos/asf/tapestry-5/commit/0f320744
Tree: http://git-wip-us.apache.org/repos/asf/tapestry-5/tree/0f320744
Diff: http://git-wip-us.apache.org/repos/asf/tapestry-5/diff/0f320744

Branch: refs/heads/5.3
Commit: 0f32074419ad610856ace173461d312adcb9dccc
Parents: 1ed180c
Author: Kalle Korhonen <ka...@apache.org>
Authored: Sun Apr 7 00:01:19 2013 -0700
Committer: Kalle Korhonen <ka...@apache.org>
Committed: Sun Apr 7 00:02:37 2013 -0700

----------------------------------------------------------------------
 tapestry-jpa/build.gradle                          |   10 +++++-----
 .../java/org/example/app5/services/AppModule.java  |    2 +-
 2 files changed, 6 insertions(+), 6 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/tapestry-5/blob/0f320744/tapestry-jpa/build.gradle
----------------------------------------------------------------------
diff --git a/tapestry-jpa/build.gradle b/tapestry-jpa/build.gradle
index c197c19..62d21bb 100644
--- a/tapestry-jpa/build.gradle
+++ b/tapestry-jpa/build.gradle
@@ -3,13 +3,13 @@ description = "Provides support for simple CRUD applications built on top of Tap
 dependencies {
   compile project(':tapestry-core')
   compile "org.apache.geronimo.specs:geronimo-jpa_2.0_spec:1.1"
-  
+
   testCompile project(':tapestry-test')
-  
+
   testRuntime "com.h2database:h2:1.2.145"
-  testRuntime "org.eclipse.persistence:org.eclipse.persistence.jpa:2.2.0"
-  testRuntime "org.apache.tomcat:dbcp:6.0.32"   
-  
+  testCompile "org.eclipse.persistence:org.eclipse.persistence.jpa:2.2.0"
+  testRuntime "org.apache.tomcat:dbcp:6.0.32"
+
 }
 
 repositories {

http://git-wip-us.apache.org/repos/asf/tapestry-5/blob/0f320744/tapestry-jpa/src/test/java/org/example/app5/services/AppModule.java
----------------------------------------------------------------------
diff --git a/tapestry-jpa/src/test/java/org/example/app5/services/AppModule.java b/tapestry-jpa/src/test/java/org/example/app5/services/AppModule.java
index 93dc5d1..3b0afea 100644
--- a/tapestry-jpa/src/test/java/org/example/app5/services/AppModule.java
+++ b/tapestry-jpa/src/test/java/org/example/app5/services/AppModule.java
@@ -37,7 +37,7 @@ public class AppModule {
         PersistenceProviderResolverHolder.setPersistenceProviderResolver(new PersistenceProviderResolver() {
             @Override
             public List<PersistenceProvider> getPersistenceProviders() {
-                return Arrays.<PersistenceProvider>asList(new DummyPersistenceProvider());
+                return Arrays.<PersistenceProvider>asList(new DummyPersistenceProvider(), new org.eclipse.persistence.jpa.PersistenceProvider());
             }
 
             @Override