RFR 8005280: (props) Improve test coverage for small XML parser
huizhe.wang at oracle.com
Fri Dec 21 18:03:18 UTC 2012
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.
> 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.
> 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.
More information about the core-libs-dev