Fix behavior and test for properties

Author: sharwell

AsyncUsageAnalyzers/AsyncUsageAnalyzers.Test/Usage/IncludeCancellationParameterUnitTests.cs

@@ -39,7 +39,7 @@ public async Task TestPropertyAsync()
 using System.Threading.Tasks;
 class ClassName
 {
-    Task PropertyName { get; set; }
+    public Task PropertyName { get; set; }
 }
 ";
 

AsyncUsageAnalyzers/AsyncUsageAnalyzers/Usage/IncludeCancellationParameterAnalyzer.cs

@@ -78,6 +78,38 @@ public void HandleMethodDeclaration(SymbolAnalysisContext context)
                     return;
                 }
 
+                switch (symbol.MethodKind)
+                {
+                case MethodKind.LambdaMethod:
+                case MethodKind.Constructor:
+                case MethodKind.Conversion:
+                case MethodKind.UserDefinedOperator:
+                case MethodKind.PropertyGet:
+                case MethodKind.PropertySet:
+                case MethodKind.StaticConstructor:
+                case MethodKind.BuiltinOperator:
+                case MethodKind.Destructor:
+                case MethodKind.EventAdd:
+                case MethodKind.EventRaise:
+                case MethodKind.EventRemove:
+                    // These can't be async
+                    return;
+
+                case MethodKind.DelegateInvoke:
+                    // Not sure if this is reachable
+                    return;
+
+                case MethodKind.ExplicitInterfaceImplementation:
+                    // These are ignored
+                    return;
+
+                case MethodKind.ReducedExtension:
+                case MethodKind.Ordinary:
+                case MethodKind.DeclareMethod:
+                default:
+                    break;
+                }
+
                 if (!symbol.IsInAnalyzedSource(this.generatedHeaderCache, context.CancellationToken))
                 {
                     return;