RFR 8005280: (props) Improve test coverage for small XML parser
Lance Andersen
Lance.Andersen at oracle.com
Fri Dec 21 18:40:57 UTC 2012
+1
On Dec 21, 2012, at 1:03 PM, Joe Wang <huizhe.wang at oracle.com> wrote:
>
>
> On 12/21/2012 4:05 AM, Alan Bateman wrote:
>> On 21/12/2012 08:37, Joe Wang wrote:
>>> The cause of the LoadAndStoreXML test failure appeared to be that of 8005281 that Alan just fixed. Before the 8005281 patch, I was able to get the tests to pass when I isolated the relevant tests (that is, copy LoadAndStoreXML and remove other test cases). After the 8005281 patch, LoadAndStoreXML passed in its original form.
>>>
>>> I've also added a few more invalid xml files, plus international characters to testLoadAndStore.
>>>
>>> Webrev:
>>> http://cr.openjdk.java.net/~joehw/jdk8/8005280/webrev/
>> I concur with your observation that this issue is fixed by 8005281, in which case we can change the focus for 8005280 to extend the test coverage as you have done.
>
> Subject corrected.
>
>>
>> The new tests look good to me except that you've prefixed them all with "propertyfile_" and so are inconsistent with the existing tests. I think it would be good to rename them to be consistent before pushing this.
>
> 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
>
>>
>> -Alan
>>
Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering
1 Network Drive
Burlington, MA 01803
Lance.Andersen at oracle.com
More information about the core-libs-dev
mailing list