[google-caja] r4889 committed - speed up junit...

0 views
Skip to first unread message

googl...@googlecode.com

unread,
May 24, 2012, 10:26:02 PM5/24/12
to caja....@gmail.com
Revision: 4889
Author: fel...@gmail.com
Date: Thu May 24 19:25:43 2012
Log: speed up junit
http://codereview.appspot.com/6250057

We're running junit with fork="yes" because we want the junit vm to
have different properties than the ant vm. However, we're spawning
a new vm for each junit test, which is unnecessary.

This adds forkmode="once" to the junit tests.

This reduces 'ant runtests' on my laptop from about 11 minutes to
about 7 minutes.

R=meta...@gmail.com

http://code.google.com/p/google-caja/source/detail?r=4889

Modified:
/trunk/build.xml

=======================================
--- /trunk/build.xml Thu May 24 18:13:28 2012
+++ /trunk/build.xml Thu May 24 19:25:43 2012
@@ -234,7 +234,7 @@
</target>

<target name="RuntestsRunDefault">
- <junit printsummary="yes" fork="yes" maxmemory="512m">
+ <junit printsummary="yes" fork="yes" forkmode="once" maxmemory="512m">
<assertions><enable/></assertions> <!-- ignored unless fork="yes"
-->
<jvmarg line="${jvmarg}"/>
<classpath refid="classpath.tests.run"/>
@@ -262,7 +262,7 @@
</target>

<target name="RuntestsRunGwtBeans">
- <junit printsummary="yes" fork="yes" maxmemory="512m">
+ <junit printsummary="yes" fork="yes" forkmode="once" maxmemory="512m">
<sysproperty key="user.language" value="tr"/>
<sysproperty key="user.region" value="TR"/>
<sysproperty key="test.method.filter" value="${test.method.filter}"/>
Reply all
Reply to author
Forward
0 new messages