[jboss-cvs] JBossAS SVN: r84104 - in branches/Branch_AOP_1_5/aop/src: resources/test and 3 other directories.

jboss-cvs-commits at lists.jboss.org jboss-cvs-commits at lists.jboss.org
Wed Feb 11 16:23:00 EST 2009


Author: stalep
Date: 2009-02-11 16:23:00 -0500 (Wed, 11 Feb 2009)
New Revision: 84104

Added:
   branches/Branch_AOP_1_5/aop/src/resources/test/standalone/
   branches/Branch_AOP_1_5/aop/src/resources/test/standalone/jboss-aop.xml
   branches/Branch_AOP_1_5/aop/src/test/org/jboss/test/aop/standalone/
   branches/Branch_AOP_1_5/aop/src/test/org/jboss/test/aop/standalone/POJO.java
   branches/Branch_AOP_1_5/aop/src/test/org/jboss/test/aop/standalone/PackageInterceptor.java
   branches/Branch_AOP_1_5/aop/src/test/org/jboss/test/aop/standalone/PackageTestCase.java
Modified:
   branches/Branch_AOP_1_5/aop/src/main/org/jboss/aop/standalone/Package.java
Log:
[JBAOP-697]
Changed Package to use the correct AM.


Modified: branches/Branch_AOP_1_5/aop/src/main/org/jboss/aop/standalone/Package.java
===================================================================
--- branches/Branch_AOP_1_5/aop/src/main/org/jboss/aop/standalone/Package.java	2009-02-11 21:20:22 UTC (rev 84103)
+++ branches/Branch_AOP_1_5/aop/src/main/org/jboss/aop/standalone/Package.java	2009-02-11 21:23:00 UTC (rev 84104)
@@ -47,9 +47,9 @@
       this.name = name;
    }
 
