Reviewers: kpreid2,
Description:
right now, we generate a test report in the same directory
we save test logs, which is inconvenient because the test summary
is an index.html file, making it difficult to use a webbrowser to
look at other files in that directory.
this change:
1. moves the test report out of ant-reports/test.runtests into
a separate dir report.runtests
2. moves captured screenshots etc out of test.runtests into
capture.runtests
Please review this at https://codereview.appspot.com/10373044/
Affected files:
M build.xml
Index: build.xml
===================================================================
--- build.xml (revision 5452)
+++ build.xml (working copy)
@@ -318,13 +318,16 @@
</target>
<target name="RuntestsRunBrowser" unless="emma.enabled">
+ <mkdir dir="${reports}/capture.${test.type}" />
<junit printsummary="yes" fork="yes" forkmode="once" maxmemory="512m">
<assertions><enable/></assertions> <!-- ignored unless fork="yes"
-->
<classpath refid="classpath.tests"/>
<formatter type="xml"/>
<jvmarg line="${jvmarg}"/>
<syspropertyset refid="test.propertyset"/>
- <sysproperty key="test.capture.to"
value="${reports}/test.${test.type}"/>
+ <sysproperty
+ key="test.capture.to"
+ value="${reports}/capture.${test.type}"/>
<batchtest todir="${reports}/test.${test.type}">
<fileset dir="${tests}">
<and>
@@ -415,9 +418,10 @@
</report>
</emma>
<!-- Generate the test results report. -->
+ <mkdir dir="${reports}/report.${test.type}"/>
<junitreport todir="${reports}/test.${test.type}">
<fileset dir="${reports}/test.${test.type}" includes="TEST-*.xml"/>
- <report format="frames" todir="${reports}/test.${test.type}"
+ <report format="frames" todir="${reports}/report.${test.type}"
styledir="${third_party}/ant/etc"/>
</junitreport>
<condition property="runtests.success">
@@ -441,10 +445,10 @@
</and>
</condition>
<fail message=
- "One or more tests failed. For details, see
${reports}/test.${test.type}/index.html"
+ "One or more tests failed. For details, see
${reports}/report.${test.type}/index.html"
if="runtests.error"/>
<fail message=
- "One or more tests failed. For details, see
${reports}/test.${test.type}/index.html"
+ "One or more tests failed. For details, see
${reports}/report.${test.type}/index.html"
if="runtests.failure"/>
</target>
--
---
You received this message because you are subscribed to the Google Groups "Google Caja Discuss" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.
