Release 0.3.2

doc/source/changelog.rst

@@ -1,6 +1,13 @@
 MontePy Changelog
 =================
 
+0.3.2
+--------------
+
+**Bug fixes**
+
+* Fixed bug with trailing dollar sign comments that moved them to a new line. (:issue:`458`).
+
 0.3.1
 ----------------
 

montepy/input_parser/syntax_node.py

@@ -96,7 +96,7 @@ def comments(self):
 
     def get_trailing_comment(self):
         """
-        Get the trailing comments if any.
+        Get the trailing ``c`` style  comments if any.
 
         :returns: The trailing comments of this tree.
         :rtype: list
@@ -427,13 +427,13 @@ def comments(self):
 
     def _get_first_comment(self):
         """
-        Get the first index that is a comment.
+        Get the first index that is a ``c`` style comment.
 
         :returns: the index of the first comment, if there is no comment then None.
         :rtype: int, None
         """
         for i, item in enumerate(self.nodes):
-            if isinstance(item, CommentNode):
+            if isinstance(item, CommentNode) and not item.is_dollar:
                 return i
         return None
 

tests/inputs/test.imcnp

@@ -38,7 +38,7 @@ C Iron
 m2        26054.80c        5.85
           26056.80c       91.75
           26057.80c        2.12
-          26058.80c        0.28
+          26058.80c        0.28 $ trailing comment shouldn't move #458.
 C water
 C foo
 m3        1001.80c           2

tests/test_syntax_parsing.py

@@ -340,7 +340,7 @@ def test_value_trailing_comments(self):
         self.assertIsNone(value_node.get_trailing_comment())
         value_node._delete_trailing_comment()
         self.assertIsNone(value_node.get_trailing_comment())
-        padding = syntax_node.PaddingNode("$ hi", True)
+        padding = syntax_node.PaddingNode("c hi", True)
         value_node.padding = padding
         comment = value_node.get_trailing_comment()
         self.assertEqual(len(comment), 1)
@@ -431,7 +431,7 @@ def test_syntax_trailing_comments(self):
         test = copy.deepcopy(self.test_node)
         test["bar2"].nodes["foo"] = syntax_node.ValueNode("1.23", float)
         self.assertIsNone(test.get_trailing_comment())
-        test["bar2"]["foo"].padding = syntax_node.PaddingNode("$ hi", True)
+        test["bar2"]["foo"].padding = syntax_node.PaddingNode("c hi", True)
         self.assertEqual(len(test.get_trailing_comment()), 1)
         test._delete_trailing_comment()
         self.assertIsNone(test.get_trailing_comment())
@@ -691,7 +691,7 @@ def test_list_trailing_comment(self):
         list_node1 = syntax_node.ListNode("list")
         for i in range(20):
             list_node1.append(syntax_node.ValueNode("1.0", float))
-        padding = syntax_node.PaddingNode("$ hi", True)
+        padding = syntax_node.PaddingNode("c hi", True)
         list_node1[-1].padding = padding
         comments = list(list_node1.get_trailing_comment())
         self.assertEqual(len(comments), 1)
@@ -1481,7 +1481,9 @@ def test_parameter_trailing_comments(self):
         padding = syntax_node.PaddingNode("$ hi", True)
         param["vol"]["data"][0].padding = padding
         comment = param.get_trailing_comment()
-        self.assertEqual(len(comment), 1)
+        self.assertIsNone(comment)
+        padding.append(syntax_node.CommentNode("c hi"), True)
+        comment = param.get_trailing_comment()
         self.assertEqual(comment[0].contents, "hi")
         param._delete_trailing_comment()
         self.assertIsNone(param.get_trailing_comment())