Fixed string format error for snapshots (#1277)

armi/operators/operator.py

@@ -1126,9 +1126,9 @@ def snapshotRequest(self, cycle, node):
             if "rzmflx" in fileName:
                 pathTools.copyOrWarn("rzmflx for snapshot", fileName, newFolder)
 
-        isoFName = "ISOTXS-c{0}n{0}".format(cycle, node)
+        isoFName = f"ISOTXS-c{cycle}n{node}"
         if not os.path.exists(isoFName):
-            isoFName = "ISOTXS-c{0}".format(cycle)
+            isoFName = f"ISOTXS-c{cycle}"
         pathTools.copyOrWarn(
             "ISOTXS for snapshot", isoFName, pathTools.armiAbsPath(newFolder, "ISOTXS")
         )

armi/operators/tests/test_operators.py

@@ -125,7 +125,15 @@ def test_setStateToDefault(self):
 
     def test_snapshotRequest(self):
         with TemporaryDirectoryChanger():
-            self.o.snapshotRequest(0, 1)
+            with mockRunLogs.BufferLog() as mock:
+                self.o.snapshotRequest(0, 1)
+                self.assertIn("ISOTXS-c0", mock.getStdout())
+                self.assertIn("DIF3D output for snapshot", mock.getStdout())
+                self.assertIn("Shuffle logic for snapshot", mock.getStdout())
+                self.assertIn("Geometry file for snapshot", mock.getStdout())
+                self.assertIn("Loading definition for snapshot", mock.getStdout())
+                self.assertIn("Flow history for snapshot", mock.getStdout())
+                self.assertIn("Pressure history for snapshot", mock.getStdout())
             self.assertTrue(os.path.exists("snapShot0_1"))
 
 

doc/release/0.2.rst

@@ -12,7 +12,7 @@ What's new in ARMI
 
 Bug fixes
 ---------
-#. TBD
+#. Fixed a bug in the ISOTXS file name used for snapshots. (`PR#1277 <https://github.com/terrapower/armi/pull/1277>`_)
 
 
 ARMI v0.2.7