RFR 8005281: (props) loadFromXML with small XML parser provider does not detect bad DOCTYPE (original) (raw)
Alan Bateman Alan.Bateman at oracle.com
Fri Dec 21 12:05:49 UTC 2012
- Previous message: RFR 8005281: (props) loadFromXML with small XML parser provider does not detect bad DOCTYPE
- Next message: RFR 8005280: (props) Improve test coverage for small XML parser
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
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.
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.
-Alan
- Previous message: RFR 8005281: (props) loadFromXML with small XML parser provider does not detect bad DOCTYPE
- Next message: RFR 8005280: (props) Improve test coverage for small XML parser
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]