web-dev-qa-db-ja.com

Moqを使用してユニットテストの非同期メソッドをモックする

Web API呼び出しを行うサービスのメソッドをテストしています。 Webサービス(ソリューション内の別のプロジェクトにある)をローカルで実行している場合、単体テストでは通常のHttpClientを使用すると正常に機能します。

ただし、変更をチェックインすると、ビルドサーバーはWebサービスにアクセスできないため、テストは失敗します。

IHttpClientインターフェイスを作成し、アプリケーションで使用するバージョンを実装することにより、ユニットテスト用にこれを回避する方法を考案しました。単体テストでは、モックされた非同期のpostメソッドでモックされたバージョンを完成させます。ここで問題に遭遇しました。この特定のテストでOK HttpStatusResultを返したいです。別の同様のテストでは、悪い結果を返します。

テストは実行されますが、完了しません。待ちに待っています。私は非同期プログラミング、デリゲート、Moq自体は初めてであり、しばらくの間SOとgoogleを検索して新しいことを学びましたが、まだこの問題を乗り越えることができないようです。

私がテストしようとしている方法は次のとおりです。

public async Task<bool> QueueNotificationAsync(IHttpClient client, Email email)
{
    // do stuff
    try
    {
        // The test hangs here, never returning
        HttpResponseMessage response = await client.PostAsync(uri, content);

        // more logic here
    }
    // more stuff
}

私の単体テスト方法は次のとおりです。

[TestMethod]
public async Task QueueNotificationAsync_Completes_With_ValidEmail()
{
    Email email = new Email()
    {
        FromAddress = "[email protected]",
        ToAddress = "[email protected]",
        CCAddress = "[email protected]",
        BCCAddress = "[email protected]",
        Subject = "Hello",
        Body = "Hello World."
    };
    var mockClient = new Mock<IHttpClient>();
    mockClient.Setup(c => c.PostAsync(
        It.IsAny<Uri>(),
        It.IsAny<HttpContent>()
        )).Returns(() => new Task<HttpResponseMessage>(() => new HttpResponseMessage(System.Net.HttpStatusCode.OK)));

    bool result = await _notificationRequestService.QueueNotificationAsync(mockClient.Object, email);

    Assert.IsTrue(result, "Queue failed.");
}

何が間違っていますか?

ご協力ありがとうございました。

152
mvanella

あなたはタスクを作成しているが、決して開始しないので、決して完了しない。ただし、単にタスクを開始するのではなく、代わりに Task.FromResult<TResult> を使用するように変更して、既に完了したタスクを提供します。

...
.Returns(Task.FromResult(new HttpResponseMessage(System.Net.HttpStatusCode.OK)));

この方法で実際の非同期性をテストするわけではないことに注意してください-それを行うには、より細かな方法で制御できるTask<T>を作成するためにもう少し作業を行う必要がありますが、それは何かです別の日のために。

また、すべてをモックするのではなく、IHttpClientに偽物を使用することを検討することもできます。これは、実際に必要な頻度によって異なります。

306
Jon Skeet

上記の@Stuart Grassieの回答をお勧めします。

var moqCredentialMananger = new Mock<ICredentialManager>();
moqCredentialMananger
                    .Setup(x => x.GetCredentialsAsync(It.IsAny<string>()))
                    .ReturnsAsync(new Credentials() { .. .. .. });
1
DineshNS