Update tests

Author: CyrusNajmabadi

src/EditorFeatures/Test/CodeGeneration/ExpressionGenerationTests.cs

@@ -265,7 +265,7 @@ public void TestConditionalExpression1()
                 f.IdentifierName("E"),
                 f.IdentifierName("T"),
                 f.IdentifierName("F")),
-            cs: "(E) ? (T) : (F)",
+            cs: "E ? T : F",
             csSimple: "E ? T : F",
             vb: "If(E, T, F)",
             vbSimple: "If(E, T, F)");
@@ -412,9 +412,9 @@ public void TestIsExpression()
             f => f.IsTypeExpression(
                 f.IdentifierName("a"),
                 CreateClass("SomeType")),
-            cs: "(a) is SomeType",
+            cs: "a is SomeType",
             csSimple: "a is SomeType",
-            vb: "TypeOf (a) Is SomeType",
+            vb: "TypeOf a Is SomeType",
             vbSimple: "TypeOf a Is SomeType");
 
     [Fact]
@@ -423,7 +423,7 @@ public void TestAsExpression()
             f => f.TryCastExpression(
                 f.IdentifierName("a"),
                 CreateClass("SomeType")),
-            cs: "(a) as SomeType",
+            cs: "a as SomeType",
             csSimple: "a as SomeType",
             vb: "TryCast(a, SomeType)",
             vbSimple: "TryCast(a, SomeType)");
@@ -433,9 +433,9 @@ public void TestNotExpression()
         => Test(
             f => f.LogicalNotExpression(
                 f.IdentifierName("a")),
-            cs: "!(a)",
+            cs: "!a",
             csSimple: "!a",
-            vb: "Not (a)",
+            vb: "Not a",
             vbSimple: "Not a");
 
     [Fact]
@@ -444,7 +444,7 @@ public void TestCastExpression()
             f => f.CastExpression(
                 CreateClass("SomeType"),
                 f.IdentifierName("a")),
-            cs: "(SomeType)(a)",
+            cs: "(SomeType)a",
             csSimple: "(SomeType)a",
             vb: "DirectCast(a, SomeType)",
             vbSimple: "DirectCast(a, SomeType)");
@@ -454,8 +454,8 @@ public void TestNegateExpression()
         => Test(
             f => f.NegateExpression(
                 f.IdentifierName("a")),
-            cs: "-(a)",
+            cs: "-a",
             csSimple: "-a",
-            vb: "-(a)",
+            vb: "-a",
             vbSimple: "-a");
 }

src/EditorFeatures/Test/CodeGeneration/ExpressionPrecedenceGenerationTests.cs

@@ -190,7 +190,7 @@ public void TestConditionalExpression1()
                     f.IdentifierName("E2")),
                 f.IdentifierName("T"),
                 f.IdentifierName("F")),
-            cs: "(E1 = (E2)) ? (T) : (F)",
+            cs: "(E1 = E2) ? T : F",
             csSimple: "(E1 = E2) ? T : F",
             vb: null,
             vbSimple: null);
@@ -207,7 +207,7 @@ public void TestConditionalExpression2()
                         f.IdentifierName("E2"),
                         f.IdentifierName("T2"),
                         f.IdentifierName("F2"))),
-            cs: "((E1) ? (T1) : (F1)) + ((E2) ? (T2) : (F2))",
+            cs: "(E1 ? T1 : F1) + (E2 ? T2 : F2)",
             csSimple: "(E1 ? T1 : F1) + (E2 ? T2 : F2)",
             vb: null,
             vbSimple: null);
@@ -233,9 +233,9 @@ public void TestMemberAccessOffOfIsExpression()
                     f.IdentifierName("a"),
                     CreateClass("SomeType")),
                 f.IdentifierName("M")),
-            cs: "((a) is SomeType).M",
+            cs: "(a is SomeType).M",
             csSimple: "(a is SomeType).M",
-            vb: "(TypeOf (a) Is SomeType).M",
+            vb: "(TypeOf a Is SomeType).M",
             vbSimple: "(TypeOf a Is SomeType).M");
 
     [Fact]
@@ -259,7 +259,7 @@ public void TestMemberAccessOffOfAsExpression()
                     f.IdentifierName("a"),
                     CreateClass("SomeType")),
                 f.IdentifierName("M")),
-            cs: "((a) as SomeType).M",
+            cs: "(a as SomeType).M",
             csSimple: "(a as SomeType).M",
             vb: "(TryCast(a, SomeType)).M",
             vbSimple: "TryCast(a, SomeType).M");
@@ -284,9 +284,9 @@ public void TestMemberAccessOffOfNotExpression()
                 f.LogicalNotExpression(
                     f.IdentifierName("a")),
                 f.IdentifierName("M")),
-            cs: "(!(a)).M",
+            cs: "(!a).M",
             csSimple: "(!a).M",
-            vb: "(Not (a)).M",
+            vb: "(Not a).M",
             vbSimple: "(Not a).M");
 
     [Fact]
@@ -309,7 +309,7 @@ public void TestMemberAccessOffOfCastExpression()
                     CreateClass("SomeType"),
                     f.IdentifierName("a")),
                 f.IdentifierName("M")),
-            cs: "((SomeType)(a)).M",
+            cs: "((SomeType)a).M",
             csSimple: "((SomeType)a).M",
             vb: "(DirectCast(a, SomeType)).M",
             vbSimple: "DirectCast(a, SomeType).M");
@@ -322,9 +322,9 @@ public void TestCastOfAddExpression()
                 f.AddExpression(
                     f.IdentifierName("a"),
                     f.IdentifierName("b"))),
-            cs: "(SomeType)((a) + (b))",
+            cs: "(SomeType)(a + b)",
             csSimple: "(SomeType)(a + b)",
-            vb: "DirectCast((a) + (b), SomeType)",
+            vb: "DirectCast(a + b, SomeType)",
             vbSimple: "DirectCast(a + b, SomeType)");
 
     [Fact]
@@ -334,9 +334,9 @@ public void TestNegateOfAddExpression()
                 f.AddExpression(
                     f.IdentifierName("a"),
                     f.IdentifierName("b"))),
-            cs: "-((a) + (b))",
+            cs: "-(a + b)",
             csSimple: "-(a + b)",
-            vb: "-((a) + (b))",
+            vb: "-(a + b)",
             vbSimple: "-(a + b)");
 
     [Fact]
@@ -346,9 +346,9 @@ public void TestMemberAccessOffOfNegate()
                 f.NegateExpression(
                     f.IdentifierName("a")),
                 f.IdentifierName("M")),
-            cs: "(-(a)).M",
+            cs: "(-a).M",
             csSimple: "(-a).M",
-            vb: "(-(a)).M",
+            vb: "(-a).M",
             vbSimple: "(-a).M");
 
     [Fact]