add missing test tags

terrapower · Mar 26, 2024 · 55a40f7 · 55a40f7
1 parent 567e02c
commit 55a40f7
Show file tree

Hide file tree

Showing 2 changed files with 27 additions and 2 deletions.
diff --git a/armi/reactor/converters/axialExpansion/tests/test_axialExpansionChanger.py b/armi/reactor/converters/axialExpansion/tests/test_axialExpansionChanger.py
@@ -293,6 +293,10 @@ def _getMass(a):
     def test_PrescribedExpansionContractionConservation(self):
         """Expand all components and then contract back to original state.
 
+        .. test:: Expand all components and then contract back to original state.
+            :id: T_ARMI_AXIAL_EXP_PRESC0
+            :tests: R_ARMI_AXIAL_EXP_PRESC
+
         Notes
         -----
         - uniform expansion over all components within the assembly
@@ -374,7 +378,12 @@ def test_TargetComponentMassConservation(self):
             )
 
     def test_NoMovementACLP(self):
-        """Ensures that above core load pad (ACLP) does not move during fuel-only expansion."""
+        """Ensures that above core load pad (ACLP) does not move during fuel-only expansion.
+
+        .. test:: Ensure the ACLP does not move during fuel-only expansion.
+            :id: T_ARMI_AXIAL_EXP_PRESC1
+            :tests: R_ARMI_AXIAL_EXP_PRESC
+        """
         # build test assembly with ACLP
         assembly = HexAssembly("testAssemblyType")
         assembly.spatialGrid = grids.axialUnitGrid(numCells=1)
@@ -555,8 +564,19 @@ def setUp(self):
     def test_coldAssemblyExpansion(self):
         """Block heights are cold and should be expanded.
 
+        .. test:: Preserve the total height of a compatible ARMI assembly.
+            :id: T_ARMI_ASSEM_HEIGHT_PRES
+            :tests: R_ARMI_ASSEM_HEIGHT_PRES
+
+        .. test:: Axial expansion can be prescribed in blueprints for core constuction.
+            :id: T_ARMI_INP_COLD_HEIGHT
+            :tests: R_ARMI_INP_COLD_HEIGHT
+
         Notes
         -----
+        For R_ARMI_INP_COLD_HEIGHT, the action of axial expansion occurs in setUp() during core
+        construction, specifically in :py:meth:`constructAssem <armi.reactor.blueprints.Blueprints.constructAssem>`
+
         Two assertions here:
             1. total assembly height should be preserved (through use of top dummy block)
             2. in armi.tests.detailedAxialExpansion.refSmallReactorBase.yaml,

diff --git a/armi/reactor/converters/axialExpansion/tests/test_expansionData.py b/armi/reactor/converters/axialExpansion/tests/test_expansionData.py
@@ -308,7 +308,12 @@ def test_specifyTargetComponet_MultipleFound(self):
             self.assertEqual(the_exception.error_code, 3)
 
     def test_manuallySetTargetComponent(self):
-        """Ensures that target components can be manually set (is done in practice via blueprints)."""
+        """Ensures that target components can be manually set (is done in practice via blueprints).
+
+        .. test:: Allow user-specified target axial expansion components on a given block.
+            :id: T_ARMI_MANUAL_TARG_COMP
+            :tests: R_ARMI_MANUAL_TARG_COMP
+        """
         b = HexBlock("dummy", height=10.0)
         ductDims = {"Tinput": 25.0, "Thot": 25.0, "op": 17, "ip": 0.0, "mult": 1.0}
         duct = Hexagon("duct", "HT9", **ductDims)