BUGFIX: REFL016 suggests using nameof(Foo.get_Bar). Fix #94.

Author: JohanLarsson

ReflectionAnalyzers.Tests/REFL016UseNameofTests/ValidCode.cs

@@ -126,6 +126,25 @@ public class Generic<T>
             AnalyzerAssert.Valid(Analyzer, ExpectedDiagnostic, code);
         }
 
+        [Test]
+        public void IEnumeratorGetCurrent()
+        {
+            var testCode = @"
+namespace RoslynSandbox
+{
+    using System.Collections;
+
+    public class Foo
+    {
+        public void Meh(object value)
+        {
+            _ = typeof(IEnumerator).GetMethod(""get_Current"");
+        }
+    }
+}";
+            AnalyzerAssert.Valid(Analyzer, testCode);
+        }
+
         [Test]
         public void WhenThrowingArgumentException()
         {

ReflectionAnalyzers/NodeAnalzers/NameofAnalyzer.cs

@@ -107,12 +107,12 @@ private static bool TryGetX(InvocationExpressionSyntax invocation, SyntaxNodeAna
             if (invocation.Expression is MemberAccessExpressionSyntax memberAccess &&
                 !(memberAccess.Expression is InstanceExpressionSyntax))
             {
-                return invocation.TryGetTarget(KnownSymbol.Type.GetEvent,      context.SemanticModel, context.CancellationToken, out getX) ||
-                       invocation.TryGetTarget(KnownSymbol.Type.GetField,      context.SemanticModel, context.CancellationToken, out getX) ||
-                       invocation.TryGetTarget(KnownSymbol.Type.GetMember,     context.SemanticModel, context.CancellationToken, out getX) ||
-                       invocation.TryGetTarget(KnownSymbol.Type.GetMethod,     context.SemanticModel, context.CancellationToken, out getX) ||
+                return invocation.TryGetTarget(KnownSymbol.Type.GetEvent, context.SemanticModel, context.CancellationToken, out getX) ||
+                       invocation.TryGetTarget(KnownSymbol.Type.GetField, context.SemanticModel, context.CancellationToken, out getX) ||
+                       invocation.TryGetTarget(KnownSymbol.Type.GetMember, context.SemanticModel, context.CancellationToken, out getX) ||
+                       invocation.TryGetTarget(KnownSymbol.Type.GetMethod, context.SemanticModel, context.CancellationToken, out getX) ||
                        invocation.TryGetTarget(KnownSymbol.Type.GetNestedType, context.SemanticModel, context.CancellationToken, out getX) ||
-                       invocation.TryGetTarget(KnownSymbol.Type.GetProperty,   context.SemanticModel, context.CancellationToken, out getX);
+                       invocation.TryGetTarget(KnownSymbol.Type.GetProperty, context.SemanticModel, context.CancellationToken, out getX);
             }
 
             getX = null;
@@ -126,9 +126,20 @@ private static bool TryGetTarget(InvocationExpressionSyntax invocation, IMethodS
             if (GetX.TryGetTargetType(invocation, context, out var targetType, out instance))
             {
                 _ = GetX.TryGetMember(getX, targetType, name, BindingFlags.Public | BindingFlags.NonPublic | BindingFlags.Static | BindingFlags.Instance | BindingFlags.FlattenHierarchy, GetX.AnyTypes, context, out var targetSymbol);
-                target = targetSymbol != null
-                    ? new Optional<ISymbol>(targetSymbol)
-                    : default(Optional<ISymbol>);
+
+                if (targetSymbol is IMethodSymbol method)
+                {
+                    target = method.AssociatedSymbol == null
+                        ? new Optional<ISymbol>(targetSymbol)
+                        : default(Optional<ISymbol>);
+                }
+                else
+                {
+                    target = targetSymbol != null
+                        ? new Optional<ISymbol>(targetSymbol)
+                        : default(Optional<ISymbol>);
+                }
+
                 return true;
             }