RFR (and sponsor): 7148488: Whitebox tests for the Diagnostic Framework Parser
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,
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
> Nils Loodin
More information about the hotspot-dev