Enable SA1133 at "Error" severity, and fix all violations

Author: sharwell

StyleCop.Analyzers/StyleCop.Analyzers.Test/LayoutRules/SA1503UnitTests.cs

@@ -410,7 +410,8 @@ protected override CodeFixProvider GetCSharpCodeFixProvider()
         /// <param name="statementText">The source code for the first part of a compound statement whose child can be
         /// either a statement block or a single statement.</param>
         /// <returns>A <see cref="Task"/> representing the asynchronous unit test.</returns>
-        [Theory, MemberData(nameof(TestStatements))]
+        [Theory]
+        [MemberData(nameof(TestStatements))]
         private async Task TestCodeFixForStatementAsync(string statementText)
         {
             await this.VerifyCSharpFixAsync(this.GenerateTestStatement(statementText), this.GenerateFixedTestStatement(statementText)).ConfigureAwait(false);

StyleCop.Analyzers/StyleCop.Analyzers.ruleset

@@ -86,6 +86,7 @@
     <Rule Id="SA1119" Action="Error" />
     <Rule Id="SA1121" Action="Error" />
     <Rule Id="SA1122" Action="Error" />
+    <Rule Id="SA1133" Action="Error" />
     <Rule Id="SA1300" Action="Error" />
     <Rule Id="SA1302" Action="Error" />
     <Rule Id="SA1303" Action="Error" />