add tests for white space only files not beeing classified as empty

Author: DWIAltonaAnalytics

StyleCop.Analyzers/StyleCop.Analyzers.Test/LayoutRules/SA1518UnitTests.cs

@@ -29,6 +29,8 @@ public void Bar(int i)
     }
 }";
 
+        private const string WhiteSpace = "\t  ";
+
         /// <summary>
         /// Verifies that blank lines at the end of the file will produce a warning.
         /// </summary>
@@ -49,6 +51,60 @@ internal async Task TestWithBlankLinesAtEndOfFileAsync(OptionSetting? newlineAtE
             await VerifyCSharpFixAsync(newlineAtEndOfFile, testCode, expected, fixedCode, CancellationToken.None).ConfigureAwait(false);
         }
 
+        /// <summary>
+        /// Verifies file with white space only and no cr/lf at end of file will produce a warning when setting requires.
+        /// </summary>
+        /// <param name="newlineAtEndOfFile">The effective <see cref="OptionSetting"/> setting.</param>
+        /// <param name="expectedText">The expected text to appear at the end of the file.</param>
+        /// <returns>A <see cref="Task"/> representing the asynchronous unit test.</returns>
+        [Theory]
+        [InlineData(null, null)]
+        [InlineData(OptionSetting.Allow, null)]
+        [InlineData(OptionSetting.Require, "\r\n")]
+        [InlineData(OptionSetting.Omit, null)]
+        internal async Task TestWithWhiteSpaceOnlyAsync(OptionSetting? newlineAtEndOfFile, string expectedText)
+        {
+            var testCode = WhiteSpace;
+            var fixedCode = expectedText;
+
+            if (expectedText == null)
+            {
+                await VerifyCSharpDiagnosticAsync(newlineAtEndOfFile, testCode, DiagnosticResult.EmptyDiagnosticResults, CancellationToken.None).ConfigureAwait(false);
+            }
+            else
+            {
+                var expected = Diagnostic(this.GetDescriptor(newlineAtEndOfFile)).WithLocation(1, 1);
+                await VerifyCSharpFixAsync(newlineAtEndOfFile, testCode, expected, fixedCode, CancellationToken.None).ConfigureAwait(false);
+            }
+        }
+
+        /// <summary>
+        /// Verifies file with white space only and cr/lf at end of file will produce a warning when setting requires.
+        /// </summary>
+        /// <param name="newlineAtEndOfFile">The effective <see cref="OptionSetting"/> setting.</param>
+        /// <param name="expectedText">The expected text to appear at the end of the file.</param>
+        /// <returns>A <see cref="Task"/> representing the asynchronous unit test.</returns>
+        [Theory]
+        [InlineData(null, null)]
+        [InlineData(OptionSetting.Allow, null)]
+        [InlineData(OptionSetting.Require, null)]
+        [InlineData(OptionSetting.Omit, "")]
+        internal async Task TestWithWhiteSpaceAndNewlineOnlyAsync(OptionSetting? newlineAtEndOfFile, string expectedText)
+        {
+            var testCode = WhiteSpace + "\r\n";
+            var fixedCode = expectedText;
+
+            if (expectedText == null)
+            {
+                await VerifyCSharpDiagnosticAsync(newlineAtEndOfFile, testCode, DiagnosticResult.EmptyDiagnosticResults, CancellationToken.None).ConfigureAwait(false);
+            }
+            else
+            {
+                var expected = Diagnostic(this.GetDescriptor(newlineAtEndOfFile)).WithLocation(1, 1);
+                await VerifyCSharpFixAsync(newlineAtEndOfFile, testCode, expected, fixedCode, CancellationToken.None).ConfigureAwait(false);
+            }
+        }
+
         /// <summary>
         /// Verifies that empty files will not produce a warning.
         /// </summary>

StyleCop.Analyzers/StyleCop.Analyzers/Helpers/SyntaxTreeHelpers.cs

@@ -77,7 +77,7 @@ public static bool IsEmpty(this SyntaxTree tree, CancellationToken cancellationT
             var root = tree.GetRoot(cancellationToken);
             var firstToken = root.GetFirstToken(includeZeroWidth: true);
 
-            return firstToken.IsKind(SyntaxKind.EndOfFileToken) && firstToken.FullSpan.Length == 0;
+            return firstToken.IsKind(SyntaxKind.EndOfFileToken) && firstToken.FullSpan.IsEmpty;
         }
 
         internal static bool ContainsUsingAlias(this SyntaxTree tree, ConcurrentDictionary<SyntaxTree, bool> cache)