Add regression tests for #1834

Author: sharwell

StyleCop.Analyzers/StyleCop.Analyzers.Test/SpacingRules/SA1009UnitTests.cs

@@ -60,6 +60,33 @@ public void Method()
             await this.VerifyCSharpFixAsync(testCode, fixedCode).ConfigureAwait(false);
         }
 
+        [Fact]
+        public async Task TestDocumentationMethodReferenceWithWhitespaceBeforeClosingParenthesisAsync()
+        {
+            const string testCode = @"
+public class Foo
+{
+    /// <see cref=""Method( )""/>
+    public void Method()
+    {
+    }
+}";
+            const string fixedCode = @"
+public class Foo
+{
+    /// <see cref=""Method()""/>
+    public void Method()
+    {
+    }
+}";
+
+            DiagnosticResult expected = this.CSharpDiagnostic().WithArguments(" not", "preceded").WithLocation(4, 28);
+
+            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);
+        }
+
         [Fact]
         public async Task TestMethodWith2CorrectlySpacedParametersAsync()
         {
@@ -656,6 +683,13 @@ public void TestMethod2(
     }
 
     public int TestMethod3(int a) { return a; }
+
+    public int TestMethod4(string[] args)
+    {
+#if !(X || NOT )
+        return 1;
+#endif
+    }
 }
 ";
 
@@ -687,6 +721,13 @@ public void TestMethod2(
     }
 
     public int TestMethod3(int a) { return a; }
+
+    public int TestMethod4(string[] args)
+    {
+#if !(X || NOT)
+        return 1;
+#endif
+    }
 }
 ";
 
@@ -695,7 +736,8 @@ public void TestMethod2(
                 this.CSharpDiagnostic().WithLocation(10, 9).WithArguments(" not", "preceded"),
                 this.CSharpDiagnostic().WithLocation(16, 7).WithArguments(" not", "preceded"),
                 this.CSharpDiagnostic().WithLocation(21, 19).WithArguments(" not", "preceded"),
-                this.CSharpDiagnostic().WithLocation(25, 17).WithArguments(" not", "preceded")
+                this.CSharpDiagnostic().WithLocation(25, 17).WithArguments(" not", "preceded"),
+                this.CSharpDiagnostic().WithLocation(32, 16).WithArguments(" not", "preceded"),
             };
 
             await this.VerifyCSharpDiagnosticAsync(testCode, expected, CancellationToken.None).ConfigureAwait(false);