Surrounding negated assignments in parenthesis

Author: SteveWilkes

ReadableExpressions.UnitTests/WhenTranslatingAssignments.cs

@@ -221,6 +221,19 @@ public void ShouldNotWrapAnAssignmentValueInParentheses()
             Assert.AreEqual("i = 1 * 2", translated);
         }
 
+        [TestMethod]
+        public void ShouldTranslateANegatedBooleanAssignment()
+        {
+            var boolVariable1 = Expression.Variable(typeof(bool), "isItNot");
+            var boolVariable2 = Expression.Variable(typeof(bool), "isIt");
+            var assignBool = Expression.Assign(boolVariable1, Expression.IsFalse(boolVariable2));
+            var negated = Expression.Not(assignBool);
+
+            var translated = negated.ToReadableString();
+
+            Assert.AreEqual("!(isItNot = !isIt)", translated);
+        }
+
         [TestMethod]
         public void ShouldWrapAnAssignmentTernaryTestInParentheses()
         {

ReadableExpressions/Translators/NegationExpressionTranslator.cs

@@ -32,7 +32,14 @@ public string TranslateNot(Expression expression, TranslationContext context)
 
         private static string Translate(ExpressionType negationType, Expression expression, TranslationContext context)
         {
-            return _negationsByNodeType[negationType] + context.Translate(expression);
+            var valueToNegate = context.Translate(expression);
+
+            if (valueToNegate.Contains(" "))
+            {
+                valueToNegate = valueToNegate.WithSurroundingParentheses(checkExisting: true);
+            }
+
+            return _negationsByNodeType[negationType] + valueToNegate;
         }
     }
 }