Index: plugin.jelly
===================================================================
RCS file: /home/cvspublic/maven-plugins/aspectj/plugin.jelly,v
retrieving revision 1.12
diff -u -r1.12 plugin.jelly
--- plugin.jelly	3 Jun 2004 11:49:45 -0000	1.12
+++ plugin.jelly	3 Jun 2004 16:12:00 -0000
@@ -30,16 +30,85 @@
 
 <project
   xmlns:j="jelly:core"
-  xmlns:ant="jelly:ant">
+  xmlns:ant="jelly:ant"
+  xmlns:define="jelly:define"
+  xmlns:aspectj="aspectj">
 
   <!--
+  Taglib to compile aspects
+  
+  Parameters:
+    - sourcePathRefid
+    - classpathRefid
+    - destDir
+  -->
+  <define:taglib uri="aspectj">
+    <define:tag name="compile">
+
+      <ant:iajc
+          fork="${maven.aspectj.fork}"
+          incremental="${maven.aspectj.incremental}"
+          destDir="${destDir}"
+          sourceRootCopyFilter="${maven.aspectj.sourceRootCopyFilter}"
+          debug="${maven.aspectj.debug}"
+          emacssym="${maven.aspectj.emacssym}"
+          verbose="${maven.aspectj.verbose}">
+
+        <j:if test="${context.getVariable('maven.aspectj.fork') == true}">
+          <j:if test="${context.getVariable('maven.aspectj.maxmem') != null}">
+            <ant:setProperty name="maxmem" value="${maven.aspectj.maxmem}" />
+          </j:if>
+        </j:if>
+        
+        <ant:sourceroots>
+          <ant:path refid="${sourcePathRefid}"/>
+          <j:if test="${aspectSourcesPresent == 'true'}">
+            <ant:pathelement location="${pom.build.aspectSourceDirectory}"/>
+          </j:if>
+        </ant:sourceroots>
+
+        <ant:classpath>
+          <ant:path refid="${classpathRefid}"/>
+          <ant:pathelement path="${plugin.getDependencyPath('aspectj:aspectjrt')}"/>
+          <ant:pathelement path="${maven.build.dest}"/>
+        </ant:classpath>
+
+        <!-- Look for aspect libraries to use for weaving -->
+        <ant:aspectpath>
+          <j:forEach var="artifact" items="${pom.artifacts}">
+            <j:set var="dep" value="${artifact.dependency}"/>
+            <j:if test="${dep.getProperty('aspectj.weaveWith')=='true'}">
+              <ant:echo>Weaving with: ${dep.artifactId}</ant:echo>
+              <ant:pathelement location="${artifact.path}"/>
+            </j:if>
+          </j:forEach>
+        </ant:aspectpath>
+
+        <!-- Look for libraries for weaving into -->
+        <ant:inpath>
+          <j:forEach var="artifact" items="${pom.artifacts}">
+            <j:set var="dep" value="${artifact.dependency}"/>
+            <j:if test="${dep.getProperty('aspectj.weaveInto')=='true'}">
+              <ant:echo>Weaving into: ${dep.artifactId}</ant:echo>
+              <ant:pathelement location="${artifact.path}"/>
+            </j:if>
+          </j:forEach>
+        </ant:inpath>
+
+      </ant:iajc>
+      
+
+    </define:tag>
+
+  </define:taglib>
+  
+  <!--
     ========================================================================
     Default goal.
     ========================================================================
   -->
-  <goal name="aspectj" description="Weave with AspectJ">
-      <attainGoal name="aspectj:compile"/>
-  </goal>
+  <goal name="aspectj" description="Weave with AspectJ"
+    prereqs="aspectj:compile, aspectj:test-compile"/>
 
   <!--
     ========================================================================
@@ -87,64 +156,47 @@
 
   <!--
     ========================================================================
-    Weave aspect classes
+    Weave classes
     ========================================================================
   -->
   <goal name="aspectj:compile" prereqs="aspectj:init"
-    description="Weave with AspectJ">
+    description="Weave classes with AspectJ">
 
     <j:if test="${shouldWeave == 'true'}">
 
-      <ant:iajc
-          fork="${maven.aspectj.fork}"
-          maxmem="${maven.aspectj.maxmem}"
-          incremental="${maven.aspectj.incremental}"
-          destDir="${maven.build.dest}"
-          sourceRootCopyFilter="${maven.aspectj.sourceRootCopyFilter}"
-          debug="${maven.aspectj.debug}"
-          emacssym="${maven.aspectj.emacssym}"
-          verbose="${maven.aspectj.verbose}"
-          source="${maven.aspectj.source}"
-          time="${maven.aspectj.time}">
-
-        <ant:sourceroots>
-          <ant:path refid="maven.compile.src.set"/>
-          <j:if test="${aspectSourcesPresent == 'true'}">
-            <ant:pathelement location="${pom.build.aspectSourceDirectory}"/>
-          </j:if>
-        </ant:sourceroots>
+      <!-- Weave classes -->
+      <aspectj:compile
+        sourcePathRefid="maven.compile.src.set"
+        classpathRefid="maven.dependency.classpath"
+        destDir="${maven.build.dest}"/>
 
