Update SA1013 for nullable reference types

Author: sharwell

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

@@ -88,6 +88,54 @@ public void TestMethod()
         };
     }
 }
+";
+
+            await VerifyCSharpDiagnosticAsync(testCode, DiagnosticResult.EmptyDiagnosticResults, CancellationToken.None).ConfigureAwait(false);
+        }
+
+        /// <summary>
+        /// Validates that a closing brace followed by a null-forgiving operator is allowed after an object initializer.
+        /// </summary>
+        /// <returns>A <see cref="Task"/> representing the asynchronous unit test.</returns>
+        [Fact]
+        [WorkItem(3006, "https://github.com/DotNetAnalyzers/StyleCopAnalyzers/issues/3006")]
+        public async Task TestObjectInitializerWithNullForgivingOperatorAsync()
+        {
+            const string testCode = @"#nullable enable
+public class Foo
+{
+    public Foo? Value { get; set; }
+
+    public void TestMethod()
+    {
+        var item = new Foo { Value = null }!;
+        var other = new Foo { }!;
+    }
+}
+";
+
+            await VerifyCSharpDiagnosticAsync(testCode, DiagnosticResult.EmptyDiagnosticResults, CancellationToken.None).ConfigureAwait(false);
+        }
+
+        /// <summary>
+        /// Validates that a closing brace followed by a null-forgiving operator is allowed after a collection initializer.
+        /// </summary>
+        /// <returns>A <see cref="Task"/> representing the asynchronous unit test.</returns>
+        [Fact]
+        [WorkItem(3006, "https://github.com/DotNetAnalyzers/StyleCopAnalyzers/issues/3006")]
+        public async Task TestCollectionInitializerWithNullForgivingOperatorAsync()
+        {
+            const string testCode = @"#nullable enable
+using System.Collections.Generic;
+
+public class Foo
+{
+    public void TestMethod()
+    {
+        var numbers = new List<int> { 1, 2, 3 }!;
+        var nested = new List<List<int>> { new List<int> { 1, 2 }!, new List<int> { 3 }! };
+    }
+}
 ";
 
             await VerifyCSharpDiagnosticAsync(testCode, DiagnosticResult.EmptyDiagnosticResults, CancellationToken.None).ConfigureAwait(false);

documentation/SA1013.md

@@ -23,7 +23,7 @@ A closing brace within a C# element is not spaced correctly.
 
 A violation of this rule occurs when the spacing around a closing brace is not correct.
 
-A closing brace should always be followed by a single space, unless it is the last character on the line, or unless it is followed by a closing parenthesis, a comma, or a semicolon.
+A closing brace should always be followed by a single space, unless it is the last character on the line, or unless it is followed by a closing parenthesis, a comma, or a semicolon. The null-forgiving operator (`!`) may also appear immediately after a closing brace without a space (for example, `new Foo { Value = null }!`).
 
 A closing brace should always be preceded by a single space, unless it is the first character on the line.