Merge pull request #2671 from dlemstra/SA1119

Enabled unit tests (#2398).

Author: sharwell

StyleCop.Analyzers/StyleCop.Analyzers.Test.CSharp7/MaintainabilityRules/SA1119CSharp7UnitTests.cs

@@ -12,11 +12,12 @@ namespace StyleCop.Analyzers.Test.CSharp7.MaintainabilityRules
     public class SA1119CSharp7UnitTests : SA1119UnitTests
     {
         /// <summary>
-        /// Verifies that extra parentheses in pattern matching are reported.
+        /// Verifies that extra parentheses in pattern matching is not reported.
         /// </summary>
         /// <returns>A <see cref="Task"/> representing the asynchronous unit test.</returns>
         /// <seealso cref="SA1408CSharp7UnitTests.TestPatternMatchingAsync"/>
-        [Fact(Skip = "https://github.com/DotNetAnalyzers/StyleCopAnalyzers/issues/2398")]
+        [Fact]
+        [WorkItem(2372, "https://github.com/DotNetAnalyzers/StyleCopAnalyzers/issues/2398")]
         public async Task TestPatternMatchingAsync()
         {
             var testCode = @"public class Foo
@@ -29,27 +30,8 @@ public void Bar()
         }
     }
 }";
-            var fixedCode = @"public class Foo
-{
-    public void Bar()
-    {
-        if (new object() is bool b && b)
-        {
-            return;
-        }
-    }
-}";
-
-            DiagnosticResult[] expected =
-            {
-                this.CSharpDiagnostic(DiagnosticId).WithLocation(5, 13),
-                this.CSharpDiagnostic(ParenthesesDiagnosticId).WithLocation(5, 13),
-                this.CSharpDiagnostic(ParenthesesDiagnosticId).WithLocation(5, 36),
-            };
 
-            await this.VerifyCSharpDiagnosticAsync(testCode, expected, CancellationToken.None).ConfigureAwait(false);
-            await this.VerifyCSharpDiagnosticAsync(fixedCode, EmptyDiagnosticResults, CancellationToken.None).ConfigureAwait(false);
-            await this.VerifyCSharpFixAsync(testCode, fixedCode).ConfigureAwait(false);
+            await this.VerifyCSharpDiagnosticAsync(testCode, EmptyDiagnosticResults, CancellationToken.None).ConfigureAwait(false);
         }
 
         [Fact]