+1, this makes more sense
Jacques
From: "Paul Foxworthy" <p...@cohsoft.com.au>
If we do have two tasks, how about
run-tests
which would revert to the older style without cobertura and would not
depend on an Internet connection, and
run-tests-with-coverage
which would use cobertura and would have the dependency?
Better to say what tasks do and what they are, rather than name a task based
on a dependency or the lack of one.
What do people think?
Cheers
Paul Foxworthy
Erwan de FERRIERES wrote:
Le 24/03/2011 12:52, Ruth Hoffman a écrit :
> Hi Jacques:
> I discovered this last week during a training class I was
> giving...Unfortunately, I was not hooked up to the Internet so I was
not
> able to demonstrate. IMHO, this should be reverted back to the
expected
> behavior, or alternatively, an option provided to bypass and still
run
> non-network based tests.
>
> Regards,
> Ruth
>
>
> On 3/24/11 2:49 AM, Jacques Le Roux wrote:
>> I have nothing against (as it can be easily bypasssed) it but
it's
>> worth to be noted that run-tests depends now on
download-cobertura,
>> which itself depends on an internet connection. Hence run-tests
can't
>> be used on a server w/out Internet connection
>>
>> Jacques
>>
>>
>
Hi Ruth,
it's really easy to bypass, just remove the download-cobertura call.
<target name="run-tests" depends="download-cobertura,
build"
I have been added this as there was an error in the logs when lauching
tests and the lib wasn't found.
a solution I see is creating a new ant target, like
run-tests-no-internet, or whatever you want which is *not* using
cobertura nor test coverage.
This has been added in OFBiz in last december
(https://fisheye6.atlassian.com/changelog/ofbiz?cs=1050915).
I've just added a better description for the ant target.
So, waiting for your patch !
Cheers,
--
Erwan de FERRIERES
www.nereide.biz
--
View this message in context:
http://ofbiz.135035.n4.nabble.com/Run-tests-can-t-be-used-on-a-server-w-out-Internet-connection-tp3401867p3404519.html
Sent from the OFBiz - Dev mailing list archive at Nabble.com.
