Fixing issue where case mistakenly made absolute paths relative (#914)

terrapower · Oct 3, 2022 · 7be2ad9 · 7be2ad9
1 parent 6557412
commit 7be2ad9
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 1 deletion.
diff --git a/armi/cases/case.py b/armi/cases/case.py
@@ -889,7 +889,8 @@ def copyInterfaceInputs(
                     try:
                         if path.is_absolute() and path.exists() and path.is_file():
                             # Path is absolute, no settings modification or filecopy needed
-                            pass
+                            newFiles.append(path)
+                            continue
                     except OSError:
                         pass
                 # Attempt to construct an absolute file path

diff --git a/armi/cases/tests/test_cases.py b/armi/cases/tests/test_cases.py
@@ -543,6 +543,20 @@ def test_copyInterfaceInputs_relPath(self):
             newFilepath = os.path.join(newDir.destination, shuffleFile)
             self.assertEqual(newSettings[testSetting], newFilepath)
 
+    def test_copyInterfaceInputs_absPath(self):
+        testSetting = "shuffleLogic"
+        cs = settings.Settings(ARMI_RUN_PATH)
+        shuffleFile = cs[testSetting]
+        absFile = os.path.dirname(os.path.abspath(ARMI_RUN_PATH))
+        absFile = str(os.path.join(absFile, os.path.basename(shuffleFile)))
+        cs = cs.modified(newSettings={testSetting: absFile})
+
+        with directoryChangers.TemporaryDirectoryChanger() as newDir:
+            newSettings = cases.case.copyInterfaceInputs(
+                cs, destination=newDir.destination
+            )
+            self.assertEqual(str(newSettings[testSetting]), absFile)
+
 
 if __name__ == "__main__":
     unittest.main()