[jboss-cvs] JBossAS SVN: r67479 - in projects/metadata/trunk/src: test/java/org/jboss/test/metadata/annotation/ejb3/runas and 1 other directory.

jboss-cvs-commits at lists.jboss.org jboss-cvs-commits at lists.jboss.org
Tue Nov 27 01:48:05 EST 2007


Author: scott.stark at jboss.org
Date: 2007-11-27 01:48:05 -0500 (Tue, 27 Nov 2007)
New Revision: 67479

Modified:
   projects/metadata/trunk/src/main/java/org/jboss/metadata/ejb/jboss/JBossMetaData.java
   projects/metadata/trunk/src/test/java/org/jboss/test/metadata/annotation/ejb3/runas/RunAsUnitTestCase.java
Log:
Change the test to properly validate xml parsing followed by a merge with an annotation based model

Modified: projects/metadata/trunk/src/main/java/org/jboss/metadata/ejb/jboss/JBossMetaData.java
===================================================================
--- projects/metadata/trunk/src/main/java/org/jboss/metadata/ejb/jboss/JBossMetaData.java	2007-11-27 06:44:34 UTC (rev 67478)
+++ projects/metadata/trunk/src/main/java/org/jboss/metadata/ejb/jboss/JBossMetaData.java	2007-11-27 06:48:05 UTC (rev 67479)
@@ -721,7 +721,9 @@
                // Update the run-as indentity to use the principal name
                if (ejbXmlRunAs == null)
                {
-                  throw new IllegalStateException("run-as-principal: " + principalName + " found in jboss.xml but there was no run-as in ejb-jar.xml");
+                  // Can't make this check since the name may come from an annotation
+                  //throw new IllegalStateException("run-as-principal: " + principalName + " found in jboss.xml but there was no run-as in ejb-jar.xml");
+                  ejbXmlRunAs = "anonymous";
                }
                // See if there are any additional roles for this principal
                Set<String> extraRoles = null;

Modified: projects/metadata/trunk/src/test/java/org/jboss/test/metadata/annotation/ejb3/runas/RunAsUnitTestCase.java
===================================================================
--- projects/metadata/trunk/src/test/java/org/jboss/test/metadata/annotation/ejb3/runas/RunAsUnitTestCase.java	2007-11-27 06:44:34 UTC (rev 67478)
+++ projects/metadata/trunk/src/test/java/org/jboss/test/metadata/annotation/ejb3/runas/RunAsUnitTestCase.java	2007-11-27 06:48:05 UTC (rev 67479)
@@ -79,7 +79,7 @@
    }
 
    /**
-    * 
+    * Validate 
     * @throws Exception
     */
    public void testXmlAnnotationMerge()
@@ -95,11 +95,15 @@
       EjbJarMetaData specMetaData = unmarshal("ejb-jar.xml", EjbJarMetaData.class, null);
       EjbJarMetaData specMerged = new EjbJar30MetaData();
       AnnotationMergedView.merge(specMerged, specMetaData, metaData);
-   
+
+      // First parse the jboss xml and merge with the spec xml
       JBossMetaData xmlMetaData = unmarshal("jboss.xml", JBossMetaData.class, null);
       JBossMetaData mergedData = new JBossMetaData();
-      mergedData.merge(xmlMetaData, specMerged);
+      mergedData.merge(xmlMetaData, specMetaData);
 
+      // Now merge with the annotation data
+      mergedData.merge(null, specMerged);
+
       // Validate the proxy view
       JBossEnterpriseBeanMetaData InterMediateBean = mergedData.getEnterpriseBean("InterMediateBean");
       assertNotNull(InterMediateBean);




More information about the jboss-cvs-commits mailing list