5

MVC4プロジェクトのヘルパークラスの拡張メソッドがあります。

public static class ExtensionMethods
{

    public static object Value<TModel, TProperty>(this Expression<Func<TModel, TProperty>> expression, ViewDataDictionary<TModel> viewData)
    {
        return ModelMetadata.FromLambdaExpression(expression, viewData).Model;
    }

}

これは、単純ではない方法で非常に単純なことを行います。

では、これをユニットテストするにはどうすればよいでしょうか。静的メソッドをモックしたり、依存性注入を使用したりすることは避けたほうがよいでしょうが、この場合、これらが本当に唯一の実行可能なアプローチであるかどうか、私は心を開いています。

これは、単体テストをより受け入れやすくするために改善できる欠陥のある設計ですか?

4

1 に答える 1

3

これは確かに欠陥のある設計であり、おそらく一般的です。正しいメソッドシグネチャでは、パラメータの順序を切り替える必要があります。

public static object Value<TModel, TProperty>(this ViewDataDictionary<TModel> viewData, Expression<Func<TModel, TProperty>> expression)
{
    return ModelMetadata.FromLambdaExpression(expression, viewData).Model;
}

これは、単体テストから呼び出すことができます。

Expression<Func<Model, string>> expression = (t => t.PropertyName);
ExtensionMethods.Value<Model, string>(viewData, expression));
于 2012-12-31T09:11:29.897 に答える