Skip to content

Commit

Permalink
Fix erroneous parameter count mismatch by using HasCompatibleParamete…
Browse files Browse the repository at this point in the history
…rList method
  • Loading branch information
idigra committed Aug 8, 2018
1 parent 70be99c commit dac25d6
Showing 1 changed file with 9 additions and 5 deletions.
14 changes: 9 additions & 5 deletions src/Moq/MethodCallReturn.cs
Original file line number Diff line number Diff line change
Expand Up @@ -171,15 +171,19 @@ private void ValidateReturnDelegate(Delegate callback)
{
var callbackMethod = callback.GetMethodInfo();

var expectedParams = this.Method.GetParameters();
var actualParams = callback.GetMethodInfo().GetParameters();
if (callback.HasCompatibleParameterList(expectedParams))
{
return;
}

// validate number of parameters:

var numberOfActualParameters = callbackMethod.GetParameters().Length;
if (callbackMethod.IsStatic)
if (callbackMethod.IsExtensionMethod())
{
if (callbackMethod.IsExtensionMethod() || callback.Target != null)
{
numberOfActualParameters--;
}
numberOfActualParameters--;
}

if (numberOfActualParameters > 0)
Expand Down

0 comments on commit dac25d6

Please sign in to comment.