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

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

Author: kabir.khan at jboss.com
Date: 2008-01-22 08:43:05 -0500 (Tue, 22 Jan 2008)
New Revision: 69190

Modified:
   projects/aop/trunk/aop/src/test/org/jboss/test/aop/rebuildingchain/RebuildThread.java
   projects/aop/trunk/aop/src/test/org/jboss/test/aop/rebuildingchain/RebuildingChainTestCase.java
   projects/aop/trunk/aop/src/test/org/jboss/test/aop/rebuildingchain/SyncInterceptor.java
   projects/aop/trunk/aop/src/test/org/jboss/test/aop/rebuildingchain/SyncThread.java
Log:
Improve test

Modified: projects/aop/trunk/aop/src/test/org/jboss/test/aop/rebuildingchain/RebuildThread.java
===================================================================
--- projects/aop/trunk/aop/src/test/org/jboss/test/aop/rebuildingchain/RebuildThread.java	2008-01-22 11:37:42 UTC (rev 69189)
+++ projects/aop/trunk/aop/src/test/org/jboss/test/aop/rebuildingchain/RebuildThread.java	2008-01-22 13:43:05 UTC (rev 69190)
@@ -35,15 +35,13 @@
 {
    private static volatile boolean done = false;
 
-   private int id = 0;
-   
    @Override
    public void run()
    {
       for(int i=0; i < 30; i++)
       {
-         linkNewAdvice();
-         unlinkAdvice();
+         linkNewAdvice("Test" + i);
+         unlinkAdvice("Test" + i);
 
          if(isDone())
             return;
@@ -63,7 +61,12 @@
    
    public void linkNewAdvice()
    {
-//      System.out.println("adding new advice");
+      linkNewAdvice("Base");
+   }
+    
+   private void linkNewAdvice(String name)
+   {
+     System.out.println("adding new advice" + name);
       AdviceBinding binding1 = null;
       try
       {
@@ -74,15 +77,14 @@
          e.printStackTrace();
       }
       binding1.addInterceptor(SyncInterceptor.class);
-      binding1.setName(Integer.toString(id));
+      binding1.setName(name);
       AspectManager.instance().addBinding(binding1);
-      id++;
    }
    
-   public void unlinkAdvice()
+   private void unlinkAdvice(String name)
    {
-//      System.out.println("unlinking "+(id-1)); 
-      AspectManager.instance().removeBinding(Integer.toString((id-1)));
+      System.out.println("unlinking " + name); 
+      AspectManager.instance().removeBinding(name);
    }
   
 }

Modified: projects/aop/trunk/aop/src/test/org/jboss/test/aop/rebuildingchain/RebuildingChainTestCase.java
===================================================================
--- projects/aop/trunk/aop/src/test/org/jboss/test/aop/rebuildingchain/RebuildingChainTestCase.java	2008-01-22 11:37:42 UTC (rev 69189)
+++ projects/aop/trunk/aop/src/test/org/jboss/test/aop/rebuildingchain/RebuildingChainTestCase.java	2008-01-22 13:43:05 UTC (rev 69190)
@@ -23,6 +23,7 @@
 
 import junit.framework.Test;
 import junit.framework.TestSuite;
+import junit.textui.TestRunner;
 
 import org.jboss.aop.AspectManager;
 import org.jboss.test.aop.AOPTestWithSetup;
@@ -83,4 +84,17 @@
    {
       failed = true;
    }
+   
+   public static void main(String[] args)throws Exception
+   {
+      try
+      {
+         RebuildingChainTestCase test = new RebuildingChainTestCase("XXX");
+         test.testRebuildingChain();
+      }
+      catch (RuntimeException e)
+      {
+         System.err.println(e);
+      }
+   }
 }

Modified: projects/aop/trunk/aop/src/test/org/jboss/test/aop/rebuildingchain/SyncInterceptor.java
===================================================================
--- projects/aop/trunk/aop/src/test/org/jboss/test/aop/rebuildingchain/SyncInterceptor.java	2008-01-22 11:37:42 UTC (rev 69189)
+++ projects/aop/trunk/aop/src/test/org/jboss/test/aop/rebuildingchain/SyncInterceptor.java	2008-01-22 13:43:05 UTC (rev 69190)
@@ -44,6 +44,7 @@
    {
       try
       {
+         System.out.println("SyncInterceptor");
          SyncThread.setStatus(true);
          return invocation.invokeNext();
       }

Modified: projects/aop/trunk/aop/src/test/org/jboss/test/aop/rebuildingchain/SyncThread.java
===================================================================
--- projects/aop/trunk/aop/src/test/org/jboss/test/aop/rebuildingchain/SyncThread.java	2008-01-22 11:37:42 UTC (rev 69189)
+++ projects/aop/trunk/aop/src/test/org/jboss/test/aop/rebuildingchain/SyncThread.java	2008-01-22 13:43:05 UTC (rev 69190)
@@ -48,7 +48,7 @@
    
    private void checkStatus()
    {
-//      System.out.println("checking status...");
+      System.out.println("ST checking status...");
       if(getStatus() == false)
       {
          RebuildingChainTestCase.setTestFailed();