You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@geronimo.apache.org by dj...@apache.org on 2004/09/16 04:29:49 UTC

svn commit: rev 46145 - geronimo/trunk/modules/maven-itest-plugin

Author: djencks
Date: Wed Sep 15 19:29:49 2004
New Revision: 46145

Modified:
   geronimo/trunk/modules/maven-itest-plugin/plugin.jelly
Log:
Fix tests to actually work for maven.itest.skip

Modified: geronimo/trunk/modules/maven-itest-plugin/plugin.jelly
==============================================================================
--- geronimo/trunk/modules/maven-itest-plugin/plugin.jelly	(original)
+++ geronimo/trunk/modules/maven-itest-plugin/plugin.jelly	Wed Sep 15 19:29:49 2004
@@ -24,7 +24,7 @@
         description="Compile the Integration Tests"
         prereqs="java:compile,java:jar-resources,itest:prepare-filesystem,itest:test-resources">
 
-        <j:if test="${maven.itest.skip != 'true'}">
+        <j:if test="${context.getVariable('maven.itest.skip') != 'true'}">
 
             <javac
                 destdir="${maven.itest.dest}"
@@ -98,7 +98,7 @@
         description="Run Application Integration Tests"
         prereqs="itest:compile">
 
-        <j:if test="${maven.itest.skip != 'true'}">
+        <j:if test="${context.getVariable('maven.itest.skip') != 'true'}">
 
             <!-- call our setup code -->
             <attainGoal name="itest:setup"/>
@@ -183,7 +183,8 @@
         description="Copy any resources that must be present for the integration tests to run"
         prereqs="itest:prepare-filesystem">
 
-        <j:if test="${maven.itest.skip != 'true' and context.getVariable('maven.itest.resources') != ''}">
+        <j:if test="${context.getVariable('maven.itest.skip') != 'true'
+                     and context.getVariable('maven.itest.resources') != ''}">
             <ant:copy todir="${maven.itest.dest}">
                 <u:tokenize var="resources" delim=", ">${maven.itest.resources}</u:tokenize>
                 <j:forEach var="resource" items="${resources}">