Tweaked test specification.

Originally I had intended to remove any extraneous parentheses such as
those that surround language expressions. Unfortunately, this proved to be
more difficult than I had anticipated as it is hard to differentiate said
extraneous parentheses from meaningful ones such as those used to invoke a
method. It is for these reasons that I have changed the test specification
to allow extraneous parentheses.

Author: AlexArchive

TestStack.FluentMVCTesting.Tests/Internal/ExpressionInspectorTests.cs

@@ -14,7 +14,7 @@ public void Correctly_parse_equality_comparison_with_string_operands()
             Expression<Func<string, bool>> func = text => text == "any";
             ExpressionInspector sut = new ExpressionInspector();
             var actual = sut.Inspect(func);
-            Assert.AreEqual("text => text == \"any\"", actual);
+            Assert.AreEqual("text => (text == \"any\")", actual);
         }
 
         [Test]
@@ -23,7 +23,7 @@ public void Correctly_parse_equality_comparison_with_int_operands()
             Expression<Func<int, bool>> func = number => number == 5;
             ExpressionInspector sut = new ExpressionInspector();
             var actual = sut.Inspect(func);
-            Assert.AreEqual("number => number == 5", actual);
+            Assert.AreEqual("number => (number == 5)", actual);
         }
 
         [Test]
@@ -32,7 +32,7 @@ public void Correctly_parse_equality_comparison_with_property_operand()
             Expression<Func<PostViewModel, bool>> func = post => post.Title == "A";
             ExpressionInspector sut = new ExpressionInspector();
             var actual = sut.Inspect(func);
-            Assert.AreEqual("post => post.Title == \"A\"", actual);
+            Assert.AreEqual("post => (post.Title == \"A\")", actual);
         }
 
         [Test]
@@ -41,7 +41,7 @@ public void Correctly_parse_equality_comparison_with_property_operands()
             Expression<Func<PostViewModel, bool>> func = post => post.Title == post.Slug;
             ExpressionInspector sut = new ExpressionInspector();
             var actual = sut.Inspect(func);
-            Assert.AreEqual("post => post.Title == post.Slug", actual);
+            Assert.AreEqual("post => (post.Title == post.Slug)", actual);
         }
 
         [Test]
@@ -50,7 +50,7 @@ public void Correctly_parse_inequality_comparison()
             Expression<Func<int, bool>> func = number => number != 5;
             ExpressionInspector sut = new ExpressionInspector();
             var actual = sut.Inspect(func);
-            Assert.AreEqual("number => number != 5", actual);
+            Assert.AreEqual("number => (number != 5)", actual);
         }
 
         [Test]
@@ -60,7 +60,8 @@ public void Correctly_parse_equality_comparison_with_captured_constant_operand()
             Expression<Func<int, bool>> func = number => number == Number;
             ExpressionInspector sut = new ExpressionInspector();
             var actual = sut.Inspect(func);
-            Assert.AreEqual("number => number == " + Number, actual);
+            Assert.AreEqual(
+                string.Concat("number => (number == ", Number, ")"), actual);
         }
 
         [Test]
@@ -69,7 +70,7 @@ public void Correctly_parse_relational_comparison()
             Expression<Func<int, bool>> func = number => number < 5;
             ExpressionInspector sut = new ExpressionInspector();
             var actual = sut.Inspect(func);
-            Assert.AreEqual("number => number < 5", actual);
+            Assert.AreEqual("number => (number < 5)", actual);
         }
 
         [Test]
@@ -79,7 +80,7 @@ public void Correctly_parse_conditional_or_operator()
                 text => text == "any" || text.Length == 3;
             ExpressionInspector sut = new ExpressionInspector();
             var actual = sut.Inspect(func);
-            Assert.AreEqual("text => text == \"any\" || text.Length == 3", actual);
+            Assert.AreEqual("text => ((text == \"any\") || (text.Length == 3))", actual);
         }
 
         [Test]
@@ -89,7 +90,7 @@ public void Correctly_parse_two_conditional_or_operators()
                 text => text == "any" || text.Length == 3 || text.Length == 9;
             ExpressionInspector sut = new ExpressionInspector();
             var actual = sut.Inspect(func);
-            Assert.AreEqual("text => text == \"any\" || text.Length == 3 || text.Length == 9", actual);
+            Assert.AreEqual("text => (((text == \"any\") || (text.Length == 3)) || (text.Length == 9))", actual);
         }
 
         [Test]
@@ -99,7 +100,7 @@ public void Not_mistake_property_called_OrElse_for_conditional_or_operator()
                 post => post.Title == "" || post.OrElse == "";
             ExpressionInspector sut = new ExpressionInspector();
             var actual = sut.Inspect(func);
-            Assert.AreEqual("post => post.Title == \"\" || post.OrElse == \"\"", actual);
+            Assert.AreEqual("post => ((post.Title == \"\") || (post.OrElse == \"\"))", actual);
         }
 
         [Test]

TestStack.FluentMvcTesting/Internal/ExpressionInspector.cs

@@ -7,7 +7,7 @@ internal class ExpressionInspector
     {
         internal string Inspect(LambdaExpression expression)
         {
-            return Regex.Replace(expression.ToString(), "[()]", "")
+            return expression.ToString()
                 .Replace(" OrElse ", " || ");
         }
     }