私は自分のASP.NET MVCコントローラをモックすると、私のActionMethodはビューを返しません。どうして？

Question

私の単純なIndex() ActionMethodでは、User.Identityプロパティを参照しています。だから私はそれを嘲笑する必要があると思った。

私はHomeControllerモックをいくつか作成し、これを私の単体テストで使用します。これを行うと、ActionMethodはビューとしてnullを返します。モックされたコントローラを具体的なインスタンスに置き換えると（もちろん、User.Identityへの参照をコメントアウトすると）、正しいビューが返されます。

例えば、

// Arrange. 
var homeController = Mock<HomeController>(..); 
homeController.Setup(x => x.User).Returns(new GenericPrincipal(..)); 

// Act. 
var result = homeController.Index(); 

// Assert. 
Assert.IsNotNull(result); // This fails here. result is NULL. 

が、私はこれを行う（および任意のUser参照をコメントアウト）する場合、それが動作...

// Arrange. 
var homeController = new HomeController(..); 

// Act. 
var result = homeController.Index(); 

// Assert. 
Assert.IsNotNull(result); // Tick! 

これがなぜ任意のアイデア？

Answer 1

：

は、ここでは、ユニットテストに喜んでいるコントローラに嘲笑ユーザーを注入するための正しい方法です。 I provided one on another answer that you could use。 Steve Rowbothamによれば、テスト中のシステム（つまりコントローラの依存関係）の依存関係を嘲笑し、テスト対象のシステム自体を嘲笑しないでください。あなたは単にあなたのテストでは、次の

var controllerToTest = new HomeController(); 
var context = new MockHttpContextBase(controllerToTest); 

// do stuff that you want to test e.g. something goes into session 

Assert.IsTrue(context.HttpContext.Session.Count > 0); 

をするだろうあなたはさらに一歩行くことが、コントローラではないモックの本当のバージョンをテストするために:)

リンクからHttpContextBaseクラスを使用したいとセットアップとTearDownメソッドを作成して、コントローラを虚偽のコンテキストで設定します。

Answer 2

正直言って、これはテスト対象システム（SUT）を嘲笑している、つまりHomeControllerだからこそ、非常に奇妙なテストのようです。通常、SUTの依存関係を擬似的に模擬し、模擬に対する期待値を設定し、模擬テストをSUTに挿入して、それがその依存関係と適切に相互作用していることを確認する。

HomeControllerのモックを作成すると、MoqはHomeControllerから継承するクラスを作成し、仮想メソッドIndexを上書きします。したがって、Indexをモックで呼び出すと、HomeControllerクラスで定義したオーバーライドされた実装であるIndexの実装を呼び出さないことになります。あなたが明示的にSetupのメソッドを使っていないので、この場合はnullというデフォルト値が返されます。

HomeControllerクラスの実際のインスタンスを構築しているため、2番目のテストではIndexという実際の実装を呼び出しています。モックオブジェクトのインスタンス上でGetType()を呼び出すと、HomeControllerから派生したプロキシのインスタンスであることがわかります。これは、基本クラスの公開されたオーバーライド可能なメソッドへの呼び出しを傍受します（目的の大部分です）。モックオブジェクトの）。

Answer 3

ユニットテストには奇妙なことがいくつかあります。あなたはコントローラをユニットテストしていますが、テスト中のオブジェクトの作成を嘲笑しています：var homeController = Mock<HomeController>(..);これは間違っています。私はあなたがで動作するように、コントローラ用のHttpContextをからかっされるべきだと思う

// arrange 
var sut = new HomeController(); 
var user = new GenericPrincipal(new GenericIdentity("foo"), null); 
var httpContext = new Mock<HttpContextBase>(); 
httpContext.Setup(x => x.User).Returns(user); 
var context = new ControllerContext(new RequestContext(httpContext.Object, new RouteData()), sut); 
sut.ControllerContext = context; 

// act 
var actual = sut.Index(); 

// assert 
... 

Answer 4

あなたのIndexメソッドはおそらく仮想であり、Moqはその機能を模擬関数で置き換えると思います。これを防ぐには、モックにCallBaseプロパティを設定する必要があります。

しかし、あなたのコントローラを嘲笑してはならないという他の回答に同意しますが、あなたはあなたの依存関係を模倣すべきです。

（より簡単な方法は、のHttpContextからプリンシパルを取得することができ、特殊なモデルバインダーを作成することで、その後、あなたは自分のメソッドへの入力パラメータとして校長を渡すことができます）