Fixed #26

Fixed: unexpected interceptor is chosen when attribute is applied to any
of the derived classes.

CHANGELOG.md

@@ -4,6 +4,11 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
 and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
 
+## [Unreleased]
+
+### Fixed
+- Unexpected interceptor is chosen when attribute is applied to any of the derived classes. [#26](https://github.com/ninject/Ninject.Extensions.Interception/issues/26)
+
 ## [3.3.2] - 2017-10-22
 
 ### Added

src/Ninject.Extensions.Interception.Test/Fakes/Derived.cs

@@ -9,4 +9,10 @@ public void DoDerived()
         {
         }
     }
+
+    public class Derived2 : Base, IDerived
+    {
+        public void DoDerived()
+        { }
+    }
 }

src/Ninject.Extensions.Interception.Test/MethodInterceptionContext.cs

@@ -1,5 +1,6 @@
 namespace Ninject.Extensions.Interception
 {
+    using System.Linq;
     using FluentAssertions;
 
     using Ninject.Extensions.Interception.Fakes;
@@ -177,6 +178,26 @@ public void MethodsFromDerivedClassesCanBeIntercepted()
             }
         }
 
+        [Fact]
+        public void MethodsFromDerivedClassesWithAttributeCanBeIntercepted()
+        {
+            using (var kernel = CreateDefaultInterceptionKernel())
+            {
+                CountInterceptor.Reset();
+
+                kernel.Bind<IDerived>().To<Derived>();
+                kernel.Bind<IDerived>().To<Derived2>();
+
+                var objs = kernel.GetAll<IDerived>().ToList();
+
+                objs[0].DoDerived();
+                CountInterceptor.Count.Should().Be(1);
+
+                objs[1].DoDerived();
+                CountInterceptor.Count.Should().Be(1);
+            }
+        }
+
         [Fact]
         public void ClassesWithNoDefaultConstructorCanBeIntercepted()
         {

src/Ninject.Extensions.Interception/Advice/Advice.cs

@@ -164,6 +164,11 @@ private bool MatchesMethodPredicate(IProxyRequest request)
         /// <returns><c>true</c> if XXXX, <c>false</c> otherwise.</returns>
         private bool MatchesMethod(IProxyRequest request)
         {
+            if (!this.method.DeclaringType.IsAssignableFrom(request.Target.GetType()))
+            {
+                return false;
+            }
+
             if (request.Method.GetMethodHandle().Equals(this.MethodHandle))
             {
                 return true;
@@ -186,6 +191,6 @@ private bool MatchesMethod(IProxyRequest request)
             }
 
             return map.TargetMethods[index].GetMethodHandle() == this.method.GetMethodHandle();
-         }
+        }
     }
 }