RFR (and sponsor): 7148488: Whitebox tests for the Diagnostic Framework Parser

Mikael Gerdin mikael.gerdin at oracle.com
Fri Mar 16 08:31:20 PDT 2012


just a small nitpick in parserTests.cpp:
41: const char* lookup_diagnosticArgumentEnum(const char* field_name, 
oop object)
52: void fill_in_parser(DCmdParser* parser, oop argument)
should probably be static (in the C sense).

Otherwise it looks good.


On 2012-03-15 13:45, Nils Loodin wrote:
> Hey all!
> Here's an implementation of a nice way of doing parser testing from a
> jtreg-test, through the whitebox testing framework.
> This patch makes it easy to do parser testing (which will be necessary
> if we want to change it with any sort of confidence in the future) and
> partly to show off what can be possible to do with the whitebox testing api.
> In the added JTREG test, parser testing now works like this from java:
> //test that we can parse without exceptions
> wb.parseCommandLine("myIntArgument=10", args);
> //test that the value get's parsed to what we want
> parse("myIntArgument", "10", "myIntArgument=10", args);
> //test that illegal command lines gives exception and aren't silently broken
> shouldFail("myLongArgument=12m", args); //<-- should fail, doesn't
> http://cr.openjdk.java.net/~nloodin/7148488/webrev.00/
> Regards,
> Nils Loodin

More information about the hotspot-dev mailing list