[jboss-cvs] JBossAS SVN: r63954 - projects/aop/trunk/aop/src/test/org/jboss/test/aop/beforeafterArgs.

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

Author: flavia.rainone at jboss.com
Date: 2007-07-10 14:21:14 -0400 (Tue, 10 Jul 2007)
New Revision: 63954

Modified:
   projects/aop/trunk/aop/src/test/org/jboss/test/aop/beforeafterArgs/ArgAspectGenerics.java
Log:
[JBAOP-415] Tests enabled for method execution interception of the new feature.

Modified: projects/aop/trunk/aop/src/test/org/jboss/test/aop/beforeafterArgs/ArgAspectGenerics.java
===================================================================
--- projects/aop/trunk/aop/src/test/org/jboss/test/aop/beforeafterArgs/ArgAspectGenerics.java	2007-07-10 18:14:41 UTC (rev 63953)
+++ projects/aop/trunk/aop/src/test/org/jboss/test/aop/beforeafterArgs/ArgAspectGenerics.java	2007-07-10 18:21:14 UTC (rev 63954)
@@ -109,7 +109,7 @@
       before1 = true;
    }
    
-   public void before2(@Arg List<Object> arg)
+   public void before2(@Arg List<?> arg)
    {
       before2 = true;
    }
@@ -162,7 +162,7 @@
       CurrentInvocation.proceed();
    }
    
-   public void around4(@Arg List<Object> arg) throws Throwable
+   public void around4(@Arg List<? extends Object> arg) throws Throwable
    {
       around4 = true;
       CurrentInvocation.proceed();
@@ -170,7 +170,7 @@
    
    public void around5(@Arg List<Interface> arg) throws Throwable
    {
-      //Assert.fail("This advice should never be executed");
+      Assert.fail("This advice should never be executed");
       CurrentInvocation.proceed();
    }
    
@@ -195,7 +195,7 @@
       after1 = true;
    }
    
-   public void after2(@Arg List<Object> arg)
+   public <A> void after2(@Arg List<A> arg)
    {
       after2 = true;
    }
@@ -212,7 +212,7 @@
    
    public void after5(@Arg Collection<Object> arg)
    {
-      //Assert.fail("This advice should never be executed");
+      Assert.fail("This advice should never be executed");
    }
    
    public void after6(@Arg Collection arg)
@@ -237,7 +237,7 @@
    
    public void throwing2(@Arg Collection<SubValue> arg, @Thrown Throwable t)
    {
-      //Assert.fail("This advice should never be executed");
+      Assert.fail("This advice should never be executed");
    }
    
    public void throwing3(@Arg Collection arg, @Thrown Throwable t)
@@ -250,7 +250,7 @@
       throwing4 = true;
    }
    
-   public void throwing5(@Arg List<Object> arg, @Thrown Throwable t)
+   public void throwing5(@Arg List<?> arg, @Thrown Throwable t)
    {
       throwing5 = true;
    }
@@ -275,7 +275,7 @@
       finally1 = true;
    }
    
-   public void finally2(@Arg List<Object> arg)
+   public <A extends Object> void finally2(@Arg List<A> arg)
    {
       finally2 = true;
    }
@@ -292,7 +292,7 @@
    
    public void finally5(@Arg Collection<Interface> arg)
    {
-      //Assert.fail("This advice should never be executed");
+      Assert.fail("This advice should never be executed");
    }
    
    public void finally6(@Arg Collection arg)