[jboss-cvs] JBossAS SVN: r59528 - in projects/aop/trunk/aop/src: test/org/jboss/test/aop/field and 1 other directory.

[jboss-cvs-commits at lists.jboss.org]

Author: kabir.khan at jboss.com
Date: 2007-01-11 09:13:58 -0500 (Thu, 11 Jan 2007)
New Revision: 59528

Modified:
   projects/aop/trunk/aop/src/resources/test/field/jboss-aop.xml
   projects/aop/trunk/aop/src/test/org/jboss/test/aop/field/FieldTestCase.java
   projects/aop/trunk/aop/src/test/org/jboss/test/aop/field/POJO.java
Log:
Test superclass private fields get intercepted

Modified: projects/aop/trunk/aop/src/resources/test/field/jboss-aop.xml
===================================================================
--- projects/aop/trunk/aop/src/resources/test/field/jboss-aop.xml	2007-01-11 14:11:28 UTC (rev 59527)
+++ projects/aop/trunk/aop/src/resources/test/field/jboss-aop.xml	2007-01-11 14:13:58 UTC (rev 59528)
@@ -15,7 +15,11 @@
   <bind pointcut="field(* org.jboss.test.aop.field.SubSubPOJO->mine)">                
        <interceptor class="org.jboss.test.aop.field.TraceInterceptor"/>
   </bind>
+  <bind pointcut="field(* org.jboss.test.aop.field.POJO->privateField)">                
+       <interceptor class="org.jboss.test.aop.field.TraceInterceptor"/>
+  </bind>
 
+
   <interceptor name="field" class="org.jboss.test.aop.field.FieldPerJoinpointInterceptor" scope="PER_JOINPOINT"/>
   <interceptor name="staticField" class="org.jboss.test.aop.field.FieldPerJoinpointInterceptor" scope="PER_CLASS_JOINPOINT"/>
   <bind pointcut="field(* org.jboss.test.aop.field.ScopedPojo->field*)">                

Modified: projects/aop/trunk/aop/src/test/org/jboss/test/aop/field/FieldTestCase.java
===================================================================
--- projects/aop/trunk/aop/src/test/org/jboss/test/aop/field/FieldTestCase.java	2007-01-11 14:11:28 UTC (rev 59527)
+++ projects/aop/trunk/aop/src/test/org/jboss/test/aop/field/FieldTestCase.java	2007-01-11 14:13:58 UTC (rev 59528)
@@ -22,7 +22,6 @@
 package org.jboss.test.aop.field;
 
 import java.lang.reflect.Field;
-import java.lang.reflect.Method;
 
 import junit.framework.Test;
 import junit.framework.TestSuite;
@@ -99,6 +98,29 @@
       assertTrue(TraceInterceptor.intercepted);
    }
 
+   public void testSuperPrivateField()
+   {
+      POJO pojo = new POJO();
+
+      TraceInterceptor.intercepted = false;
+      pojo.setPrivate(5);
+      assertTrue(TraceInterceptor.intercepted);
+
+      TraceInterceptor.intercepted = false;
+      assertEquals(5, pojo.getPrivate());
+      assertTrue(TraceInterceptor.intercepted);
+
+      SubPOJO sub = new SubPOJO();
+
+      TraceInterceptor.intercepted = false;
+      sub.setPrivate(5);
+      assertTrue(TraceInterceptor.intercepted);
+
+      TraceInterceptor.intercepted = false;
+      assertEquals(5, sub.getPrivate());
+      assertTrue(TraceInterceptor.intercepted);
+   }
+   
    public void testPerJoinpoint() throws Exception
    {
       ScopedPojo pojo1 = new ScopedPojo();

Modified: projects/aop/trunk/aop/src/test/org/jboss/test/aop/field/POJO.java
===================================================================
--- projects/aop/trunk/aop/src/test/org/jboss/test/aop/field/POJO.java	2007-01-11 14:11:28 UTC (rev 59527)
+++ projects/aop/trunk/aop/src/test/org/jboss/test/aop/field/POJO.java	2007-01-11 14:13:58 UTC (rev 59528)
@@ -30,6 +30,7 @@
 {
    int field;
    public int pojoInherited;
+   private int privateField;
    
    public POJO() {}
    
@@ -42,5 +43,14 @@
    {
       return field;
    }
- 
+   
+   public void setPrivate(int i)
+   {
+      privateField = i;
+   }
+   
+   public int getPrivate()
+   {
+      return privateField;
+   }
 }