You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomee.apache.org by otaviojava <gi...@git.apache.org> on 2018/11/21 12:57:16 UTC
[GitHub] tomee pull request #211: Add an integration test beteween JPA and CMP [WIP]
GitHub user otaviojava opened a pull request:
https://github.com/apache/tomee/pull/211
Add an integration test beteween JPA and CMP [WIP]
Hey, I would like to add a new test in this case.
I created a test, however, I had an issue to inject the Users interface. I tried multiples way without a good result.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/otaviojava/tomee add_cmp_test
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/tomee/pull/211.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #211
----
commit 169da6342530f39091d25a6b95dc7cb6f241597a
Author: Otavio Santana <ot...@...>
Date: 2018-11-19T14:43:21Z
adds entity and CMD test
commit 82137efd4c5ef1c1e1a994566f9c568704ee1081
Author: Otavio Santana <ot...@...>
Date: 2018-11-21T12:38:19Z
adds scenario to test
----
---
[GitHub] tomee pull request #211: Add an integration test beteween JPA and CMP [WIP]
Posted by otaviojava <gi...@git.apache.org>.
Github user otaviojava commented on a diff in the pull request:
https://github.com/apache/tomee/pull/211#discussion_r235388470
--- Diff: container/openejb-core/src/test/java/org/apache/openejb/core/webservices/JPACMDIntegrationTest.java ---
@@ -0,0 +1,339 @@
+package org.apache.openejb.core.webservices;
--- End diff --
Fixed, thank you
---
[GitHub] tomee pull request #211: Add an integration test beteween JPA and CMP [WIP]
Posted by otaviojava <gi...@git.apache.org>.
Github user otaviojava commented on a diff in the pull request:
https://github.com/apache/tomee/pull/211#discussion_r235388354
--- Diff: container/openejb-core/src/test/java/org/apache/openejb/core/webservices/JPACMDIntegrationTest.java ---
@@ -0,0 +1,339 @@
+package org.apache.openejb.core.webservices;
+
+import org.apache.openejb.assembler.classic.ReloadableEntityManagerFactory;
+import org.apache.openejb.config.AppModule;
+import org.apache.openejb.config.EjbModule;
+import org.apache.openejb.jee.CmpField;
+import org.apache.openejb.jee.ContainerTransaction;
+import org.apache.openejb.jee.EjbJar;
+import org.apache.openejb.jee.EntityBean;
+import org.apache.openejb.jee.PersistenceType;
+import org.apache.openejb.jee.Query;
+import org.apache.openejb.jee.QueryMethod;
+import org.apache.openejb.jee.SingletonBean;
+import org.apache.openejb.jee.TransAttribute;
+import org.apache.openejb.jee.jpa.Attributes;
+import org.apache.openejb.jee.jpa.Basic;
+import org.apache.openejb.jee.jpa.Column;
+import org.apache.openejb.jee.jpa.EntityMappings;
+import org.apache.openejb.jee.jpa.NamedQuery;
+import org.apache.openejb.jee.jpa.unit.Persistence;
+import org.apache.openejb.jee.jpa.unit.PersistenceUnit;
+import org.apache.openejb.junit.ApplicationComposer;
+import org.apache.openejb.testing.Configuration;
+import org.apache.openejb.testing.Module;
+import org.junit.Assert;
+import org.junit.Test;
+import org.junit.runner.RunWith;
+
+import javax.annotation.Resource;
+import javax.ejb.CreateException;
+import javax.ejb.EJB;
+import javax.ejb.EJBException;
+import javax.ejb.EntityContext;
+import javax.ejb.FinderException;
+import javax.ejb.LocalHome;
+import javax.ejb.RemoteHome;
+import javax.ejb.RemoveException;
+import javax.ejb.SessionContext;
+import javax.naming.Context;
+import javax.persistence.Entity;
+import javax.persistence.EntityManager;
+import javax.persistence.EntityManagerFactory;
+import javax.persistence.GeneratedValue;
+import javax.persistence.Id;
+import java.io.File;
+import java.rmi.RemoteException;
+import java.util.Collection;
+import java.util.List;
+import java.util.Properties;
+
+import static org.junit.Assert.assertEquals;
+
+@RunWith(ApplicationComposer.class)
+public class JPACMDIntegrationTest {
--- End diff --
Fixed, thanks
---
[GitHub] tomee pull request #211: Add an integration test beteween JPA and CMP [WIP]
Posted by jeanouii <gi...@git.apache.org>.
Github user jeanouii commented on a diff in the pull request:
https://github.com/apache/tomee/pull/211#discussion_r235385420
--- Diff: container/openejb-core/src/test/java/org/apache/openejb/core/webservices/JPACMDIntegrationTest.java ---
@@ -0,0 +1,339 @@
+package org.apache.openejb.core.webservices;
+
+import org.apache.openejb.assembler.classic.ReloadableEntityManagerFactory;
+import org.apache.openejb.config.AppModule;
+import org.apache.openejb.config.EjbModule;
+import org.apache.openejb.jee.CmpField;
+import org.apache.openejb.jee.ContainerTransaction;
+import org.apache.openejb.jee.EjbJar;
+import org.apache.openejb.jee.EntityBean;
+import org.apache.openejb.jee.PersistenceType;
+import org.apache.openejb.jee.Query;
+import org.apache.openejb.jee.QueryMethod;
+import org.apache.openejb.jee.SingletonBean;
+import org.apache.openejb.jee.TransAttribute;
+import org.apache.openejb.jee.jpa.Attributes;
+import org.apache.openejb.jee.jpa.Basic;
+import org.apache.openejb.jee.jpa.Column;
+import org.apache.openejb.jee.jpa.EntityMappings;
+import org.apache.openejb.jee.jpa.NamedQuery;
+import org.apache.openejb.jee.jpa.unit.Persistence;
+import org.apache.openejb.jee.jpa.unit.PersistenceUnit;
+import org.apache.openejb.junit.ApplicationComposer;
+import org.apache.openejb.testing.Configuration;
+import org.apache.openejb.testing.Module;
+import org.junit.Assert;
+import org.junit.Test;
+import org.junit.runner.RunWith;
+
+import javax.annotation.Resource;
+import javax.ejb.CreateException;
+import javax.ejb.EJB;
+import javax.ejb.EJBException;
+import javax.ejb.EntityContext;
+import javax.ejb.FinderException;
+import javax.ejb.LocalHome;
+import javax.ejb.RemoteHome;
+import javax.ejb.RemoveException;
+import javax.ejb.SessionContext;
+import javax.naming.Context;
+import javax.persistence.Entity;
+import javax.persistence.EntityManager;
+import javax.persistence.EntityManagerFactory;
+import javax.persistence.GeneratedValue;
+import javax.persistence.Id;
+import java.io.File;
+import java.rmi.RemoteException;
+import java.util.Collection;
+import java.util.List;
+import java.util.Properties;
+
+import static org.junit.Assert.assertEquals;
+
+@RunWith(ApplicationComposer.class)
+public class JPACMDIntegrationTest {
--- End diff --
I think it's CMP and not CMD
---
[GitHub] tomee pull request #211: Add an integration test beteween JPA and CMP [WIP]
Posted by otaviojava <gi...@git.apache.org>.
Github user otaviojava closed the pull request at:
https://github.com/apache/tomee/pull/211
---
[GitHub] tomee pull request #211: Add an integration test beteween JPA and CMP [WIP]
Posted by jeanouii <gi...@git.apache.org>.
Github user jeanouii commented on a diff in the pull request:
https://github.com/apache/tomee/pull/211#discussion_r235385458
--- Diff: container/openejb-core/src/test/java/org/apache/openejb/core/webservices/JPACMDIntegrationTest.java ---
@@ -0,0 +1,339 @@
+package org.apache.openejb.core.webservices;
--- End diff --
Missing Apache header
---