Merge pull request #79 from LEMS/test_file_close

To v0.6.7; improve file closure

Author: pgleeson

lems/model/model.py

@@ -286,7 +286,8 @@ def include_file(self, path, include_dirs=[]):
             parser = LEMSFileParser(self, inc_dirs, self.include_includes)
             if os.access(path, os.F_OK):
                 if not path in self.included_files:
-                    parser.parse(open(path).read())
+                    with open(path) as f:
+                        parser.parse(f.read())
                     self.included_files.append(path)
                     return
                 else:
@@ -298,7 +299,8 @@ def include_file(self, path, include_dirs=[]):
                     new_path = inc_dir + "/" + path
                     if os.access(new_path, os.F_OK):
                         if not new_path in self.included_files:
-                            parser.parse(open(new_path).read())
+                            with open(new_path) as f:
+                                parser.parse(f.read())
                             self.included_files.append(new_path)
                             return
                         else:
@@ -388,9 +390,11 @@ def export_to_file(self, filepath, level_prefix="  "):
             "\n",
         )
 
-        f = open(filepath, "w")
-        f.write(xmlstr)
-        f.close()
+        with open(filepath, "w") as f:
+            f.write(xmlstr)
+            f.flush()
+            os.fsync(f.fileno())
+
 
     def resolve(self) -> lems.model.Model:
         """

setup.cfg

@@ -1,6 +1,6 @@
 [metadata]
 name = PyLEMS
-version = 0.6.5
+version = 0.6.7
 author = PyLEMS authors and contributors
 author_email = [email protected], [email protected]
 maintainer_email = [email protected]