Also add records as supported member kinds

Author: sharwell

StyleCop.Analyzers/StyleCop.Analyzers.Test/OrderingRules/SA1202UnitTests.cs

@@ -104,20 +104,22 @@ private class TestClass10 { }
         }
 
         /// <summary>
-        /// Verifies that the analyzer will properly handle class access levels.
+        /// Verifies that the analyzer will properly handle type access levels.
         /// </summary>
+        /// <param name="keyword">The keyword used to declare the type.</param>
         /// <returns>A <see cref="Task"/> representing the asynchronous unit test.</returns>
-        [Fact]
-        public async Task TestClassOrderingAsync()
+        [Theory]
+        [MemberData(nameof(CommonMemberData.TypeDeclarationKeywords), MemberType = typeof(CommonMemberData))]
+        public async Task TestTypeOrderingAsync(string keyword)
         {
-            var testCode = @"internal class TestClass1 { }
-public class TestClass2 { }
+            var testCode = $@"internal {keyword} TestClass1 {{ }}
+public {keyword} {{|#0:TestClass2|}} {{ }}
 ";
 
-            var expected = Diagnostic().WithLocation(2, 14).WithArguments("public", "internal");
+            var expected = Diagnostic().WithLocation(0).WithArguments("public", "internal");
 
-            var fixedCode = @"public class TestClass2 { }
-internal class TestClass1 { }
+            var fixedCode = $@"public {keyword} TestClass2 {{ }}
+internal {keyword} TestClass1 {{ }}
 ";
 
             await VerifyCSharpFixAsync(testCode, expected, fixedCode, CancellationToken.None).ConfigureAwait(false);
@@ -126,12 +128,14 @@ internal class TestClass1 { }
         /// <summary>
         /// Verifies that the analyzer will properly handle interfaces before classes.
         /// </summary>
+        /// <param name="keyword">The keyword used to declare the type.</param>
         /// <returns>A <see cref="Task"/> representing the asynchronous unit test.</returns>
-        [Fact]
-        public async Task TestInternalInterfaceBeforePublicClassAsync()
+        [Theory]
+        [MemberData(nameof(CommonMemberData.DataTypeDeclarationKeywords), MemberType = typeof(CommonMemberData))]
+        public async Task TestInternalInterfaceBeforePublicClassAsync(string keyword)
         {
-            var testCode = @"internal interface ITestInterface { }
-public class TestClass2 { }
+            var testCode = $@"internal interface ITestInterface {{ }}
+public {keyword} TestClass2 {{ }}
 ";
 
             await VerifyCSharpDiagnosticAsync(testCode, DiagnosticResult.EmptyDiagnosticResults, CancellationToken.None).ConfigureAwait(false);

StyleCop.Analyzers/StyleCop.Analyzers/OrderingRules/SA1202ElementsMustBeOrderedByAccess.cs

@@ -66,7 +66,9 @@ internal class SA1202ElementsMustBeOrderedByAccess : DiagnosticAnalyzer
             SyntaxKind.IndexerDeclaration,
             SyntaxKind.MethodDeclaration,
             SyntaxKind.ConversionOperatorDeclaration,
-            SyntaxKind.OperatorDeclaration);
+            SyntaxKind.OperatorDeclaration,
+            SyntaxKindEx.RecordDeclaration,
+            SyntaxKindEx.RecordStructDeclaration);
 
         private static readonly Action<SyntaxNodeAnalysisContext, StyleCopSettings> CompilationUnitAction = HandleCompilationUnit;
         private static readonly Action<SyntaxNodeAnalysisContext, StyleCopSettings> BaseNamespaceDeclarationAction = HandleBaseNamespaceDeclaration;