But now we're running them twice. I'll give you the LGTM as testing is good, but I'm a bit worried for the time penalty. But if it's a problem, we can fall back to the other approach when it's clear it's a problem.
On Thu, Apr 23, 2009 at 1:50 PM, Amit Manjhi <amitman...@google.com> wrote: > It just requires emma.jar which is pulled in from the tools dir. The time > is basically the same as running hosted mode user tests. > > Amit > > > On Thu, Apr 23, 2009 at 10:41 AM, Freeland Abbott <fabb...@google.com>wrote: > >> Well, that will run emma tests for everyone everywhere who does "ant >> test"... >> Does it require anything in particular to work, which people might not >> have installed? And is the time significant? >> >> We can easily enough tweak the continuous builder configuration to >> explicitly run the emma tests, if either of those questions gets a bad >> answer. If they're both good, then maybe it's reasonable for all users to >> run all tests (with the caveat that non-local web tests also need properties >> set, or they become no-ops...) >> >> >> On Thu, Apr 23, 2009 at 1:10 PM, Amit Manjhi <amitman...@google.com>wrote: >> >>> Hi Freeland, >>> >>> The patch makes the emma tests run as part of our continuous build. The >>> tests basically run all tests in user, except where sun's and openjdk's >>> javac are broken, with emma.jar on the classpath. >>> >>> Regards, >>> Amit >>> >> >> > --~--~---------~--~----~------------~-------~--~----~ http://groups.google.com/group/Google-Web-Toolkit-Contributors -~----------~----~----~----~------~----~------~--~---