- add more cases

- apply WTG1001 fix that removes 'private' keyword if the method is 'partial' and 'void'
- keep 'private' keyword if a partial method is non-void or having 'out' keyword

Author: mj-wtg

src/WTG.Analyzers.Test/TestData/VisibilityAnalyzer/Partial/Diagnostics.xml

@@ -1,10 +1,17 @@
 <?xml version="1.0" encoding="utf-8" ?>
 <diagnostics severity="Hidden">
 	<languageVersion>9.0</languageVersion>
+	<suppressId justtification="To ignore when only one method in one class have the code fix removing 'private' applied.">CS8799</suppressId>
 	<diagnostic id="WTG1001" message="Our convention is to omit the 'private' modifier where it is already the default.">
-		<location>Test0.cs: (15,3-10)</location>
+		<location>Test0.cs: (9,3-10)</location>
+	</diagnostic>
+	<diagnostic id="WTG1001" message="Our convention is to omit the 'private' modifier where it is already the default.">
+		<location>Test0.cs: (19,3-10)</location>
+	</diagnostic>
+	<diagnostic id="WTG1001" message="Our convention is to omit the 'private' modifier where it is already the default.">
+		<location>Test0.cs: (25,3-10)</location>
 	</diagnostic>
 	<diagnostic id="WTG1006" message="Our convention is to omit the 'internal' modifier on types where it is already the default." >
-		<location>Test0.cs: (20,2-10)</location>
+		<location>Test0.cs: (30,2-10)</location>
 	</diagnostic>
 </diagnostics>

src/WTG.Analyzers.Test/TestData/VisibilityAnalyzer/Partial/Result.cs

@@ -4,6 +4,7 @@ partial class Foo
 	{
 		private partial int Bar();
 		public partial void FooBar();
+		private partial int FooBarBaz(out int value);
 
 		partial void Qux();
 		private partial void Quux(out int value);
@@ -13,6 +14,7 @@ partial class Foo
 	{
 		private partial int Bar() { return default; }
 		public partial void FooBar() { }
+		private partial int FooBarBaz(out int value) { value = default; return default; }
 
 		partial void Qux() { }
 		private partial void Quux(out int value) { value = default; }

src/WTG.Analyzers.Test/TestData/VisibilityAnalyzer/Partial/Source.cs

@@ -4,6 +4,7 @@ partial class Foo
 	{
 		private partial int Bar();
 		public partial void FooBar();
+		private partial int FooBarBaz(out int value);
 
 		private partial void Qux();
 		private partial void Quux(out int value);
@@ -13,6 +14,7 @@ partial class Foo
 	{
 		private partial int Bar() { return default; }
 		public partial void FooBar() { }
+		private partial int FooBarBaz(out int value) { value = default; return default; }
 
 		private partial void Qux() { }
 		private partial void Quux(out int value) { value = default; }

src/WTG.Analyzers/Analyzers/Visibility/VisibilityAnalyzer.cs

@@ -1,6 +1,8 @@
 using System.Collections.Immutable;
+using System.Linq;
 using Microsoft.CodeAnalysis;
 using Microsoft.CodeAnalysis.CSharp;
+using Microsoft.CodeAnalysis.CSharp.Syntax;
 using Microsoft.CodeAnalysis.Diagnostics;
 using WTG.Analyzers.Utils;
 
@@ -32,7 +34,8 @@ static void Analyze(SyntaxNodeAnalysisContext context, FileDetailCache cache)
 				return;
 			}
 
-			var list = ModifierExtractionVisitor.Instance.Visit(context.Node);
+			var currentNode = context.Node;
+			var list = ModifierExtractionVisitor.Instance.Visit(currentNode);
 			var privateToken = default(SyntaxToken);
 
 			foreach (var modifier in list)
@@ -47,11 +50,10 @@ static void Analyze(SyntaxNodeAnalysisContext context, FileDetailCache cache)
 
 					case SyntaxKind.ProtectedKeyword:
 					case SyntaxKind.PublicKeyword:
-					case SyntaxKind.PartialKeyword when (context.Node.IsKind(SyntaxKind.MethodDeclaration)):
+					case SyntaxKind.PartialKeyword when (DoesMethodHaveReturnValueOrOutKeyword(currentNode)):
 						return;
-
 					case SyntaxKind.InternalKeyword:
-						if (IsTopLevel(context.Node))
+						if (IsTopLevel(currentNode))
 						{
 							context.ReportDiagnostic(Rules.CreateDoNotUseTheInternalKeywordForTopLevelTypesDiagnostic(modifier.GetLocation()));
 						}
@@ -65,6 +67,28 @@ static void Analyze(SyntaxNodeAnalysisContext context, FileDetailCache cache)
 			}
 		}
 
+		static bool DoesMethodHaveReturnValueOrOutKeyword(SyntaxNode node)
+		{
+			if (!node.IsKind(SyntaxKind.MethodDeclaration))
+			{
+				return false;
+			}
+
+			var methodNode = (MethodDeclarationSyntax)node;
+			if (methodNode.ReturnType is PredefinedTypeSyntax predefinedTypeSyntax
+				&& !predefinedTypeSyntax.Keyword.IsKind(SyntaxKind.VoidKeyword))
+			{
+				return true;
+			}
+
+			if (methodNode.ParameterList?.Parameters.Any(c => c.Modifiers.Any(SyntaxKind.OutKeyword)) == true)
+			{
+				return true;
+			}
+
+			return false;
+		}
+
 		static bool IsTopLevel(SyntaxNode node)
 		{
 			var parentKind = node.Parent?.Kind() ?? SyntaxKind.None;