RFR 8057742: ant clean test should not fail if one or more external test suites are missing

[Marcus Lagergren]

Awesome. 

On 08 Sep 2014, at 09:57, A. Sundararajan <sundararajan.athijegannathan at oracle.com> wrote:

> Actually we were silently skipping tests that depend on missing "external" scripts  -- just that the silent skipping was incomplete in those three tests. So, I'm just making sure we consistently skip -- if external test suite is missing.
> 
> That said, I think we can add a warning for each test that is skipped due to missing external test suite. I'll make that change.
> 
> -Sundar
> 
> On Sunday 07 September 2014 02:29 PM, Attila Szegedi wrote:
>> I actually appreciated that I had a signal that some tests were not run. If I do a fresh checkout and not run "ant externals", now I won't get a signal that some tests weren't run, and maybe I'll mistakenly think everything passed. How about at least an <echo> in build.xml that'll print a warning after a successful test run that some tests were skipped as externals aren't available?
>> 
>> Attila.
>> 
>> On Sep 7, 2014, at 7:04 AM, Marcus Lagergren <marcus.lagergren at oracle.com> wrote:
>> 
>>> Why did you remove some of the test runs? Just redundancy?
>>> 
>>> Otherwise, looks fine form me. +1
>>> 
>>> In the future I think it could be nice to have a proxy check before ant externals, which certainly helps me when I am behind a firewall.
>>> 
>>> /M
>>> 
>>> On 07 Sep 2014, at 06:40, A. Sundararajan <sundararajan.athijegannathan at oracle.com> wrote:
>>> 
>>>> Please review http://cr.openjdk.java.net/~sundar/8057742/
>>>> 
>>>> Tested that w/without the test/script/external directory, "ant clean test" is fine.
>>>> 
>>>> Thanks
>>>> -Sundar
>