MoqのSetupで引数を2つ指定すると、正しくSetupをしてくれない場合がある

Question

C#、ASP.NET Core MVCでユニットテストを作成しています。MoqのSetupで引数を2つ指定すると、正しくSetupをしてくれない場合があり、困っています。
どなたかご教授お願いします。

わかっていることとしては、

①Contollerの引数をListで渡す時
this._userCheckService.IsCheckedAsyncの引数が
1つの時はtrueを返していた。2つの時にfalseになっていました。

②Contollerの引数を単一データで渡す時
this._userCheckService.IsCheckedAsyncの引数が2つでも、問題なくtrueになりました。

①の作り方をした時のみ、現象が発生しています。

(追記)
開発環境: Windows10 Pro, Visual Studio Professional 2019, .NET 5

using ApplicationService;
using DomainModel;
using DomainService;
using Microsoft.AspNetCore.Http;
using Microsoft.AspNetCore.Mvc;
using Microsoft.Extensions.Logging;
using Moq;
using System;
using System.Collections.Generic;
using System.Linq;
using System.Security.Claims;
using System.Threading.Tasks;
using UserInterface.Controllers;
using Xunit;

namespace Test.Controllers.Transaction {

    public class TestsControllerTest : Controller {
        private Mock<ILogger<TestsController>> _loggerMock = null;
        private Mock<IUserCheckService> _userCheckServiceMock = null;

        private ClaimsPrincipal _user = null;
        private TestsController _controller = null;

        public TestsControllerTest() {
            this._loggerMock = new Mock<ILogger<TestsController>>();
            this._userCheckServiceMock = new Mock<IUserCheckService>();

            this._user = new ClaimsPrincipal(new ClaimsIdentity(new Claim[] {
                new Claim(ClaimTypes.Name, "Administrator"),
                new Claim(ClaimTypes.Role, "Administrator")
            }, "mock"));

            // この場合はtrueを返す
            // this._userCheckServiceMock.Setup(e => e.IsCheckedAsync(this._user)).ReturnsAsync(true);

            // この方法だとうまくいかなかった
            this._userCheckServiceMock.Setup(e => e.IsCheckedAsync(this._user, null)).ReturnsAsync(true);

            this._controller = new TestsController(
                this._loggerMock.Object,
                this._userCheckServiceMock.Object
            );

            this._controller.ControllerContext = new ControllerContext() {
                HttpContext = new DefaultHttpContext() { User = this._user }
            };
        }

        [Fact]
        public async Task<int> RegisterAsync() {
            // 適当にnullにしています。
            var result = await this._controller.Register(null);

            return 1;
        }
    }
}

using Microsoft.AspNetCore.Authorization;
using Microsoft.AspNetCore.Mvc;
using Microsoft.AspNetCore.Http;
using Microsoft.Extensions.Logging;
using System;
using System.Collections.Generic;
using System.Linq;
using System.Net;
using System.Threading.Tasks;
using ApplicationService;
using DomainModel;
using DomainService;
using System.Security.Claims;
using Microsoft.AspNetCore.Identity;

namespace UserInterface.Controllers {
    public class TestsController : Controller {
        private readonly ILogger<TestsController> _logger;
        private readonly IUserCheckService _userCheckService;

        public TestsController(ILogger<TestsController> logger, IUserCheckService userCheckService) {
            this._logger = logger;
            this._userCheckService = userCheckService;
        }

        // 説明のため属性は省略しています。
        public async Task<IActionResult> Register(IList<Test> tests) {
            // ①Contollerの引数をListで渡す時
            //  this._userCheckService.IsCheckedAsyncの引数が
            //  1つの時はtrueを返していた。
            //  2つの時にfalseになってしまう。

            // true
            bool isCheckedUser1 = Task.FromResult(await this._userCheckService.IsCheckedAsync(this.User)).Result;

            // false
            bool isCheckedUser2 = Task.FromResult(await this._userCheckService.IsCheckedAsync(this.User, null)).Result;

            // ②Contollerの引数を単一データで渡す時
            // this._userCheckService.IsCheckedAsyncの引数が2つでも、trueになりました。
        }
    }
}

