You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tomee.apache.org by an...@apache.org on 2014/10/28 10:12:23 UTC

[1/2] git commit: TOMEE-1408 use distinct app name for each test and revert expected ejbJar count back to 1

Repository: tomee
Updated Branches:
  refs/heads/develop b12b59f4a -> 736c33041


TOMEE-1408 use distinct app name for each test and revert expected ejbJar count back to 1


Project: http://git-wip-us.apache.org/repos/asf/tomee/repo
Commit: http://git-wip-us.apache.org/repos/asf/tomee/commit/91e9840f
Tree: http://git-wip-us.apache.org/repos/asf/tomee/tree/91e9840f
Diff: http://git-wip-us.apache.org/repos/asf/tomee/diff/91e9840f

Branch: refs/heads/develop
Commit: 91e9840fe035ba433cffa3d4f64a8f5151882be2
Parents: 2a38e33
Author: Mark Struberg <st...@apache.org>
Authored: Sat Oct 25 22:28:36 2014 +0200
Committer: Mark Struberg <st...@apache.org>
Committed: Sat Oct 25 22:28:36 2014 +0200

----------------------------------------------------------------------
 .../openejb/config/DeploymentsElementTest.java  | 60 ++++++++++++--------
 1 file changed, 37 insertions(+), 23 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/tomee/blob/91e9840f/container/openejb-core/src/test/java/org/apache/openejb/config/DeploymentsElementTest.java
