Report SA1108 on line end (fix #2941)

Author: pantosha

StyleCop.Analyzers/StyleCop.Analyzers.Test/SpacingRules/SA1008UnitTests.cs

@@ -137,11 +137,44 @@ public void TestMethod1(
             int z)
         {
         }
+
+        public void TestMethod2 (
+            int x,
+            int y,
+            int z)
+        {
+        }
     }
 }
 ";
 
-            await VerifyCSharpDiagnosticAsync(testCode, DiagnosticResult.EmptyDiagnosticResults, CancellationToken.None).ConfigureAwait(false);
+            var fixedTestCode = @"namespace TestNamespace
+{
+    public class TestClass
+    {
+        public void TestMethod1(
+            int x,
+            int y,
+            int z)
+        {
+        }
+
+        public void TestMethod2(
+            int x,
+            int y,
+            int z)
+        {
+        }
+    }
+}
+";
+
+            DiagnosticResult[] expectedDiagnostics =
+            {
+                Diagnostic(DescriptorNotPreceded).WithLocation(12, 33),
+            };
+
+            await VerifyCSharpFixAsync(testCode, expectedDiagnostics, fixedTestCode, CancellationToken.None).ConfigureAwait(false);
         }
 
         /// <summary>
@@ -781,6 +814,71 @@ public TestClass()
             await VerifyCSharpFixAsync(testCode, expectedDiagnostics, fixedTestCode, CancellationToken.None).ConfigureAwait(false);
         }
 
+        /// <summary>
+        /// Verifies that spacing for multiline argument lists is handled properly.
+        /// </summary>
+        /// <returns>A <see cref="Task"/> representing the asynchronous unit test.</returns>
+        [Fact]
+        public async Task TestMultiLineArgumentListAsync()
+        {
+            var testCode = @"using System;
+
+namespace TestNamespace
+{
+    public class TestClass
+    {
+        public TestClass(int x, int y)
+        {
+            var s1 = new String(
+                'a',
+                x);
+
+            var s2 = new String (
+                'a',
+                y);
+
+            // Opening parenthesis followed by space
+            var s3 = new String( 
+                'a',
+                x);
+        }
+    }
+}
+";
+
+            var fixedTestCode = @"using System;
+
+namespace TestNamespace
+{
+    public class TestClass
+    {
+        public TestClass(int x, int y)
+        {
+            var s1 = new String(
+                'a',
+                x);
+
+            var s2 = new String(
+                'a',
+                y);
+
+            // Opening parenthesis followed by space
+            var s3 = new String('a',
+                x);
+        }
+    }
+}
+";
+
+            DiagnosticResult[] expectedDiagnostics =
+            {
+                Diagnostic(DescriptorNotPreceded).WithLocation(13, 33),
+                Diagnostic(DescriptorNotFollowed).WithLocation(18, 32),
+            };
+
+            await VerifyCSharpFixAsync(testCode, expectedDiagnostics, fixedTestCode, CancellationToken.None).ConfigureAwait(false);
+        }
+
         /// <summary>
         /// Verifies that spacing for while statements is handled properly.
         /// </summary>

StyleCop.Analyzers/StyleCop.Analyzers/SpacingRules/SA1008OpeningParenthesisMustBeSpacedCorrectly.cs

@@ -92,12 +92,6 @@ private static void HandleOpenParenToken(SyntaxTreeAnalysisContext context, Synt
                 return;
             }
 
-            if (token.IsLastInLine())
-            {
-                // ignore open parenthesis when last on line.
-                return;
-            }
-
             var prevToken = token.GetPreviousToken();
 
             // Don't check leading spaces when preceded by a keyword that is already handled by SA1000