Merge pull request #60 from autofac/feature/issue-48-delegate-mock

Support mocking delegate types (#48)

Author: tillig

src/Autofac.Extras.Moq/AutoMock.cs

@@ -202,8 +202,12 @@ public void Dispose()
     public Mock<T> Mock<T>(params Parameter[] parameters)
         where T : class
     {
-        var obj = (IMocked<T>)Create<T>(true, parameters);
-        return obj.Mock;
+        // Mock.Get retrieves the Mock<T> from Moq's internal registry rather
+        // than casting the resolved object to IMocked<T>. This is important for
+        // delegate mocks (issue #48): the Object of a delegate mock is the
+        // delegate itself and does not implement IMocked<T>, so a direct cast
+        // would throw an InvalidCastException.
+        return global::Moq.Mock.Get(Create<T>(true, parameters));
     }
 
     /// <summary>

test/Autofac.Extras.Moq.Test/AutoMockFixture.cs

@@ -345,6 +345,20 @@ public void CreateClassWithParameter()
         }
     }
 
+    [Fact]
+    public void MockOfDelegateTypeSucceeds()
+    {
+        // Issue #48 - mocking a delegate type used to throw InvalidCastException
+        // because the delegate Object does not implement IMocked<T>.
+        using (var mock = AutoMock.GetLoose())
+        {
+            var delegateMock = mock.Mock<TestDelegate>();
+            delegateMock.Setup(x => x(It.IsAny<int>())).Returns(20);
+
+            Assert.Equal(20, delegateMock.Object.Invoke(5));
+        }
+    }
+
     [Fact]
     public void MockedClassWithConstructorThrows()
     {
@@ -428,6 +442,8 @@ public ClassWithParameters(int param1, TimeSpan param2)
         }
     }
 
+    public delegate int TestDelegate(int x);
+
     internal class ConsumesDisposable
     {
         private readonly ITestDisposable _disposable;