-        <ant:classpath>
-          <ant:path refid="maven.dependency.classpath"/>
-          <ant:pathelement path="${plugin.getDependencyPath('aspectj:aspectjrt')}"/>
-          <ant:pathelement path="${maven.build.dest}"/>
-        </ant:classpath>
+    </j:if>
 
-        <!-- Look for aspect libraries to use for weaving -->
-        <ant:aspectpath>
-          <j:forEach var="artifact" items="${pom.artifacts}">
-            <j:set var="dep" value="${artifact.dependency}"/>
-            <j:if test="${dep.getProperty('aspectj.weaveWith')=='true'}">
-              <ant:echo>Weaving with: ${dep.artifactId}</ant:echo>
-              <ant:pathelement location="${artifact.path}"/>
-            </j:if>
-          </j:forEach>
-        </ant:aspectpath>
+  </goal>
+  
+  <!--
+    ========================================================================
+    Weave test classes
+    ========================================================================
+  -->
+  <goal name="aspectj:test-compile" prereqs="aspectj:init"
+    description="Weave tests with AspectJ">
 
-        <!-- Look for libraries for weaving into -->
-        <ant:inpath>
-          <j:forEach var="artifact" items="${pom.artifacts}">
-            <j:set var="dep" value="${artifact.dependency}"/>
-            <j:if test="${dep.getProperty('aspectj.weaveInto')=='true'}">
-              <ant:echo>Weaving into: ${dep.artifactId}</ant:echo>
-              <ant:pathelement location="${artifact.path}"/>
-            </j:if>
-          </j:forEach>
-        </ant:inpath>
+    <j:if test="${shouldWeave == 'true'}">
 
-      </ant:iajc>
+      <!-- Weave test classes -->
+      <ant:path id="testClasspath">
+        <ant:path refid="maven.dependency.classpath"/>
+        <ant:pathelement path="${plugin.getDependencyPath('junit')}"/>
+      </ant:path>
+
+      <aspectj:compile
+        sourcePathRefid="maven.test.compile.src.set"
+        classpathRefid="testClasspath"
+        destDir="${maven.test.dest}"/>
 
     </j:if>
+
   </goal>
   
 </project>
Index: project.xml
===================================================================
RCS file: /home/cvspublic/maven-plugins/aspectj/project.xml,v
retrieving revision 1.36
diff -u -r1.36 project.xml
--- project.xml	3 Jun 2004 11:53:16 -0000	1.36
+++ project.xml	3 Jun 2004 16:12:01 -0000
@@ -100,5 +100,10 @@
       <artifactId>aspectjtools</artifactId>
       <version>1.2</version>
     </dependency>
+    <dependency>
+      <groupId>junit</groupId>
+      <artifactId>junit</artifactId>
+      <version>3.8.1</version>
+    </dependency>
   </dependencies>
 </project>
Index: xdocs/goals.xml
===================================================================
RCS file: /home/cvspublic/maven-plugins/aspectj/xdocs/goals.xml,v
retrieving revision 1.5
diff -u -r1.5 goals.xml
--- xdocs/goals.xml	7 May 2004 19:19:10 -0000	1.5
+++ xdocs/goals.xml	3 Jun 2004 16:12:01 -0000
@@ -32,13 +32,19 @@
         <name>aspectj</name>
         <description>
             This is the default goal of the plugin and simply attains
-            the <code>aspectj:compile</code> goal.
+            the <code>aspectj:compile</code> and <code>aspectj:test-compile</code> goals.
         </description>
       </goal>
       <goal>
         <name>aspectj:compile</name>
         <description>
             Weaves project classes and dependency libraries.
+        </description>
+      </goal>
+      <goal>
+        <name>aspectj:test-compile</name>
+        <description>
+            Weaves project test classes.
         </description>
       </goal>
     </goals>
Index: xdocs/index.xml
===================================================================
RCS file: /home/cvspublic/maven-plugins/aspectj/xdocs/index.xml,v
retrieving revision 1.4
diff -u -r1.4 index.xml
--- xdocs/index.xml	7 May 2004 19:19:10 -0000	1.4
+++ xdocs/index.xml	3 Jun 2004 16:12:01 -0000
@@ -45,6 +45,7 @@
         document.
       </p>
     </section>
+
     <section name="Installing">
       <p>
         To install or update the plugin do the following:<br/>
@@ -118,11 +128,16 @@
       add the following to your maven.xml file.
       </p>
       <source>
-          <![CDATA[
+         <![CDATA[
+           <!-- Compile classes -->
            <preGoal name="java:compile">
              <attainGoal name="aspectj:compile"/>
            </preGoal>
-          ]]>
+           <!-- Compile tests -->
+           <preGoal name="test:compile">
+             <attainGoal name="aspectj:test-compile"/>
+           </preGoal>
+         ]]>
       </source>
     </section>