Maintaining surrounding parentheses when assigning a method call result executed on the result of a maths operation

Author: SteveWilkes

ReadableExpressions.UnitTests/WhenTranslatingAssignments.cs

@@ -357,9 +357,20 @@ public void ShouldAssignAVariableInAMethodCallArgument()
 
             var translated = intToString.ToReadableString();
 
-            const string EXPECTED = @"i.ToString(value = null)";
+            Assert.AreEqual("i.ToString(value = null)", translated);
+        }
 
-            Assert.AreEqual(EXPECTED.TrimStart(), translated);
+        [TestMethod]
+        public void ShouldAssignAMultiplicationToStringResult()
+        {
+            Expression<Func<int, string>> timesThreeToString = i => (i * 3).ToString();
+
+            var stringVariable = Expression.Variable(typeof(string), "value");
+            var stringAssignment = Expression.Assign(stringVariable, timesThreeToString.Body);
+
+            var translated = stringAssignment.ToReadableString();
+
+            Assert.AreEqual("value = (i * 3).ToString()", translated);
         }
 
         [TestMethod]

ReadableExpressions/StringExtensions.cs

@@ -117,7 +117,16 @@ private static bool KeepSurroundingParentheses(Expression expression)
                         parentExpression = expression.GetParentOrNull();
                     }
 
-                    return (expression.NodeType == ExpressionType.Convert);
+                    switch (expression.NodeType)
+                    {
+                        case ExpressionType.Add:
+                        case ExpressionType.Convert:
+                        case ExpressionType.Multiply:
+                        case ExpressionType.Subtract:
+                            return true;
+                    }
+
+                    return false;
 
                 case ExpressionType.Invoke:
                     var invocation = (InvocationExpression)expression;