Merge 363bc7f into c1aa430

pyiron_base/jobs/job/extension/files.py

@@ -115,13 +115,24 @@ def __getattr__(self, item):
                 raise FileNotFoundError(item) from None
 
 
-class File(str):
+class File:
+    __slots__ = ("_path",)
+
+    def __init__(self, path):
+        self._path = path
+
+    def __str__(self):
+        return self._path
+
     def tail(self, lines: int = 100):
         print(
             *_working_directory_read_file(
-                working_directory=os.path.dirname(self),
-                file_name=os.path.basename(self),
+                working_directory=os.path.dirname(self._path),
+                file_name=os.path.basename(self._path),
                 tail=lines,
             ),
             sep="",
         )
+
+    def __eq__(self, other):
+        return self.__str__().__eq__(other)

pyiron_base/jobs/job/generic.py

@@ -23,6 +23,7 @@
     _doc_str_job_core_attr,
 )
 from pyiron_base.jobs.job.extension.executable import Executable
+from pyiron_base.jobs.job.extension.files import File
 from pyiron_base.jobs.job.extension.jobstatus import JobStatus
 from pyiron_base.jobs.job.runfunction import (
     run_job_with_parameter_repair,
@@ -286,6 +287,8 @@ def restart_file_list(self, filenames):
             filenames (list):
         """
         for f in filenames:
+            if isinstance(f, File):
+                f = str(f)
             if not (os.path.isfile(f)):
                 raise IOError("File: {} does not exist".format(f))
             self.restart_file_list.append(f)
@@ -296,6 +299,8 @@ def restart_file_dict(self):
         A dictionary of the new name of the copied restart files
         """
         for actual_name in [os.path.basename(f) for f in self._restart_file_list]:
+            if isinstance(actual_name, File):
+                actual_name = str(actual_name)
             if actual_name not in self._restart_file_dict.keys():
                 self._restart_file_dict[actual_name] = actual_name
         return self._restart_file_dict
@@ -1043,8 +1048,8 @@ def to_dict(self):
         data_dict = self._type_to_dict()
         data_dict["status"] = self.status.string
         data_dict["input/generic_dict"] = {
-            "restart_file_list": self._restart_file_list,
-            "restart_file_dict": self._restart_file_dict,
+            "restart_file_list": [str(f) for f in self._restart_file_list],
+            "restart_file_dict": {str(f): n for f, n in self._restart_file_dict.items()},
             "exclude_nodes_hdf": self._exclude_nodes_hdf,
             "exclude_groups_hdf": self._exclude_groups_hdf,
         }

tests/flex/test_executablecontainer.py

@@ -102,7 +102,7 @@ def test_job_files(self):
         if os.name != "nt":
             self.assertEqual(job.files.error_out, output_file_path)
         else:
-            self.assertEqual(job.files.error_out.replace("/", "\\"), output_file_path)
+            self.assertEqual(job.files.error_out, output_file_path.replace("\\", "/"))
 
     def test_create_job_factory_typeerror(self):
         create_catjob = create_job_factory(