On Fri, 26 Aug 2005 [EMAIL PROTECTED] wrote: The ordering of the link elements was not wrong, because the test passed here before I committed.
The problem is that it uses a HashMap and iterates over the keys. This is nondeterministic. Guess the memory profile has changed because of your other changes and now the order is different. Might even fail on different jvm versions.. i'll modify to keep a fixed ordering so the tests don't fail. > Author: brett > Date: Fri Aug 26 00:46:12 2005 > New Revision: 240180 > > URL: http://svn.apache.org/viewcvs?rev=240180&view=rev > Log: > fix tests: > - ordering of link elements was wrong > - windows path separator was wrong > > Modified: > > maven/components/trunk/maven-plugins/maven-eclipse-plugin/src/test/java/org/apache/maven/plugin/eclipse/EclipsePluginTest.java > > maven/components/trunk/maven-plugins/maven-eclipse-plugin/src/test/projects/project-4/project > > Modified: > maven/components/trunk/maven-plugins/maven-eclipse-plugin/src/test/java/org/apache/maven/plugin/eclipse/EclipsePluginTest.java > URL: > http://svn.apache.org/viewcvs/maven/components/trunk/maven-plugins/maven-eclipse-plugin/src/test/java/org/apache/maven/plugin/eclipse/EclipsePluginTest.java?rev=240180&r1=240179&r2=240180&view=diff > ============================================================================== > --- > maven/components/trunk/maven-plugins/maven-eclipse-plugin/src/test/java/org/apache/maven/plugin/eclipse/EclipsePluginTest.java > (original) > +++ > maven/components/trunk/maven-plugins/maven-eclipse-plugin/src/test/java/org/apache/maven/plugin/eclipse/EclipsePluginTest.java > Fri Aug 26 00:46:12 2005 > @@ -135,7 +135,7 @@ > // replace some vars in the expected line, to account > // for absolute paths that are different on each installation. > > - expected = StringUtils.replace( expected, "${basedir}", basedir > ); > + expected = StringUtils.replace( expected, "${basedir}", > basedir.replace( '\\', '/' ) ); > > if ( actualLines.size() < i ) > { > > Modified: > maven/components/trunk/maven-plugins/maven-eclipse-plugin/src/test/projects/project-4/project > URL: > http://svn.apache.org/viewcvs/maven/components/trunk/maven-plugins/maven-eclipse-plugin/src/test/projects/project-4/project?rev=240180&r1=240179&r2=240180&view=diff > ============================================================================== > --- > maven/components/trunk/maven-plugins/maven-eclipse-plugin/src/test/projects/project-4/project > (original) > +++ > maven/components/trunk/maven-plugins/maven-eclipse-plugin/src/test/projects/project-4/project > Fri Aug 26 00:46:12 2005 > @@ -18,11 +18,6 @@ > <location>${basedir}/src/test/projects/project-4/project.xml</location> > </link> > <link> > - <name>src-main-java</name> > - <type>2</type> > - > <location>${basedir}/src/test/projects/project-4/src/main/java</location> > - </link> > - <link> > <name>src-test-resources</name> > <type>2</type> > > <location>${basedir}/src/test/projects/project-4/src/test/resources</location> > @@ -36,6 +31,11 @@ > <name>src-main-resources</name> > <type>2</type> > > <location>${basedir}/src/test/projects/project-4/src/main/resources</location> > + </link> > + <link> > + <name>src-main-java</name> > + <type>2</type> > + > <location>${basedir}/src/test/projects/project-4/src/main/java</location> > </link> > </linkedResources> > </projectDescription> > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > -- Kenney Westerhof http://www.neonics.com GPG public key: http://www.gods.nl/~forge/kenneyw.key --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
