Skip to content

Commit ac6688d

Browse files
committed
Treat JUnit errors as failures to increase visibility
Before this change `cd app; ant test` would return "BUILD SUCCESS" even when unit tests were returning errors.
1 parent 0dbff59 commit ac6688d

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

app/build.xml

+2-1
Original file line numberDiff line numberDiff line change
@@ -109,7 +109,7 @@
109109
<fileset dir="test" includes="**/*.pac" />
110110
</copy>
111111

112-
<junit printsummary="yes" dir="${work.dir}" fork="true">
112+
<junit printsummary="yes" dir="${work.dir}" fork="true" showoutput="yes" failureproperty="test.failed">
113113
<jvmarg value="-Djava.library.path=${java.additional.library.path}"/>
114114
<jvmarg value="-DWORK_DIR=."/>
115115
<jvmarg value="-ea"/>
@@ -131,6 +131,7 @@
131131
</batchtest>
132132
</junit>
133133

134+
<fail if="test.failed"/>
134135
</target>
135136

136137
<target name="build" depends="compile" description="Build PDE">

0 commit comments

Comments
 (0)