Merge pull request #64 from bjornhellander/feature/sa1003-null-suppression-before-invocation

Update SA1003SymbolsMustBeSpacedCorrectly to forbid space between a postfix operator and an opening parenthesis

Author: bjornhellander

StyleCop.Analyzers/StyleCop.Analyzers.Test.CSharp8/SpacingRules/SA1003CSharp8UnitTests.cs

@@ -185,6 +185,38 @@ public void TestMethod(string? x)
         }
     }
 }
+";
+
+            await VerifyCSharpFixAsync(testCode, DiagnosticResult.EmptyDiagnosticResults, fixedCode, CancellationToken.None).ConfigureAwait(false);
+        }
+
+        [Fact]
+        public async Task TestNullForgivingOperatorBeforeInvocationAsync()
+        {
+            var testCode = @"
+namespace TestNamespace
+{
+    public class TestClass
+    {
+        public void TestMethod(System.Action? x)
+        {
+            x[|!|] ();
+        }
+    }
+}
+";
+
+            var fixedCode = @"
+namespace TestNamespace
+{
+    public class TestClass
+    {
+        public void TestMethod(System.Action? x)
+        {
+            x!();
+        }
+    }
+}
 ";
 
             await VerifyCSharpFixAsync(testCode, DiagnosticResult.EmptyDiagnosticResults, fixedCode, CancellationToken.None).ConfigureAwait(false);

StyleCop.Analyzers/StyleCop.Analyzers/SpacingRules/SA1003SymbolsMustBeSpacedCorrectly.cs

@@ -304,6 +304,7 @@ private static void HandlePostfixUnaryExpression(SyntaxNodeAnalysisContext conte
             {
             case SyntaxKind.CloseParenToken:
             case SyntaxKind.OpenBracketToken: // Example: x![i]
+            case SyntaxKind.OpenParenToken: // Example: x!()
             case SyntaxKind.CloseBracketToken:
             case SyntaxKind.SemicolonToken:
             case SyntaxKind.CommaToken: