Unit test checks that Output file can be parsed as XML file re #7617

Also some unnecessary includes were removed from the ConvertFullprofToXML.cpp.

Signed-off-by: Karl Palmen <karl.palmen@stfc.ac.uk>

Code/Mantid/Framework/DataHandling/src/ConvertFullprofToXML.cpp

@@ -14,12 +14,6 @@ Convert the initial fitting parameters in a Fullprof file to XML format in an [[
 #include <Poco/DOM/Document.h>
 #include <Poco/DOM/DOMWriter.h>
 #include <Poco/DOM/Element.h>
-#include <Poco/DOM/Text.h>
-#include <Poco/DOM/DOMParser.h>
-#include <Poco/DOM/Document.h>
-#include <Poco/DOM/NodeList.h>
-#include <Poco/DOM/NodeIterator.h>
-#include <Poco/DOM/NodeFilter.h>
 
 #include <boost/lexical_cast.hpp>
 

Code/Mantid/Framework/DataHandling/test/ConvertFullprofToXMLTest.h

@@ -7,6 +7,10 @@
 #include <fstream>
 #include <Poco/File.h>
 
+#include <Poco/DOM/Document.h>
+#include <Poco/DOM/DOMParser.h>
+#include <Poco/DOM/Element.h>
+
 using Mantid::DataHandling::ConvertFullprofToXML;
 
 using namespace Mantid;
@@ -62,6 +66,13 @@ class ConvertFullprofToXMLTest : public CxxTest::TestSuite
     outputFilename = alg.getPropertyValue("outputFilename"); //Get absolute path
     TS_ASSERT( Poco::File(outputFilename).exists() );
 
+    // Check output file
+    std::string xmlText = Strings::loadFile(outputFilename);
+    Poco::XML::DOMParser pParser;
+    Poco::XML::Document* doc;
+    TS_ASSERT_THROWS_NOTHING(doc = pParser.parseString(xmlText));
+    TS_ASSERT(doc);
+
     //Clean up
     Poco::File(inputFilename).remove();
     Poco::File(outputFilename).remove();