Fix analyzer RCS0005 (#1533)

dotnet · Sep 21, 2024 · 48d201b · 48d201b
1 parent e07914c
commit 48d201b
Show file tree

Hide file tree

Showing 3 changed files with 27 additions and 1 deletion.
diff --git a/ChangeLog.md b/ChangeLog.md
@@ -16,6 +16,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Fix analyzer [RCS0053](https://josefpihrt.github.io/docs/roslynator/analyzers/RCS0053) ([PR](https://github.com/dotnet/roslynator/pull/1518))
 - Fix analyzer [RCS0056](https://josefpihrt.github.io/docs/roslynator/analyzers/RCS0056) ([PR](https://github.com/dotnet/roslynator/pull/1521))
 - Fix analyzer [RCS1181](https://josefpihrt.github.io/docs/roslynator/analyzers/RCS1181) ([PR](https://github.com/dotnet/roslynator/pull/1526))
+- Fix analyzer [RCS0005](https://josefpihrt.github.io/docs/roslynator/analyzers/RCS0005) ([PR](https://github.com/dotnet/roslynator/pull/1533))
 
 ## [4.12.5] - 2024-09-13
 

diff --git a/src/Formatting.Analyzers/CSharp/AddBlankLineBeforeEndRegionDirectiveAnalyzer.cs b/src/Formatting.Analyzers/CSharp/AddBlankLineBeforeEndRegionDirectiveAnalyzer.cs
@@ -81,7 +81,8 @@ static bool IsPrecededWithEmptyLineOrRegionDirective(EndRegionDirectiveTriviaSyn
                     return en.Current.IsKind(
                         SyntaxKind.EndOfLineTrivia,
                         SyntaxKind.RegionDirectiveTrivia,
-                        SyntaxKind.EndRegionDirectiveTrivia);
+                        SyntaxKind.EndRegionDirectiveTrivia,
+                        SyntaxKind.PragmaWarningDirectiveTrivia);
                 }
             }
 

diff --git a/src/Tests/Formatting.Analyzers.Tests/RCS0005AddBlankLineBeforeEndRegionDirectiveTests.cs b/src/Tests/Formatting.Analyzers.Tests/RCS0005AddBlankLineBeforeEndRegionDirectiveTests.cs
@@ -227,4 +227,28 @@ await VerifyNoDiagnosticAsync(@"
     #endregion
 ");
     }
+
+    [Fact, Trait(Traits.Analyzer, DiagnosticIdentifiers.AddBlankLineBeforeEndRegionDirective)]
+    public async Task TestNoDiagnostic_PragmaWarningDirective()
+    {
+        await VerifyNoDiagnosticAsync(@"
+namespace N
+{
+    /// <summary>
+    /// x
+    /// </summary>
+    class C
+    {
+        #region R
+
+#pragma warning disable 1591
+
+        public int P { get; set; }
+
+#pragma warning restore 1591
+
+        #endregion
+    }
+}");
+    }
 }