Made the Test project compatible with StyleCop.

Author: RyanSchimpel

DotNet.SystemCollections.Analyzers.Test/Arrays/DoNotHaveFieldOfArrayTypeAnalyzerTest.cs

@@ -1,25 +1,38 @@
-using System;
-using DotNet.SystemCollections.Analyzers.Arrays;
-using DotNet.SystemCollections.Analyzers.Test.Verifiers;
-using Microsoft.CodeAnalysis;
-using Microsoft.CodeAnalysis.Diagnostics;
-using Microsoft.VisualStudio.TestTools.UnitTesting;
-
-namespace DotNet.SystemCollections.Analyzers.Test.Arrays
+namespace DotNet.SystemCollections.Analyzers.Test.Arrays
 {
+    using System;
+    using DotNet.SystemCollections.Analyzers.Arrays;
+    using DotNet.SystemCollections.Analyzers.Test.Verifiers;
+    using Microsoft.CodeAnalysis;
+    using Microsoft.CodeAnalysis.Diagnostics;
+    using Microsoft.VisualStudio.TestTools.UnitTesting;
+
+    /// <summary>
+    ///     This tester is used to test the <see cref="DoNotHaveFieldOfArrayTypeAnalyzer"/>.
+    /// </summary>
     [TestClass]
     public class DoNotHaveFieldOfArrayTypeAnalyzerTest : DiagnosticVerifier
     {
-        //No diagnostics expected to show up
+        /// <summary>
+        ///     Test when given an empty input.
+        /// </summary>
+        /// <remarks>
+        ///     No diagnostics expected to show up.
+        /// </remarks>
         [TestMethod]
         public void TestEmptyInput()
         {
-            var test = @"";
+            var test = string.Empty;
 
             this.VerifyCSharpDiagnostic(test);
         }
 
-        //Diagnostic triggered and checked for
+        /// <summary>
+        ///     Test when given a matching case.
+        /// </summary>
+        /// <remarks>
+        ///     Diagnostic triggered and checked for.
+        /// </remarks>
         [TestMethod]
         public void TestMatchingCase()
         {
@@ -41,18 +54,23 @@ class TypeName
             var expected = new DiagnosticResult
             {
                 Id = DoNotHaveFieldOfArrayTypeAnalyzer.DiagnosticId,
-                Message = String.Format(DoNotHaveFieldOfArrayTypeAnalyzer.MessageFormat, "field"),
+                Message = string.Format(DoNotHaveFieldOfArrayTypeAnalyzer.MessageFormat, "field"),
                 Severity = DiagnosticSeverity.Warning,
-                Locations =
-                    new[]
-                    {
-                        new DiagnosticResultLocation("Test0.cs", 13, 30)
-                    }
+                Locations = new[]
+                {
+                    new DiagnosticResultLocation("Test0.cs", 13, 30),
+                },
             };
 
             this.VerifyCSharpDiagnostic(test, expected);
         }
 
+        /// <summary>
+        ///     Gets the relevant C# diagnostic analyzer to test.
+        /// </summary>
+        /// <returns>
+        ///     This returns a new <see cref="DoNotHaveFieldOfArrayTypeAnalyzer"/>.
+        /// </returns>
         protected override DiagnosticAnalyzer GetCSharpDiagnosticAnalyzer()
         {
             return new DoNotHaveFieldOfArrayTypeAnalyzer();

DotNet.SystemCollections.Analyzers.Test/Arrays/DoNotHaveMethodReturnArrayTypeAnalyzerTest.cs

@@ -1,25 +1,38 @@
-using System;
-using DotNet.SystemCollections.Analyzers.Arrays;
-using DotNet.SystemCollections.Analyzers.Test.Verifiers;
-using Microsoft.CodeAnalysis;
-using Microsoft.CodeAnalysis.Diagnostics;
-using Microsoft.VisualStudio.TestTools.UnitTesting;
-
 namespace DotNet.SystemCollections.Analyzers.Test.Arrays
 {
+    using System;
+    using DotNet.SystemCollections.Analyzers.Arrays;
+    using DotNet.SystemCollections.Analyzers.Test.Verifiers;
+    using Microsoft.CodeAnalysis;
+    using Microsoft.CodeAnalysis.Diagnostics;
+    using Microsoft.VisualStudio.TestTools.UnitTesting;
+
+    /// <summary>
+    ///     This tester is used to test the <see cref="DoNotHaveMethodReturnArrayTypeAnalyzer"/>.
+    /// </summary>
     [TestClass]
     public class DoNotHaveMethodReturnArrayTypeAnalyzerTest : DiagnosticVerifier
     {
-        //No diagnostics expected to show up
+        /// <summary>
+        ///     Test when given an empty input.
+        /// </summary>
+        /// <remarks>
+        ///     No diagnostics expected to show up.
+        /// </remarks>
         [TestMethod]
         public void TestEmptyInput()
         {
-            var test = @"";
+            var test = string.Empty;
 
             this.VerifyCSharpDiagnostic(test);
         }
 
-        //Diagnostic triggered and checked for
+        /// <summary>
+        ///     Test when given a matching case.
+        /// </summary>
+        /// <remarks>
+        ///     Diagnostic triggered and checked for.
+        /// </remarks>
         [TestMethod]
         public void TestMatchingCase()
         {
@@ -43,18 +56,23 @@ object[] MethodName()
             var expected = new DiagnosticResult
             {
                 Id = DoNotHaveMethodReturnArrayTypeAnalyzer.DiagnosticId,
-                Message = String.Format(DoNotHaveMethodReturnArrayTypeAnalyzer.MessageFormat, "MethodName"),
+                Message = string.Format(DoNotHaveMethodReturnArrayTypeAnalyzer.MessageFormat, "MethodName"),
                 Severity = DiagnosticSeverity.Warning,
-                Locations =
-                    new[]
-                    {
-                        new DiagnosticResultLocation("Test0.cs", 13, 22)
-                    }
+                Locations = new[]
+                {
+                    new DiagnosticResultLocation("Test0.cs", 13, 22),
+                },
             };
 
             this.VerifyCSharpDiagnostic(test, expected);
         }
 
+        /// <summary>
+        ///     This is used to return the appropriate C# analyzer to test.
+        /// </summary>
+        /// <returns>
+        ///     Returns a new <see cref="DoNotHaveMethodReturnArrayTypeAnalyzer"/>.
+        /// </returns>
         protected override DiagnosticAnalyzer GetCSharpDiagnosticAnalyzer()
         {
             return new DoNotHaveMethodReturnArrayTypeAnalyzer();

DotNet.SystemCollections.Analyzers.Test/Arrays/DoNotHavePropertyOfArrayTypeAnalyzerTest.cs

@@ -1,25 +1,38 @@
-using System;
-using DotNet.SystemCollections.Analyzers.Arrays;
-using DotNet.SystemCollections.Analyzers.Test.Verifiers;
-using Microsoft.CodeAnalysis;
-using Microsoft.CodeAnalysis.Diagnostics;
-using Microsoft.VisualStudio.TestTools.UnitTesting;
-
 namespace DotNet.SystemCollections.Analyzers.Test.Arrays
 {
+    using System;
+    using DotNet.SystemCollections.Analyzers.Arrays;
+    using DotNet.SystemCollections.Analyzers.Test.Verifiers;
+    using Microsoft.CodeAnalysis;
+    using Microsoft.CodeAnalysis.Diagnostics;
+    using Microsoft.VisualStudio.TestTools.UnitTesting;
+
+    /// <summary>
+    ///     This tester is used to test the <see cref="DoNotHavePropertyOfArrayTypeAnalyzer"/>.
+    /// </summary>
     [TestClass]
     public class DoNotHavePropertyOfArrayTypeAnalyzerTest : DiagnosticVerifier
     {
-        //No diagnostics expected to show up
+        /// <summary>
+        ///     Test when given empty input.
+        /// </summary>
+        /// <remarks>
+        ///     No diagnostics expected to show up.
+        /// </remarks>
         [TestMethod]
         public void TestEmptyInput()
         {
-            var test = @"";
+            var test = string.Empty;
 
             this.VerifyCSharpDiagnostic(test);
         }
 
-        //Diagnostic triggered and checked for
+        /// <summary>
+        ///     Test when given a matching case,.
+        /// </summary>
+        /// <remarks>
+        ///     Diagnostic triggered and checked for.
+        /// </remarks>
         [TestMethod]
         public void TestMatchingCase()
         {
@@ -41,18 +54,23 @@ class TypeName
             var expected = new DiagnosticResult
             {
                 Id = DoNotHavePropertyOfArrayTypeAnalyzer.DiagnosticId,
-                Message = String.Format(DoNotHavePropertyOfArrayTypeAnalyzer.MessageFormat, "Property"),
+                Message = string.Format(DoNotHavePropertyOfArrayTypeAnalyzer.MessageFormat, "Property"),
                 Severity = DiagnosticSeverity.Warning,
-                Locations =
-                    new[]
-                    {
-                        new DiagnosticResultLocation("Test0.cs", 13, 30)
-                    }
+                Locations = new[]
+                {
+                    new DiagnosticResultLocation("Test0.cs", 13, 30),
+                },
             };
 
             this.VerifyCSharpDiagnostic(test, expected);
         }
 
+        /// <summary>
+        ///     This is used to get the relevant C# analyzer to test.
+        /// </summary>
+        /// <returns>
+        ///     Returns a new <see cref="DoNotHavePropertyOfArrayTypeAnalyzer"/>.
+        /// </returns>
         protected override DiagnosticAnalyzer GetCSharpDiagnosticAnalyzer()
         {
             return new DoNotHavePropertyOfArrayTypeAnalyzer();