Hi everyone.

Please review this unit test fix. Sue found this bug when she changed the DTD.

https://cr.opensolaris.org/action/browse/caiman/schwartz/7177531_1/webrev.1/

Bug ID: http://monaco.us.oracle.com/detail.jsf?cr=7177531
Bug: unit test that assembles manifest from pieces reads the wrong DTD

The problem was that the manifest was validating against the system's DTD instead of the DTD in the proto area. (Adding the schema arg to the mim constructor fixed that.) The rest of the fix is around performing the diff of original manifest vs the created one.

Testing:
1) Copied Sue's new DTD and manifest to the proto area. All unit tests in the MIM overlay module now pass. 2) Restored original DTD and manifest to the proto area. All unit tests in the MIM overlay module still pass.
3) pep8 and pylint checked.

    Thanks,
    Jack
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss

Reply via email to