[Rev 02] RFR: 6725: Formalize agent probe definition XML schema and enforce validations
Kangcheng Xu
kxu at openjdk.java.net
Sun Apr 19 22:35:45 UTC 2020
On Mon, 6 Apr 2020 16:59:04 GMT, Alex Macdonald <aptmac at openjdk.org> wrote:
>> Kangcheng Xu has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Fix TestRetrieveCurrentTransforms
>
> This looks good to me, I do have a question and a request though.
>
> 1. Mainly out of curiosity, how are you doing your XML formatting? Some of the `<description>`s were changed to be
> mulit-line, but only have a single word in the new line.. it initially looked odd to me but I guess it makes sense.
> 2. Could you add a unit test or two verifying the behaviour of an invalid probe definition? The current unit tests only
> verify the happy-path, and while the regex look good to me I would feel better if there was a test case for some
> invalid patterns.
@aptmac Sorry for taking so long
> 1. Mainly out of curiosity, how are you doing your XML formatting? ...
I've re-run the auto-formatter in Eclipse after importing the JMC formatted config.
> 2. Could you add a unit test or two verifying the behaviour of an invalid probe definition? ...
I've updated with more unit tests.
Also, `.classpath` is updated to include the `resources` directory which contains the XSD schema.
-------------
PR: https://git.openjdk.java.net/jmc/pull/66
More information about the jmc-dev
mailing list