RFR 8054461: Add @file support to sjavac
Jonathan Gibbons
jonathan.gibbons at oracle.com
Fri Aug 8 18:53:08 UTC 2014
On 08/08/2014 09:11 AM, Andreas Lundblad wrote:
> On Fri, Aug 08, 2014 at 09:44:23AM +0200, Fredrik Öhrström wrote:
>> See update:
>> http://cr.openjdk.java.net/~ohrstrom/webrev-8054461-atfile-v2/
>>
> I don't really like the idea of Sjavac.java:
>
> * @summary Test all aspects of sjavac.
>
> Can't we try to have one test per feature / bug?
>
>
> Code looks good.
>
> -- Andreas
Note that the need for the overhead of the test wrapper classes has gone
away, which makes it easier to have a test per feature/bug.
I'm OK with multiple test cases within a test (it's a style issue that
can go either way) but I would note that these days we are focussing on
verifying that the tests for a feature do at least minimally execute the
new or modified code for the feature. You can do this with the jcov
support for checking the coverage of the new or modified code in a
patch, and you can easily access this functionality from recent releases
of jtreg.
-- Jon
More information about the compiler-dev
mailing list