如何Mock被Mock方法内部的方法？解决xUnit仓储测试返回false问题

你的测试代码核心错误 #

1. 错误Mock了整个接口：直接MockIPatientRepo接口后，调用Create时执行的是Moq生成的空实现，而非你写的包含try-catch和saveChanges调用的真实逻辑。
2. 未设置Mock方法返回值：仅Setup了Create方法的调用匹配，但未指定返回值，Moq对bool类型的默认返回值是false，这是断言失败的直接原因。
3. 测试逻辑偏离目标：测试应该验证真实仓储类的Create方法行为，而非Mock接口的虚拟行为。

正确测试方案（推荐） #

测试真实仓储类的Create方法，通过Mock依赖的数据库上下文（DbContext）隔离数据库操作：

前提：补全仓储类的依赖注入 #

确保仓储类通过构造函数接收DbContext依赖：

public class PatientRepo : IPatientRepo
{
    private readonly YourDbContext _db;

    public PatientRepo(YourDbContext db)
    {
        _db = db;
    }

    public bool Create(Patient model)
    {
        try {
            _db.patients.Add(model);
            saveChanges();
            return true;
        } catch {
            return false;
        }    
    }

    public int saveChanges() {   
        return _db.SaveChanges();
    }
}

测试代码（xUnit + AutoMock） #

// 测试：当SaveChanges成功时，Create返回true
public void CanCreatePatient_WhenSaveSucceeds_ReturnsTrue()
{
    var samplePatient = GetSamplePatient();

    using(var mock = AutoMock.GetLoose())
    {
        // Mock患者集合DbSet，避免Add操作报错
        var mockPatientsSet = mock.Mock<DbSet<Patient>>();
        mock.Mock<YourDbContext>()
            .Setup(db => db.patients)
            .Returns(mockPatientsSet.Object);

        // Mock DbContext的SaveChanges，模拟成功返回1
        mock.Mock<YourDbContext>()
            .Setup(db => db.SaveChanges())
            .Returns(1);

        // 创建真实仓储实例，AutoMock自动注入Mock的DbContext
        var repo = mock.Create<PatientRepo>();

        // 执行测试方法
        var actual = repo.Create(samplePatient);

        // 断言结果
        Assert.True(actual);

        // 可选：验证Add和SaveChanges是否被正确调用
        mockPatientsSet.Verify(set => set.Add(samplePatient), Times.Once);
        mock.Mock<YourDbContext>().Verify(db => db.SaveChanges(), Times.Once);
    }
}

// 额外测试：当SaveChanges抛出异常时，Create返回false
public void CanCreatePatient_WhenSaveFails_ReturnsFalse()
{
    var samplePatient = GetSamplePatient();

    using(var mock = AutoMock.GetLoose())
    {
        var mockPatientsSet = mock.Mock<DbSet<Patient>>();
        mock.Mock<YourDbContext>()
            .Setup(db => db.patients)
            .Returns(mockPatientsSet.Object);

        // 模拟SaveChanges抛出异常
        mock.Mock<YourDbContext>()
            .Setup(db => db.SaveChanges())
            .Throws(new Exception("测试异常"));

        var repo = mock.Create<PatientRepo>();

        var actual = repo.Create(samplePatient);

        Assert.False(actual);
    }
}

备选方案（Mock内部方法，不推荐） #

如果一定要Mock仓储类的saveChanges内部方法，需先将该方法改为virtual，再用Moq的Partial Mock：

修改仓储类： #

public virtual int saveChanges() {   
    return _db.SaveChanges();
}

测试代码： #

public void CanCreatePatient_WithPartialMock_ReturnsTrue()
{
    var samplePatient = GetSamplePatient();

    using(var mock = AutoMock.GetLoose())
    {
        // 创建仓储类的Partial Mock，仅Mock指定方法，其余用真实逻辑
        var repoMock = mock.Mock<PatientRepo>();
        repoMock.Setup(r => r.saveChanges()).Returns(1);
        // 启用真实方法调用
        repoMock.CallBase = true;

        // 同样需要Mock DbContext的患者集合
        var mockPatientsSet = mock.Mock<DbSet<Patient>>();
        mock.Mock<YourDbContext>()
            .Setup(db => db.patients)
            .Returns(mockPatientsSet.Object);

        var actual = repoMock.Object.Create(samplePatient);

        Assert.True(actual);
    }
}

关键注意事项 #

• 测试核心是验证真实业务逻辑，Mock仅用于隔离外部依赖（如数据库），不要替换待测试代码。
• 确保仓储类的依赖（如DbContext）通过构造函数注入，这是实现依赖替换的基础。
• 优先Mock外部依赖而非内部方法，测试场景更贴近真实运行环境。

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