C#单元测试学习：Moq+XUnit测试RoosterService遇类型转换错误

你的错误根源是把Moq配置方法的返回值当成了预期数据：playersRepoMock.Setup(...)返回的是Moq内部的IReturnsResult<IPlayersRepo>类型（用于链式配置Mock行为），而非你期望的IList<Player>，直接把它传给Assert.Equal自然会触发类型转换错误。

修正后的测试代码 #

namespace MoqTests
{
    public class RoosterServiceTests
    {
        private readonly RoosterService _service;
        private readonly Mock<IPlayersRepo> _playersRepoMock = new Mock<IPlayersRepo>();

        public RoosterServiceTests()
        {
            _service = new RoosterService(_playersRepoMock.Object);
        }
        
        [Fact]
        public void GetPlayersFromRepo_ShouldReturnPlayers()
        {
            // Arrange：先定义预期的玩家列表
            var expectedPlayers = new List<Player>
            {
                new Player { Name = "TestPlayer1", Age = 25 },
                new Player { Name = "TestPlayer2", Age = 30 }
            };
            // 配置Mock，让它调用GetPlayerList时返回预期列表
            _playersRepoMock.Setup(p => p.GetPlayerList()).Returns(expectedPlayers);
            
            // Act：调用待测试方法
            var actualPlayers = _service.GetPlayersFromRepo();

            // Assert：对比实际返回值和预期值
            Assert.Equal(expectedPlayers, actualPlayers);
            // 额外验证：确保Mock的GetPlayerList方法确实被调用了一次
            _playersRepoMock.Verify(p => p.GetPlayerList(), Times.Once);
        }
    }
}

关键修正点说明 #

1. 分离预期数据与Mock配置：先创建独立的expectedPlayers列表，再通过Setup让Mock返回这个列表，而不是把Setup的结果赋值给预期变量。
2. 正确的断言目标：用实际返回的actualPlayers和真实的预期列表expectedPlayers对比，而非Moq的配置对象。
3. 补充验证Mock调用：使用Verify方法确认依赖的IPlayersRepo.GetPlayerList确实被调用，确保业务逻辑符合预期（RoosterService确实委托给了Repo的方法）。

额外注意事项 #

如果你的Player类没有重写Equals和GetHashCode方法，Assert.Equal会默认进行引用比较（即只有两个列表引用同一个对象时才会通过）。如果需要对比对象内容，可以：

• 重写Player类的Equals和GetHashCode，基于Name、Age等唯一属性判断相等
• 使用XUnit的集合断言重载，自定义相等逻辑：

  Assert.Equal(expectedPlayers, actualPlayers, EqualityComparer<Player>.Create((a, b) => 
      a.Name == b.Name && a.Age == b.Age && a.PlayersClub.ClubName == b.PlayersClub.ClubName));
  

内容的提问来源于stack exchange，提问作者bancqm