-   static void parse(Class clazz, Package root)
+   static void parse(AspectManager manager, Class clazz, Package root)
    {
-      Advisor advisor = AspectManager.instance().findAdvisor(clazz);
+      Advisor advisor = manager.findAdvisor(clazz);
       StringTokenizer tokenizer = new StringTokenizer(clazz.getName(), ".");
       while (tokenizer.hasMoreTokens())
       {
@@ -84,7 +84,7 @@
       while (it.hasNext())
       {
          Class clazz = (Class) it.next();
-         parse(clazz, root);
+         parse(manager, clazz, root);
       }
       return root;
    }

Added: branches/Branch_AOP_1_5/aop/src/resources/test/standalone/jboss-aop.xml
===================================================================
--- branches/Branch_AOP_1_5/aop/src/resources/test/standalone/jboss-aop.xml	                        (rev 0)
+++ branches/Branch_AOP_1_5/aop/src/resources/test/standalone/jboss-aop.xml	2009-02-11 21:23:00 UTC (rev 84104)
@@ -0,0 +1,6 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<aop>
+  <bind pointcut="execution(* org.jboss.test.aop.standalone.POJO->foo())">
+    <interceptor class="org.jboss.test.aop.standalone.PackageInterceptor"/>
+  </bind>
+</aop>

Added: branches/Branch_AOP_1_5/aop/src/test/org/jboss/test/aop/standalone/POJO.java
===================================================================
--- branches/Branch_AOP_1_5/aop/src/test/org/jboss/test/aop/standalone/POJO.java	                        (rev 0)
+++ branches/Branch_AOP_1_5/aop/src/test/org/jboss/test/aop/standalone/POJO.java	2009-02-11 21:23:00 UTC (rev 84104)
@@ -0,0 +1,38 @@
+/*
+  * JBoss, Home of Professional Open Source
+  * Copyright 2005, JBoss Inc., and individual contributors as indicated
+  * by the @authors tag. See the copyright.txt in the distribution for a
+  * full listing of individual contributors.
+  *
+  * This is free software; you can redistribute it and/or modify it
+  * under the terms of the GNU Lesser General Public License as
+  * published by the Free Software Foundation; either version 2.1 of
+  * the License, or (at your option) any later version.
+  *
+  * This software is distributed in the hope that it will be useful,
+  * but WITHOUT ANY WARRANTY; without even the implied warranty of
+  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+  * Lesser General Public License for more details.
+  *
+  * You should have received a copy of the GNU Lesser General Public
+  * License along with this software; if not, write to the Free
+  * Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA
+  * 02110-1301 USA, or see the FSF site: http://www.fsf.org.
+  */
+package org.jboss.test.aop.standalone;
+
+/**
+ * A POJO.
+ * 
+ * @author <a href="mailto:stale.pedersen at jboss.org">Stale W. Pedersen</a>
+ * @version $Revision: 1.1 $
+ */
+public class POJO
+{
+   
+   public void foo()
+   {
+      
+   }
+
+}

Added: branches/Branch_AOP_1_5/aop/src/test/org/jboss/test/aop/standalone/PackageInterceptor.java
===================================================================
--- branches/Branch_AOP_1_5/aop/src/test/org/jboss/test/aop/standalone/PackageInterceptor.java	                        (rev 0)
+++ branches/Branch_AOP_1_5/aop/src/test/org/jboss/test/aop/standalone/PackageInterceptor.java	2009-02-11 21:23:00 UTC (rev 84104)
@@ -0,0 +1,48 @@
+/*
+  * JBoss, Home of Professional Open Source
+  * Copyright 2005, JBoss Inc., and individual contributors as indicated
+  * by the @authors tag. See the copyright.txt in the distribution for a
+  * full listing of individual contributors.
+  *
+  * This is free software; you can redistribute it and/or modify it
+  * under the terms of the GNU Lesser General Public License as
+  * published by the Free Software Foundation; either version 2.1 of
+  * the License, or (at your option) any later version.
+  *
+  * This software is distributed in the hope that it will be useful,
+  * but WITHOUT ANY WARRANTY; without even the implied warranty of
+  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+  * Lesser General Public License for more details.
+  *
+  * You should have received a copy of the GNU Lesser General Public
+  * License along with this software; if not, write to the Free
+  * Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA
+  * 02110-1301 USA, or see the FSF site: http://www.fsf.org.
+  */
+package org.jboss.test.aop.standalone;
+
+import org.jboss.aop.advice.Interceptor;
+import org.jboss.aop.joinpoint.Invocation;
+
+/**
+ * A PackageInterceptor.
+ * 
+ * @author <a href="mailto:stale.pedersen at jboss.org">Stale W. Pedersen</a>
+ * @version $Revision: 1.1 $
+ */
+public class PackageInterceptor implements Interceptor
+{
+   public static boolean invoked = false;
+   
+   public String getName()
+   {
+      return "PackageInterceptor";
+   }
+
+   public Object invoke(Invocation invocation) throws Throwable
+   {
+      invoked = true; 
+      return invocation.invokeNext();
+   }
+
+}

Added: branches/Branch_AOP_1_5/aop/src/test/org/jboss/test/aop/standalone/PackageTestCase.java
===================================================================
--- branches/Branch_AOP_1_5/aop/src/test/org/jboss/test/aop/standalone/PackageTestCase.java	                        (rev 0)
+++ branches/Branch_AOP_1_5/aop/src/test/org/jboss/test/aop/standalone/PackageTestCase.java	2009-02-11 21:23:00 UTC (rev 84104)
@@ -0,0 +1,77 @@
+/*
+  * JBoss, Home of Professional Open Source
+  * Copyright 2005, JBoss Inc., and individual contributors as indicated
+  * by the @authors tag. See the copyright.txt in the distribution for a
+  * full listing of individual contributors.
+  *
+  * This is free software; you can redistribute it and/or modify it
+  * under the terms of the GNU Lesser General Public License as
+  * published by the Free Software Foundation; either version 2.1 of
+  * the License, or (at your option) any later version.
+  *
+  * This software is distributed in the hope that it will be useful,
+  * but WITHOUT ANY WARRANTY; without even the implied warranty of
+  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+  * Lesser General Public License for more details.
+  *
+  * You should have received a copy of the GNU Lesser General Public
+  * License along with this software; if not, write to the Free
+  * Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA
+  * 02110-1301 USA, or see the FSF site: http://www.fsf.org.
+  */
+package org.jboss.test.aop.standalone;
+
+import java.lang.ref.WeakReference;
+import java.util.Map;
+
+import junit.framework.Test;
+import junit.framework.TestCase;
+import junit.framework.TestSuite;
+
+import org.jboss.aop.AspectManager;
+import org.jboss.aop.Domain;
+
+/**
+ * A PackageTestCase.
+ * 
+ * @author <a href="mailto:stale.pedersen at jboss.org">Stale W. Pedersen</a>
+ * @version $Revision: 1.1 $
+ */
+public class PackageTestCase extends TestCase
+{
+   
+   public PackageTestCase(String name)
+   {
+      super(name);
+   }
+
+   public static Test suite()
+   {
+      TestSuite suite = new TestSuite("PackageTestCase");
+      suite.addTestSuite(PackageTestCase.class);
+      return suite;
+   }
+   
+   public void testPackage()
+   {
+      POJO pojo = new POJO();
+      pojo.foo();
+//      assertTrue(PackageInterceptor.invoked);
+      
+      org.jboss.aop.standalone.Package pkg = org.jboss.aop.standalone.Package.aopClassMap();
+      System.out.println("Package size: "+pkg.packages.size());
+      assertEquals(1,pkg.packages.size());
+      
+//      System.out.println("Number of advisors: "+AspectManager.getTopLevelAspectManager().getAdvisors().size());
+//      System.out.println("Number of advisors: "+AspectManager.instance().getAdvisors().size());
+//      System.out.println("Number of advisors: "+AspectManager.instance().getSubDomainsPerClass2().size());
+//      
+//      Map<Class<?>, WeakReference<Domain>> domains = AspectManager.instance().getSubDomainsPerClass2();
+//      
+//      System.out.println("domains: "+domains.toString());
+      
+      
+      System.out.println("Number of aspects: "+AspectManager.instance().getAspectDefinitions().size());
+      
+   }
+}




More information about the jboss-cvs-commits mailing list