Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(86)

Issue 6250057: speed up junit (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
14 years ago by felix8a
Modified:
14 years ago
Reviewers:
ihab.awad, metaweta
CC:
google-caja-discuss_googlegroups.com
Base URL:
http://google-caja.googlecode.com/svn/trunk/
Visibility:
Public.

Description

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.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -2 lines) Patch
M build.xml View 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 3
felix8a
14 years ago (2012-05-25 01:43:37 UTC) #1
metaweta
lgtm
14 years ago (2012-05-25 02:24:18 UTC) #2
felix8a
14 years ago (2012-05-25 02:26:06 UTC) #3
@r4889
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b