Add ServiceMethodPrepareAsyncAnalyzer

Fixes #15

Author: sharwell

OpenStackNetAnalyzers/OpenStackNetAnalyzers/OpenStackNetAnalyzers.csproj

@@ -49,6 +49,7 @@
     <Compile Include="PlaceholderDocumentationCodeFix.cs" />
     <Compile Include="Properties\AssemblyInfo.cs" />
     <Compile Include="SdkTypeSymbolExtensions.cs" />
+    <Compile Include="ServiceMethodPrepareAsyncAnalyzer.cs" />
     <Compile Include="SpacingExtensions.cs" />
     <Compile Include="TypeSymbolExtensions.cs" />
     <Compile Include="XmlSyntaxFactory.cs" />

OpenStackNetAnalyzers/OpenStackNetAnalyzers/SdkTypeSymbolExtensions.cs

@@ -9,6 +9,8 @@ internal static class SdkTypeSymbolExtensions
 
         private const string FullyQualifiedDelegatingHttpApiCallT = "global::OpenStack.Net.DelegatingHttpApiCall<T>";
 
+        private const string FullyQualifiedIHttpService = "global::OpenStack.Services.IHttpService";
+
         public static bool IsExtensibleJsonObject(this INamedTypeSymbol symbol)
         {
             while (symbol != null && symbol.SpecialType != SpecialType.System_Object)
@@ -39,5 +41,19 @@ public static bool IsDelegatingHttpApiCall(this INamedTypeSymbol symbol)
 
             return false;
         }
+
+        public static bool IsHttpServiceInterface(this INamedTypeSymbol symbol)
+        {
+            if (symbol == null || symbol.TypeKind != TypeKind.Interface)
+                return false;
+
+            foreach (INamedTypeSymbol interfaceType in symbol.AllInterfaces)
+            {
+                if (string.Equals(FullyQualifiedIHttpService, interfaceType.ToDisplayString(SymbolDisplayFormat.FullyQualifiedFormat)))
+                    return true;
+            }
+
+            return false;
+        }
     }
 }

OpenStackNetAnalyzers/OpenStackNetAnalyzers/ServiceMethodPrepareAsyncAnalyzer.cs

@@ -0,0 +1,59 @@
+namespace OpenStackNetAnalyzers
+{
+    using System;
+    using System.Collections.Immutable;
+    using System.Linq;
+    using Microsoft.CodeAnalysis;
+    using Microsoft.CodeAnalysis.Diagnostics;
+
+    [DiagnosticAnalyzer(LanguageNames.CSharp)]
+    public class ServiceMethodPrepareAsyncAnalyzer : DiagnosticAnalyzer
+    {
+        public const string DiagnosticId = "ServiceMethodPrepareAsync";
+        internal const string Title = "Service methods should be named Prepare{Name}Async";
+        internal const string MessageFormat = "Service methods should be named Prepare{Name}Async";
+        internal const string Category = "OpenStack.Maintainability";
+        internal const string Description = "Service methods should be named Prepare{Name}Async";
+
+        private static DiagnosticDescriptor Descriptor =
+            new DiagnosticDescriptor(DiagnosticId, Title, MessageFormat, Category, DiagnosticSeverity.Warning, isEnabledByDefault: true, description: Description);
+
+        private static readonly ImmutableArray<DiagnosticDescriptor> _supportedDiagnostics =
+            ImmutableArray.Create(Descriptor);
+
+        public override ImmutableArray<DiagnosticDescriptor> SupportedDiagnostics
+        {
+            get
+            {
+                return _supportedDiagnostics;
+            }
+        }
+
+        public override void Initialize(AnalysisContext context)
+        {
+            context.RegisterSymbolAction(HandleNamedType, SymbolKind.NamedType);
+        }
+
+        private void HandleNamedType(SymbolAnalysisContext context)
+        {
+            INamedTypeSymbol symbol = (INamedTypeSymbol)context.Symbol;
+            if (!symbol.IsHttpServiceInterface())
+                return;
+
+            foreach (IMethodSymbol method in symbol.GetMembers().OfType<IMethodSymbol>())
+            {
+                if (string.IsNullOrEmpty(method.Name))
+                    continue;
+
+                if (method.Name.StartsWith("Prepare", StringComparison.Ordinal) && method.Name.EndsWith("Async", StringComparison.Ordinal))
+                {
+                    // TODO check letter following 'Prepare'
+                    continue;
+                }
+
+                ImmutableArray<Location> locations = method.Locations;
+                context.ReportDiagnostic(Diagnostic.Create(Descriptor, locations.FirstOrDefault(), locations.Skip(1)));
+            }
+        }
+    }
+}