Close file handle in LoadMD. Refs #5320

Windows requires all file handles to be closed before anything can be
deleted. The handle is only closed for the histo case and is deleted
if no backend is being used.

Code/Mantid/Framework/MDEvents/src/LoadMD.cpp

@@ -284,6 +284,10 @@ namespace Mantid
         // MDHistoWorkspace case.
         this->loadHisto();
       }
+      if(!fileBacked)
+      {
+        delete file;
+      }
     }
 
 
@@ -328,6 +332,8 @@ namespace Mantid
       this->loadSlab("num_events", ws->getNumEventsArray(), ws, ::NeXus::FLOAT64);
       this->loadSlab("mask", ws->getMaskArray(), ws, ::NeXus::INT8);
 
+      file->close();
+
       // Save to output
       setProperty("OutputWorkspace", boost::dynamic_pointer_cast<IMDWorkspace>(ws));
     }

Code/Mantid/Framework/MDEvents/test/LoadMDTest.h

@@ -504,26 +504,18 @@ class LoadMDTest : public CxxTest::TestSuite
 
   void test_histo2() 
   {
-    // Fails on windows with file access error
-#ifndef _WIN32
     MDHistoWorkspace_sptr ws = MDEventsTestHelper::makeFakeMDHistoWorkspace(2.5, 2, 10, 10.0, 3.5, "histo2", 4.5);
     doTestHisto(ws);
-#endif
   }
 
   void test_histo3()
   {
-  // Fails on windows with file access error
-#ifndef _WIN32
     MDHistoWorkspace_sptr ws = MDEventsTestHelper::makeFakeMDHistoWorkspace(2.5, 3, 4, 10.0, 3.5, "histo3", 4.5);
     doTestHisto(ws);
-#endif
   }
 
-
 };
 
-
 #endif /* MANTID_MDEVENTS_LOADMDEWTEST_H_ */