Removing parenthesis wrapping from negated method calls

Author: SteveWilkes

ReadableExpressions.UnitTests/WhenTranslatingMemberAccesses.cs

@@ -291,6 +291,17 @@ public void ShouldTranslateAnOutParameterMethodCallWithoutGenericArgumentInclude
             Assert.AreEqual("new ValueConverter().TryConvert(i, out result)", translated);
         }
 
+        [TestMethod]
+        public void ShouldTranslateANegatedMethodCall()
+        {
+            Expression<Func<List<int>, bool>> listDoesNotContainZero =
+                l => !l.Contains(0, EqualityComparer<int>.Default);
+
+            var translated = listDoesNotContainZero.Body.ToReadableString();
+
+            Assert.AreEqual("!l.Contains(0, EqualityComparer<int>.Default)", translated);
+        }
+
         [TestMethod]
         public void ShouldNotIncludeCapturedInstanceNames()
         {

ReadableExpressions/Translators/NegationExpressionTranslator.cs

@@ -34,12 +34,22 @@ private static string Translate(ExpressionType negationType, Expression expressi
         {
             var valueToNegate = context.Translate(expression);
 
-            if (valueToNegate.Contains(" "))
+            if (WrapNegatedValue(valueToNegate, expression))
             {
                 valueToNegate = valueToNegate.WithSurroundingParentheses(checkExisting: true);
             }
 
             return _negationsByNodeType[negationType] + valueToNegate;
         }
+
+        private static bool WrapNegatedValue(string value, Expression expression)
+        {
+            if (expression.NodeType == ExpressionType.Call)
+            {
+                return false;
+            }
+
+            return value.Contains(" ");
+        }
     }
 }