mock.Protected() setup methods fail when argument is of type Expression

[tonyhallett]

var mock = new Mock<FooBase>();
var expression = Expression.Constant(1);
mock.Protected().SetupSet<Expression>("ExpressionProperty", expression);

Throws

System.ArgumentException : Expression of type 'System.Int32' cannot be used for parameter of type 'System.Linq.Expressions.Expression' of method 'Void set_ExpressionProperty(System.Linq.Expressions.Expression)'

ToExpressionArg should check the Type and uses Expression.Constant if the argument type is a LambdaExpression / Expression
https://github.com/moq/moq4/blob/9e148f6955a2aef09c6e4a5c117c7c7deef0756e/src/Moq/Protected/ProtectedMock.cs#L520

		private static Expression ToExpressionArg(Type type, object arg)
		{
			if (arg is LambdaExpression lambda)
			{
				return lambda.Body;
			}

			if (arg is Expression expression)
			{
				return expression;
			}

			return Expression.Constant(arg, type);
		}
``

[stakx]

@tonyhallett, could you please provide a minimal but complete repro (one including the declaration of FooBase.ExpressionProperty). Thanks!

[tonyhallett]

public class FooBase
{
	protected virtual ConstantExpression ExpressionProperty { get; set; }
}

public class Test {
      [Fact]
      public void This_Throws_But_Should_Not(){
         var mock = new Mock<FooBase>();
         var expression = Expression.Constant(1);
         mock.Protected().SetupSet<Expression>("ExpressionProperty", expression);
      }
}