RFR 8057742: ant clean test should not fail if one or more external test suites are missing
marcus.lagergren at oracle.com
Mon Sep 8 08:02:55 UTC 2014
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.
> 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?
>> 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.
>>> 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.
More information about the nashorn-dev