Bommels05/maven-plugins
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

PR: MPTEST-63, MPTEST-64

Browse Source 
Fix memory leak and performance degradation in test plugin 1.8.

git-svn-id: https://svn.apache.org/repos/asf/maven/maven-1/plugins/trunk@538917 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
ltheussl
2007-05-17 13:38:22 +00:00
parent dd4327ce78
commit 5a9db69cd0
3 changed files with 22 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files

28
test/plugin.jelly
View File

@@ -64,9 +64,9 @@
<pathelement path="${plugin.getDependencyPath('junit')}"/>
<j:if test="${!context.getVariable('maven.junit.fork').toString().trim().equalsIgnoreCase('no') and !context.getVariable('maven.test.excludeXmlApis').toString().trim().equalsIgnoreCase('yes') }">
<pathelement location="${plugin.getDependencyPath('xml-apis:xml-apis')}"/>
<pathelement location="${plugin.getDependencyPath('xerces:xercesImpl')}"/>
<pathelement location="${plugin.getDependencyPath('xml-resolver:xml-resolver')}"/>
</j:if>
<pathelement path="${plugin.getDependencyPath('xerces:xercesImpl')}"/>
<pathelement location="${plugin.getDependencyPath('xml-resolver:xml-resolver')}"/>
</classpath>
<src>
<path refid="maven.test.compile.src.set"/>
@@ -186,10 +186,10 @@
<path refid="maven.dependency.classpath"/>
<pathelement path="${plugin.getDependencyPath('junit')}"/>
<j:if test="${!context.getVariable('maven.junit.fork').toString().trim().equalsIgnoreCase('no') and !context.getVariable('maven.test.excludeXmlApis').toString().trim().equalsIgnoreCase('yes') }">
<pathelement path="${plugin.getDependencyPath('xml-apis:xml-apis')}"/>
<pathelement location="${plugin.getDependencyPath('xml-apis:xml-apis')}"/>
<pathelement location="${plugin.getDependencyPath('xerces:xercesImpl')}"/>
<pathelement location="${plugin.getDependencyPath('xml-resolver:xml-resolver')}"/>
</j:if>
<pathelement path="${plugin.getDependencyPath('xerces:xercesImpl')}"/>
<pathelement path="${plugin.getDependencyPath('xml-resolver:xml-resolver')}"/>
<j:if test="${not empty(context.getVariable('maven.test.classpath'))}">
<pathelement path="${maven.test.classpath}"/>
</j:if>
@@ -256,10 +256,10 @@
<path refid="maven.dependency.classpath"/>
<pathelement path="${plugin.getDependencyPath('junit')}"/>
<j:if test="${!context.getVariable('maven.junit.fork').toString().trim().equalsIgnoreCase('no') and !context.getVariable('maven.test.excludeXmlApis').toString().trim().equalsIgnoreCase('yes') }">
<pathelement path="${plugin.getDependencyPath('xml-apis:xml-apis')}"/>
<pathelement location="${plugin.getDependencyPath('xml-apis:xml-apis')}"/>
<pathelement location="${plugin.getDependencyPath('xerces:xercesImpl')}"/>
<pathelement location="${plugin.getDependencyPath('xml-resolver:xml-resolver')}"/>
</j:if>
<pathelement path="${plugin.getDependencyPath('xerces:xercesImpl')}"/>
<pathelement path="${plugin.getDependencyPath('xml-resolver:xml-resolver')}"/>
<j:if test="${not empty(context.getVariable('maven.test.classpath'))}">
<pathelement path="${maven.test.classpath}"/>
</j:if>
@@ -342,10 +342,10 @@
<path refid="maven.dependency.classpath"/>
<pathelement path="${plugin.getDependencyPath('junit')}"/>
<j:if test="${!context.getVariable('maven.junit.fork').toString().trim().equalsIgnoreCase('no') and !context.getVariable('maven.test.excludeXmlApis').toString().trim().equalsIgnoreCase('yes') }">
<pathelement path="${plugin.getDependencyPath('xml-apis:xml-apis')}"/>
<pathelement location="${plugin.getDependencyPath('xml-apis:xml-apis')}"/>
<pathelement location="${plugin.getDependencyPath('xerces:xercesImpl')}"/>
<pathelement location="${plugin.getDependencyPath('xml-resolver:xml-resolver')}"/>
</j:if>
<pathelement path="${plugin.getDependencyPath('xerces:xercesImpl')}"/>
<pathelement path="${plugin.getDependencyPath('xml-resolver:xml-resolver')}"/>
<j:if test="${not empty(context.getVariable('maven.test.classpath'))}">
<pathelement path="${maven.test.classpath}"/>
</j:if>
@@ -454,10 +454,10 @@
<path refid="maven.dependency.classpath"/>
<pathelement path="${plugin.getDependencyPath('junit')}"/>
<j:if test="${!context.getVariable('maven.junit.fork').toString().trim().equalsIgnoreCase('no') and !context.getVariable('maven.test.excludeXmlApis').toString().trim().equalsIgnoreCase('yes') }">
<pathelement path="${plugin.getDependencyPath('xml-apis:xml-apis')}"/>
<pathelement location="${plugin.getDependencyPath('xml-apis:xml-apis')}"/>
<pathelement location="${plugin.getDependencyPath('xerces:xercesImpl')}"/>
<pathelement location="${plugin.getDependencyPath('xml-resolver:xml-resolver')}"/>
</j:if>
<pathelement path="${plugin.getDependencyPath('xerces:xercesImpl')}"/>
<pathelement path="${plugin.getDependencyPath('xml-resolver:xml-resolver')}"/>
<j:if test="${not empty(context.getVariable('maven.test.classpath'))}">
<pathelement path="${maven.test.classpath}"/>
</j:if>

2
test/project.xml
View File

@@ -23,7 +23,7 @@
<pomVersion>3</pomVersion>
<id>maven-test-plugin</id>
<name>Maven Test Plugin</name>
<currentVersion>1.8.1</currentVersion>
<currentVersion>1.8.2-SNAPSHOT</currentVersion>
<description>Run JUnit tests.</description>
<shortDescription>Run JUnit tests</shortDescription>
<versions>

7
test/xdocs/changes.xml
View File

@@ -24,6 +24,13 @@
<author email="dion@multitask.com.au">dIon Gillard</author>
</properties>
<body>
<release version="1.8.2-SNAPSHOT" date="in SVN">
<action dev="ltheussl" type="fix">
<description>Memory leak and performance degradation in test plugin 1.8.</description>
<fixes issue="MPTEST-63"/>
<fixes issue="MPTEST-64"/>
</action>
</release>
<release version="1.8.1" date="2007-05-07">
<action dev="ltheussl" type="update" issue="MPTEST-66">Document a backward-compat problem affecting version 1.8 of the plugin.</action>
<action dev="ltheussl" type="add" issue="MPTEST-71">New property to halt a build on test failures but after all tests have been run.</action>