----------------------------------------------------------------------
diff --git a/container/openejb-core/src/test/java/org/apache/openejb/config/DeploymentsElementTest.java b/container/openejb-core/src/test/java/org/apache/openejb/config/DeploymentsElementTest.java
index 59b20d5..22c4a37 100644
--- a/container/openejb-core/src/test/java/org/apache/openejb/config/DeploymentsElementTest.java
+++ b/container/openejb-core/src/test/java/org/apache/openejb/config/DeploymentsElementTest.java
@@ -66,8 +66,10 @@ public class DeploymentsElementTest extends Assert {
     public void deploymentsDir_Ear_Packed_ApplicationXml() throws Exception {
         final Server server = new Server();
         final File apps = server.deploymentsDir("myapps");
+        
+        final String appName = "deploymentsDir_Ear_Packed_ApplicationXml";
 
-        final File ear = new File(apps, "colors.ear");
+        final File ear = new File(apps, appName + ".ear");
         {
             final Map<String, Object> contents = new HashMap<String, Object>();
             contents.put("foo.jar", Archives.jarArchive(Orange.class));
@@ -78,7 +80,7 @@ public class DeploymentsElementTest extends Assert {
         final OpenEjbConfiguration configuration = server.init();
 
         assertEquals(1, configuration.containerSystem.applications.size());
-        assertEquals(2, configuration.containerSystem.applications.get(0).ejbJars.size());
+        assertEquals(1, configuration.containerSystem.applications.get(0).ejbJars.size());
         assertEquals(1, configuration.containerSystem.applications.get(0).ejbJars.get(0).enterpriseBeans.size());
         assertEquals("Orange", configuration.containerSystem.applications.get(0).ejbJars.get(0).enterpriseBeans.get(0).ejbName);
     }
@@ -95,7 +97,9 @@ public class DeploymentsElementTest extends Assert {
         final Server server = new Server();
         final File apps = server.deploymentsDir("myapps");
 
-        final File ear = new File(apps, "colors.ear");
+        final String appName = "deploymentsDir_Ear_Packed_NoApplicationXml";
+
+        final File ear = new File(apps, appName + ".ear");
         {
             final Map<String, Object> contents = new HashMap<String, Object>();
             contents.put("foo.jar", Archives.jarArchive(Orange.class));
@@ -105,7 +109,7 @@ public class DeploymentsElementTest extends Assert {
         final OpenEjbConfiguration configuration = server.init();
 
         assertEquals(1, configuration.containerSystem.applications.size());
-        assertEquals(2, configuration.containerSystem.applications.get(0).ejbJars.size());
+        assertEquals(1, configuration.containerSystem.applications.get(0).ejbJars.size());
         assertEquals(1, configuration.containerSystem.applications.get(0).ejbJars.get(0).enterpriseBeans.size());
         assertEquals("Orange", configuration.containerSystem.applications.get(0).ejbJars.get(0).enterpriseBeans.get(0).ejbName);
     }
@@ -123,7 +127,9 @@ public class DeploymentsElementTest extends Assert {
         final Server server = new Server();
         final File apps = server.deploymentsDir("myapps");
 
-        final File ear = new File(server.getBase(), "colors.ear");
+        final String appName = "deploymentsDir_Ear_Unpacked_ApplicationXml";
+
+        final File ear = new File(server.getBase(), appName + ".ear");
         {
             final Map<String, Object> contents = new HashMap<String, Object>();
             contents.put("foo.jar", Archives.jarArchive(Orange.class));
@@ -131,12 +137,12 @@ public class DeploymentsElementTest extends Assert {
             Archives.jarArchive(ear, contents);
         }
 
-        Zips.unzip(ear, Files.mkdir(apps, "colors"));
+        Zips.unzip(ear, Files.mkdir(apps, appName));
 
         final OpenEjbConfiguration configuration = server.init();
 
         assertEquals(1, configuration.containerSystem.applications.size());
-        assertEquals(2, configuration.containerSystem.applications.get(0).ejbJars.size());
+        assertEquals(1, configuration.containerSystem.applications.get(0).ejbJars.size());
         assertEquals(1, configuration.containerSystem.applications.get(0).ejbJars.get(0).enterpriseBeans.size());
         assertEquals("Orange", configuration.containerSystem.applications.get(0).ejbJars.get(0).enterpriseBeans.get(0).ejbName);
     }
@@ -153,19 +159,21 @@ public class DeploymentsElementTest extends Assert {
         final Server server = new Server();
         final File apps = server.deploymentsDir("myapps");
 
-        final File ear = new File(server.getBase(), "colors.ear");
+        final String appName = "deploymentsDir_Ear_Unpacked_NoApplicationXml";
+
+        final File ear = new File(server.getBase(), appName + ".ear");
         {
             final Map<String, Object> contents = new HashMap<String, Object>();
             contents.put("foo.jar", Archives.jarArchive(Orange.class));
             Archives.jarArchive(ear, contents);
         }
 
-        Zips.unzip(ear, Files.mkdir(apps, "colors"));
+        Zips.unzip(ear, Files.mkdir(apps, appName));
 
         final OpenEjbConfiguration configuration = server.init();
 
         assertEquals(1, configuration.containerSystem.applications.size());
-        assertEquals(2, configuration.containerSystem.applications.get(0).ejbJars.size());
+        assertEquals(1, configuration.containerSystem.applications.get(0).ejbJars.size());
         assertEquals(1, configuration.containerSystem.applications.get(0).ejbJars.get(0).enterpriseBeans.size());
         assertEquals("Orange", configuration.containerSystem.applications.get(0).ejbJars.get(0).enterpriseBeans.get(0).ejbName);
     }
@@ -184,7 +192,9 @@ public class DeploymentsElementTest extends Assert {
         final Server server = new Server();
         final File apps = server.deploymentsDir("myapps");
 
-        final File ear = new File(server.getBase(), "colors.ear");
+        final String appName = "deploymentsDir_Ear_Packed_Unpacked_ApplicationXml";
+
+        final File ear = new File(server.getBase(), appName + ".ear");
         {
             final Map<String, Object> contents = new HashMap<String, Object>();
             contents.put("foo.jar", Archives.jarArchive(Orange.class));
@@ -192,13 +202,13 @@ public class DeploymentsElementTest extends Assert {
             Archives.jarArchive(ear, contents);
         }
 
-        IO.copy(ear, Files.path(apps, "colors.ear"));  // packed
-        Zips.unzip(ear, Files.mkdir(apps, "colors"));  // unpacked
+        IO.copy(ear, Files.path(apps, appName + ".ear"));  // packed
+        Zips.unzip(ear, Files.mkdir(apps, appName));  // unpacked
 
         final OpenEjbConfiguration configuration = server.init();
 
         assertEquals(1, configuration.containerSystem.applications.size());
-        assertEquals(2, configuration.containerSystem.applications.get(0).ejbJars.size());
+        assertEquals(1, configuration.containerSystem.applications.get(0).ejbJars.size());
         assertEquals(1, configuration.containerSystem.applications.get(0).ejbJars.get(0).enterpriseBeans.size());
         assertEquals("Orange", configuration.containerSystem.applications.get(0).ejbJars.get(0).enterpriseBeans.get(0).ejbName);
     }
@@ -216,20 +226,22 @@ public class DeploymentsElementTest extends Assert {
         final Server server = new Server();
         final File apps = server.deploymentsDir("myapps");
 
-        final File ear = new File(server.getBase(), "colors.ear");
+        final String appName = "deploymentsDir_Ear_Packed_Unpacked_NoApplicationXml";
+
+        final File ear = new File(server.getBase(), appName + ".ear");
         {
             final Map<String, Object> contents = new HashMap<String, Object>();
             contents.put("foo.jar", Archives.jarArchive(Orange.class));
             Archives.jarArchive(ear, contents);
         }
 
-        IO.copy(ear, Files.path(apps, "colors.ear"));  // packed
-        Zips.unzip(ear, Files.mkdir(apps, "colors"));  // unpacked
+        IO.copy(ear, Files.path(apps, appName + ".ear"));  // packed
+        Zips.unzip(ear, Files.mkdir(apps, appName));  // unpacked
 
         final OpenEjbConfiguration configuration = server.init();
 
         assertEquals(1, configuration.containerSystem.applications.size());
-        assertEquals(2, configuration.containerSystem.applications.get(0).ejbJars.size());
+        assertEquals(1, configuration.containerSystem.applications.get(0).ejbJars.size());
         assertEquals(1, configuration.containerSystem.applications.get(0).ejbJars.get(0).enterpriseBeans.size());
         assertEquals("Orange", configuration.containerSystem.applications.get(0).ejbJars.get(0).enterpriseBeans.get(0).ejbName);
     }
@@ -250,32 +262,34 @@ public class DeploymentsElementTest extends Assert {
         final Server server = new Server();
         final File apps = server.deploymentsDir("myapps");
 
+        final String appName = "deploymentsDir_Ear_Packed_Unpacked_Modified";
+
         { // Unpacked version -- Orange
 
-            final File ear = new File(server.getBase(), "colors.ear");
+            final File ear = new File(server.getBase(), appName + ".ear");
             final Map<String, Object> contents = new HashMap<String, Object>();
             contents.put("foo.jar", Archives.jarArchive(Orange.class));
             contents.put("META-INF/application.xml", "<application><module><ejb>foo.jar</ejb></module></application>");
             Archives.jarArchive(ear, contents);
-            Zips.unzip(ear, Files.mkdir(apps, "colors"));  // unpacked
+            Zips.unzip(ear, Files.mkdir(apps, appName));  // unpacked
         }
 
         Thread.sleep(100);
 
         { // Packed version -- Yellow
 
-            final File ear = new File(server.getBase(), "colors.ear");
+            final File ear = new File(server.getBase(), appName + ".ear");
             final Map<String, Object> contents = new HashMap<String, Object>();
             contents.put("foo.jar", Archives.jarArchive(Yellow.class));
             contents.put("META-INF/application.xml", "<application><module><ejb>foo.jar</ejb></module></application>");
             Archives.jarArchive(ear, contents);
-            Zips.unzip(ear, Files.mkdir(apps, "colors"));  // unpacked
+            Zips.unzip(ear, Files.mkdir(apps, appName));  // unpacked
         }
 
         final OpenEjbConfiguration configuration = server.init();
 
         assertEquals(1, configuration.containerSystem.applications.size());
-        assertEquals(2, configuration.containerSystem.applications.get(0).ejbJars.size());
+        assertEquals(1, configuration.containerSystem.applications.get(0).ejbJars.size());
         assertEquals(1, configuration.containerSystem.applications.get(0).ejbJars.get(0).enterpriseBeans.size());
         assertEquals("Yellow", configuration.containerSystem.applications.get(0).ejbJars.get(0).enterpriseBeans.get(0).ejbName);
     }


[2/2] git commit: Merge remote-tracking branch 'origin/master' into develop

Posted by an...@apache.org.
Merge remote-tracking branch 'origin/master' into develop


Project: http://git-wip-us.apache.org/repos/asf/tomee/repo
Commit: http://git-wip-us.apache.org/repos/asf/tomee/commit/736c3304
Tree: http://git-wip-us.apache.org/repos/asf/tomee/tree/736c3304
Diff: http://git-wip-us.apache.org/repos/asf/tomee/diff/736c3304

Branch: refs/heads/develop
Commit: 736c33041c37369f2fac51ce3da612f784f7550d
Parents: b12b59f 91e9840
Author: AndyGee <an...@gmx.de>
Authored: Tue Oct 28 10:11:20 2014 +0100
Committer: AndyGee <an...@gmx.de>
Committed: Tue Oct 28 10:11:20 2014 +0100

----------------------------------------------------------------------
 .../openejb/config/DeploymentsElementTest.java  | 46 +++++++++++++-------
 1 file changed, 30 insertions(+), 16 deletions(-)
----------------------------------------------------------------------