using System;
using System.Security.Claims;
using System.Threading.Tasks;
using DomainModel;

namespace DomainService {
    public interface IUserCheckService {
        Task<bool> IsCheckedAsync(ClaimsPrincipal user);
        Task<bool> IsCheckedAsync(ClaimsPrincipal user, Project project);
    }
}

Comments

[退会済みユーザー]

何を作っているかと開発環境を質問欄を編集して追記してください。現状、ASP.NET の何かを、開発言語 C# で作っているということがタグから分かるぐらいです。コードを読めばわかるとは言わないでください。あなたが質問の一行目に書けば済むことなのですから。

[退会済みユーザー]

コメントありがとうございます。
1行目に追記させていただきました。よろしくお願いします。

[退会済みユーザー]

> 1行目に追記させていただきました。よろしくお願いします。

開発環境（OS, Visual Studio, Core のバージョンなど）を書かないのは何か理由があるのですか？

[退会済みユーザー]

コードを拝見しました。残念ながら内容が分からないし問題を再現することもできないので回答できないのですが、ざっと見気になった点を書いておきます。

> bool isCheckedUser1 = Task.FromResult(await this._userCheckService.IsCheckedAsync(this.User)).Result;
> bool isCheckedUser2 = Task.FromResult(await this._userCheckService.IsCheckedAsync(this.User, null)).Result;

同期・非同期を混ぜるという典型的なデッドロックパターンにように見えます。戻り値が取得できたようなのでデッドロックにはなってなくて問題の原因ではなさそうですが、単純に以下のようにすべきではないでしょうか？

bool isCheckedUser1 = await this._userCheckService.IsCheckedAsync(this.User);
bool isCheckedUser2 = await this._userCheckService.IsCheckedAsync(this.User, null);

[退会済みユーザー]

開発環境について追記させて頂きました。
Windows10 Pro, Visual Studio Professional 2019, .NET 5

デッドロックパターンについて間違い指摘ありがとうございます。
bool isCheckedUser1 = await this._userCheckService.IsCheckedAsync(this.User);

修正はしましたが、これが原因ではなかったです。
他の原因についても調べているところです。

Answer 1

原因はわからないのですが、nullまたは空オブジェクトではだめで、It.IsAny<T>()を使うことで解決しました。

using ApplicationService;
using DomainModel;
using DomainService;
using Microsoft.AspNetCore.Http;
using Microsoft.AspNetCore.Mvc;
using Microsoft.Extensions.Logging;
using Moq;
using System;
using System.Collections.Generic;
using System.Linq;
using System.Security.Claims;
using System.Threading.Tasks;
using UserInterface.Controllers;
using Xunit;

namespace Test.Controllers.Transaction {

    public class TestsControllerTest : Controller {
        private Mock<ILogger<TestsController>> _loggerMock = null;
        private Mock<IUserCheckService> _userCheckServiceMock = null;

        private ClaimsPrincipal _user = null;
        private TestsController _controller = null;

        public TestsControllerTest() {
            this._loggerMock = new Mock<ILogger<TestsController>>();
            this._userCheckServiceMock = new Mock<IUserCheckService>();

            // nullまたは空オブジェクトではだめで、It.IsAny<T>()を使うことで解決しました。
            this._userCheckServiceMock.Setup(e => e.IsCheckedAsync(It.IsAny<ClaimsPrincipal>, It.IsAny<Project>())).ReturnsAsync(true);

            this._controller = new TestsController(
                this._loggerMock.Object,
                this._userCheckServiceMock.Object
            );

            // ダミーのUserはこれで対応
            this._user = new ClaimsPrincipal(new ClaimsIdentity(new Claim[] {
                new Claim(ClaimTypes.Name, "Administrator"),
                new Claim(ClaimTypes.Role, "Administrator")
            }, "mock"));
            
            this._controller.ControllerContext = new ControllerContext() {
                HttpContext = new DefaultHttpContext() { User = this._user }
            };
        }

        [Fact]
        public async Task<int> RegisterAsync() {
            // 適当にnullにしています。
            var result = await this._controller.Register(null);

            return 1;
        }
    }
}