Merge remote-tracking branch 'origin/master' into 1.0

[dcpomatic.git] / test / test.cc

diff --git a/test/test.cc b/test/test.cc
index c233a661ddf4666cb65bd89c68a413d44950fcdb..4466b74cb32d77212766b84302df0e4a03456daf 100644 (file)
--- a/test/test.cc
+++ b/test/test.cc
@@ -113,7 +113,7 @@ check_file (boost::filesystem::path ref, boost::filesystem::path check)
 {
        uintmax_t N = boost::filesystem::file_size (ref);
        BOOST_CHECK_EQUAL (N, boost::filesystem::file_size (check));
-       FILE* ref_file = fopen (ref.c_str(), "rb");
+       FILE* ref_file = fopen_boost (ref, "rb");
        BOOST_CHECK (ref_file);
        FILE* check_file = fopen_boost (check, "rb");
        BOOST_CHECK (check_file);
@@ -182,7 +182,7 @@ check_xml (xmlpp::Element* ref, xmlpp::Element* test, list<string> ignore)
 
        xmlpp::Element::NodeList::iterator k = ref_children.begin ();
        xmlpp::Element::NodeList::iterator l = test_children.begin ();
-       while (k != ref_children.end ()) {
+       while (k != ref_children.end () && l != test_children.end ()) {
                
                /* XXX: should be doing xmlpp::EntityReference, xmlpp::XIncludeEnd, xmlpp::XIncludeStart */
 
@@ -211,6 +211,9 @@ check_xml (xmlpp::Element* ref, xmlpp::Element* test, list<string> ignore)
                ++k;
                ++l;
        }
+
+       BOOST_CHECK (k == ref_children.end ());
+       BOOST_CHECK (l == test_children.end ());
 }
 
 void