web-dev-qa-db-ja.com

タスク待ちのMoq

WCFメソッドをAsyncに変換したため、ユニットテストが失敗し、それらを機能させるための正しい構文がわかりません。

クライアントプロキシクラス

_ public interface IClientProxy
{
     Task DoSomething(CredentialDataList credentialData, string store);
}
_

サービスクラス

_  public class CredentialSync : ICredentialSync
{
    private ICredentialRepository _repository;

    private IClientProxy _client;

    public CredentialSync()
    {
        this._repository = new CredentialRepository();
        this._client = new ClientProxy();
    }

    public CredentialSync(ICredentialRepository repository, IClientProxy client)
    {
        this._repository = repository;
        this._client = client;
    }

   public async Task Synchronise(string payrollNumber)
    {
        try
        {
            if (string.IsNullOrEmpty(payrollNumber))
            {
                .... some code
              }
            else
            {
                CredentialDataList credentialData = new CredentialDataList();
                List<CredentialData> credentialList = new List<CredentialData>();

                // fetch the record from the database
                List<GetCredentialData_Result> data = this._repository.GetCredentialData(payrollNumber);
                var pinData = this._repository.GetCredentialPinData(payrollNumber);

                // get the stores for this employee
                var storeList = data.Where(a => a.StoreNumber != null)
                    .GroupBy(a => a.StoreNumber)
                    .Select(x => new Store { StoreNumber = x.Key.ToString() }).ToArray();

                var credential = this.ExtractCredentialData(data, pinData, payrollNumber);

                credentialList.Add(credential);
                credentialData.CredentialList = credentialList;

                foreach (var store in storeList)
                {       
                  //this line causes an Object reference not set to an instance of an object error
                   await  _client.DoSomething(credentialData, store.StoreNumber);

                }
            }
        }
        catch (Exception ex)
        {
            throw new FaultException<Exception>(ex);
        }
    }
_

テストクラス

_ /// </summary>
[TestClass]
public class SynchTest
{

    private Mock<ICredentialRepository> _mockRepository;
    private Mock<IClientProxy> _mockService;

    [TestInitialize]
    public void Setup()
    {
       ... some setups for repository which work fine
    }

[TestMethod]      
    public async Task SynchroniseData_WithOneEmployee_CallsReplicateService()
    {
        this._mockService = new Mock<IClientProxy>();
        this._mockService.Setup(x=>x.DoSomething(It.IsAny<CredentialDataList>(), It.IsAny<string>()));
        // arrange
        string payrollNumber = "1";
        CredentialSync service = new CredentialSync(this._mockRepository.Object, this._mockService.Object);

        // act
        await service.Synchronise(payrollNumber);

        // assert                 
        this._mockService.VerifyAll();
    }
_

エラーは_ClientProxy.DoSomething_が呼び出されたときです:

オブジェクト参照がオブジェクトインスタンスに設定されていません

パラメータは両方とも問題ありません。

_ClientProxy.DoSomething_メソッドを同期メソッド(public void DoSomething(...))に変換すると、コードは正常に機能しますが、非同期で呼び出す必要があります

41
jazza1000

DoSomethingnullを返す代わりにTaskを返すため、待機中に例外が発生します。モックを作成するときに、Taskを返すように指定する必要があります。

この場合、Task.FromResultを使用して既に完了したタスクを単純に返すことができるように見えるので、模擬セットアップは次のようになります。

this._mockService.Setup(...).Returns(Task.FromResult(false));

.Netの次のバージョン(4.6)から、次のようにTask.CompletedTaskを使用できます。

this._mockService.Setup(...).Returns(Task.CompletedTask);
84
i3arnon

ReturnsAsyncを使用して、コード内の混乱を減らすことができます。

this._mockService.Setup(...).ReturnsAsync(false);

この方法で、コードのTask.FromResult部分を削除できます

23
knorman

TaskモックからDoSomethingを返す必要があると思います

this._mockService.Setup(x => x.DoSomething(It.IsAny<CredentialDataList>(), It.IsAny<string>()))
    .Returns(Task.FromResult<int>(0));
4
Ned Stoyanov