RFR 8005280: (props) Improve test coverage for small XML parser
Joe Wang
huizhe.wang at oracle.com
Fri Dec 21 19:56:35 UTC 2012
On 12/21/2012 11:08 AM, Alan Bateman wrote:
> On 21/12/2012 18:03, Joe Wang wrote:
>> :
>>
>> Files are renamed. I added a new test "CompatibilityTest" to test
>> behavior compatibility with the regular JDK XML provider.
>>
>> Webrev:
>> http://cr.openjdk.java.net/~joehw/jdk8/8005280/webrev/
>>
>> -Joe
> Thanks for renaming the propertyfile_ prefix from the new test cases,
> looks much cleaner now.
>
> On Compatibility.xml then I guess it should very rare to use this for
> Properties. I guess the test is okay, just wondering about require the
> small parser to support it.
I searched by properties.dtd and found the CDATA usage. So I went ahead
experimenting what would not be rejected by the regular provider and
tested against the small parser. I agree the cases other than CDATA are
rare.
> In CompatibilityTest.loadPropertyFile then it looks like the file is
> not closed.
Properties' loadFromXML stated that it will close the stream after
loading. But it doesn't hurt to double-check.
New webrev:
http://cr.openjdk.java.net/~joehw/jdk8/8005280/webrev/
-Joe
>
> -Alan
More information about the core-libs-dev
mailing list