Merge pull request #2708 from vweijsters/fix-2699

Fixed issue where SA1002 was reported when a semicolon is the last ch…

Author: sharwell

StyleCop.Analyzers/StyleCop.Analyzers.Test/SpacingRules/SA1002UnitTests.cs

@@ -326,12 +326,8 @@ Comment Line 2 */
             await this.VerifyCSharpFixAsync(testCode, fixedCode, cancellationToken: CancellationToken.None).ConfigureAwait(false);
         }
 
-        /// <summary>
-        /// This is a regression test for DotNetAnalyzers/StyleCopAnalyzers#1426.
-        /// https://github.com/DotNetAnalyzers/StyleCopAnalyzers/issues/1426
-        /// </summary>
-        /// <returns>A <see cref="Task"/> representing the asynchronous unit test.</returns>
         [Fact]
+        [WorkItem(1426, "https://github.com/DotNetAnalyzers/StyleCopAnalyzers/issues/1426")]
         public async Task TestPrecededByBlockCommentAsync()
         {
             string testCode = @"
@@ -360,12 +356,8 @@ void MethodName()
             await this.VerifyCSharpFixAsync(testCode, fixedCode, cancellationToken: CancellationToken.None).ConfigureAwait(false);
         }
 
-        /// <summary>
-        /// This is a regression test for DotNetAnalyzers/StyleCopAnalyzers#403.
-        /// https://github.com/DotNetAnalyzers/StyleCopAnalyzers/issues/403
-        /// </summary>
-        /// <returns>A <see cref="Task"/> representing the asynchronous unit test.</returns>
         [Fact]
+        [WorkItem(403, "https://github.com/DotNetAnalyzers/StyleCopAnalyzers/issues/403")]
         public async Task TestSemicolonAtBeginningOfLineAsync()
         {
             string testCode = @"
@@ -412,6 +404,15 @@ int x
             await this.VerifyCSharpDiagnosticAsync(testCode, expected, CancellationToken.None).ConfigureAwait(false);
         }
 
+        [Fact]
+        [WorkItem(2699, "https://github.com/DotNetAnalyzers/StyleCopAnalyzers/issues/2699")]
+        public async Task TestSemiColonAtEndOfFileAsync()
+        {
+            string testCode = @"using System;";
+
+            await this.VerifyCSharpDiagnosticAsync(testCode, EmptyDiagnosticResults, CancellationToken.None).ConfigureAwait(false);
+        }
+
         protected override IEnumerable<DiagnosticAnalyzer> GetCSharpDiagnosticAnalyzers()
         {
             yield return new SA1002SemicolonsMustBeSpacedCorrectly();

StyleCop.Analyzers/StyleCop.Analyzers/SpacingRules/SA1002SemicolonsMustBeSpacedCorrectly.cs

@@ -107,6 +107,10 @@ private static void HandleSemicolonToken(SyntaxTreeAnalysisContext context, Synt
 
                     break;
 
+                case SyntaxKind.None:
+                    // The semi colon is the last character in the file.
+                    return;
+
                 default:
                     break;
                 }