Geir Magnusson Jr wrote:
> 
> 
> Tim Ellison wrote:
>> Geir Magnusson Jr wrote:
>>> do you want to discuss what you are undoing?
>>
>> I assume the monster diff is due to EOL differences or something...
>>
>> 1. I've make the test report location a variable
> 
> I thought you pointed it back into luni?

Nope, see  http://svn.apache.org/viewcvs?rev=383948&view=rev

>> 2. I've undone somebody else's undoing of the test suite invocation
> 
> Right, because we didn't get the same info from doing the suite as we
> did form running individual tests, IIRC.

Such as...?  Launching the VM for every individual test case is not
helpful here.

> Since the suite was only a set of tests (no extra setup)

what?

> and explicitly for Eclipse users because
> Eclipse's JUnit support is/was broken ;)

I use Eclipse all the time (and use the suite all the time from
Eclipse).  This tweak makes the Ant script behave that way too.

> I thought there was no harm in
> running the individual tests.  It also makes it easy to just drop in a
> test...

I've had no problem so far adding tests.  There is value in having
multiple suites (which we have only eluded to so far on the list).

Regards,
Tim


>>> [EMAIL PROTECTED] wrote:
>>>> Author: tellison
>>>> Date: Tue Mar  7 10:08:47 2006
>>>> New Revision: 383950
>>>>
>>>> URL: http://svn.apache.org/viewcvs?rev=383950&view=rev
>>>> Log:
>>>> Use the test suite, and put the results in the reporting dir
>>>>
>>>> Modified:
>>>>   
>>>> incubator/harmony/enhanced/classlib/trunk/modules/luni/make/common/build.xml
>>>>
>>>>
>>>>
>>>> Modified:
>>>> incubator/harmony/enhanced/classlib/trunk/modules/luni/make/common/build.xml
>>>>
>>>>
>>>> URL:
>>>> http://svn.apache.org/viewcvs/incubator/harmony/enhanced/classlib/trunk/modules/luni/make/common/build.xml?rev=383950&r1=383949&r2=383950&view=diff
>>>>
>>>>
>>>> ==============================================================================
>>>>
>>>>
>>>> ---
>>>> incubator/harmony/enhanced/classlib/trunk/modules/luni/make/common/build.xml
>>>>
>>>> (original)
>>>> +++
>>>> incubator/harmony/enhanced/classlib/trunk/modules/luni/make/common/build.xml
>>>>
>>>> Tue Mar  7 10:08:47 2006
>>>> @@ -1,111 +1,99 @@
>>>> -<?xml version="1.0" encoding="UTF-8"?>
>>>> -<!--
>>>> -    Copyright 2006 The Apache Software Foundation or its licensors,
>>>> as applicable.
>>>> -  -    Licensed under the Apache License, Version 2.0 (the "License");
>>>> -    you may not use this file except in compliance with the License.
>>>> -    You may obtain a copy of the License at
>>>> -  -       http://www.apache.org/licenses/LICENSE-2.0
>>>> -  -    Unless required by applicable law or agreed to in writing,
>>>> software
>>>> -    distributed under the License is distributed on an "AS IS" BASIS,
>>>> -    WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or
>>>> implied.
>>>> -    See the License for the specific language governing permissions
>>>> and
>>>> -    limitations under the License.
>>>> --->
>>>> -
>>>> -<project name="Common_LUNI_Build">
>>>> -   -    <target name="compile.java" description="Compile LUNI java
>>>> code">
>>>> -        <echo message="Compiling LUNI classes from
>>>> ${hy.luni.src.main.java}" />
>>>> -       -        <mkdir dir="${hy.luni.bin.main}" />
>>>> -
>>>> -        <javac sourcepath=""
>>>> -            srcdir="${hy.luni.src.main.java}"
>>>> -            destdir="${hy.luni.bin.main}"
>>>> -            source="${source.ver}"
>>>> -            debug="${java.debug.option}">
>>>> -
>>>> -            <bootclasspath>
>>>> -                <fileset dir="${hy.target}/jre/lib/boot">
>>>> -                    <include name="*.jar" />
>>>> -                </fileset>
>>>> -            </bootclasspath>
>>>> -        </javac>
>>>> -    </target>
>>>> -   -    <target name="build.jar">
>>>> -        <jar destfile="${hy.target}/jre/lib/boot/luni.jar"
>>>> manifest="${hy.luni}/META-INF/MANIFEST.MF">
>>>> -            <fileset dir="${hy.luni.bin.main}" />
>>>> -        </jar>
>>>> -    </target>
>>>> -
>>>> -   -    <target name="compile.tests">
>>>> -        <echo message="Compiling LUNI tests from
>>>> ${hy.luni.src.test.java}" />
>>>> -
>>>> -        <mkdir dir="${hy.luni.bin.test}" />
>>>> -
>>>> -        <javac srcdir="${hy.luni.src.test.java}"
>>>> -               destdir="${hy.luni.bin.test}"
>>>> -               sourcepath=""
>>>> -               source="${source.ver}"
>>>> -               debug="${java.debug.option}">
>>>> -
>>>> -            <bootclasspath>
>>>> -                <fileset dir="${hy.target}/jre/lib/boot">
>>>> -                    <include name="*.jar" />
>>>> -                </fileset>
>>>> -            </bootclasspath>
>>>> -        </javac>
>>>> -    </target>
>>>> -
>>>> -
>>>> -    <target name="run.tests">
>>>> -       -        <property name="test.dir"
>>>> value="../../../../target/test_report" />
>>>> -        <mkdir dir="${test.dir}" />
>>>> -
>>>> -        <junit fork="yes"
>>>> -            forkmode="once"
>>>> -            printsummary="withOutAndErr"
>>>> -            errorproperty="test.error"
>>>> -            showoutput="on"
>>>> -            dir="${hy.luni.bin.test}"
>>>> -            jvm="${hy.target}/jre/bin/java">
>>>> -
>>>> -            <jvmarg value="-showversion"/>
>>>> -
>>>> -            <env key="JAVA_HOME" value=""/>
>>>> -
>>>> -            <classpath>
>>>> -                <pathelement path="${hy.luni.bin.test}"/>
>>>> -            </classpath>
>>>> -
>>>> -            <!--
>>>> -            <test name="org.apache.harmony.tests.luni.AllTests"
>>>> -                haltonfailure="no"
>>>> -                todir="${test.dir}">
>>>> -                <formatter type="xml" />
>>>> -            </test>
>>>> -            -->
>>>> -           -            <formatter type="xml" />
>>>> -            <batchtest todir="${test.dir}">
>>>> -                <fileset dir="../../src/test/java">
>>>> -                    <include name="**/*Test.java"/>
>>>> -                </fileset>
>>>> -            </batchtest>
>>>> -
>>>> -        </junit>
>>>> -       -    </target>
>>>> -   -   -    <target name="copy.resources">
>>>> -        <!-- Nothing for LUNI -->
>>>> -    </target>
>>>> -</project>
>>>> -
>>>> +<?xml version="1.0" encoding="UTF-8"?>
>>>> +<!--
>>>> +    Copyright 2006 The Apache Software Foundation or its licensors,
>>>> as applicable.
>>>> +  +    Licensed under the Apache License, Version 2.0 (the "License");
>>>> +    you may not use this file except in compliance with the License.
>>>> +    You may obtain a copy of the License at
>>>> +  +       http://www.apache.org/licenses/LICENSE-2.0
>>>> +  +    Unless required by applicable law or agreed to in writing,
>>>> software
>>>> +    distributed under the License is distributed on an "AS IS" BASIS,
>>>> +    WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or
>>>> implied.
>>>> +    See the License for the specific language governing permissions
>>>> and
>>>> +    limitations under the License.
>>>> +-->
>>>> +
>>>> +<project name="Common_LUNI_Build">
>>>> +   +    <target name="compile.java" description="Compile LUNI java
>>>> code">
>>>> +        <echo message="Compiling LUNI classes from
>>>> ${hy.luni.src.main.java}" />
>>>> +       +        <mkdir dir="${hy.luni.bin.main}" />
>>>> +
>>>> +        <javac sourcepath=""
>>>> +            srcdir="${hy.luni.src.main.java}"
>>>> +            destdir="${hy.luni.bin.main}"
>>>> +            source="${source.ver}"
>>>> +            debug="${java.debug.option}">
>>>> +
>>>> +            <bootclasspath>
>>>> +                <fileset dir="${hy.target}/jre/lib/boot">
>>>> +                    <include name="*.jar" />
>>>> +                </fileset>
>>>> +            </bootclasspath>
>>>> +        </javac>
>>>> +    </target>
>>>> +   +    <target name="build.jar">
>>>> +        <jar destfile="${hy.target}/jre/lib/boot/luni.jar"
>>>> manifest="${hy.luni}/META-INF/MANIFEST.MF">
>>>> +            <fileset dir="${hy.luni.bin.main}" />
>>>> +        </jar>
>>>> +    </target>
>>>> +
>>>> +   +    <target name="compile.tests">
>>>> +        <echo message="Compiling LUNI tests from
>>>> ${hy.luni.src.test.java}" />
>>>> +
>>>> +        <mkdir dir="${hy.luni.bin.test}" />
>>>> +
>>>> +        <javac srcdir="${hy.luni.src.test.java}"
>>>> +               destdir="${hy.luni.bin.test}"
>>>> +               sourcepath=""
>>>> +               source="${source.ver}"
>>>> +               debug="${java.debug.option}">
>>>> +
>>>> +            <bootclasspath>
>>>> +                <fileset dir="${hy.target}/jre/lib/boot">
>>>> +                    <include name="*.jar" />
>>>> +                </fileset>
>>>> +            </bootclasspath>
>>>> +        </javac>
>>>> +    </target>
>>>> +
>>>> +
>>>> +    <target name="run.tests">
>>>> +       +        <mkdir dir="${hy.tests.reports}" />
>>>> +
>>>> +        <junit fork="yes"
>>>> +            forkmode="once"
>>>> +            printsummary="withOutAndErr"
>>>> +            errorproperty="test.error"
>>>> +            showoutput="on"
>>>> +            dir="${hy.luni.bin.test}"
>>>> +            jvm="${hy.target}/jre/bin/java">
>>>> +
>>>> +            <jvmarg value="-showversion"/>
>>>> +
>>>> +            <env key="JAVA_HOME" value=""/>
>>>> +
>>>> +            <classpath>
>>>> +                <pathelement path="${hy.luni.bin.test}"/>
>>>> +            </classpath>
>>>> +
>>>> +        <test name="org.apache.harmony.tests.luni.AllTests"
>>>> +                haltonfailure="no"
>>>> +                todir="${hy.tests.reports}">
>>>> +                <formatter type="xml" />
>>>> +        </test>
>>>> +    </junit>
>>>> +    </target>
>>>> +   +   +    <target name="copy.resources">
>>>> +        <!-- Nothing for LUNI -->
>>>> +    </target>
>>>> +</project>
>>>> +
>>>>
>>>>
>>>>
>>
> 

-- 

Tim Ellison ([EMAIL PROTECTED])
IBM Java technology centre, UK.

Reply via email to