使用XUnit测试void方法的最佳实践有哪些？

void方法没有返回值，测试核心是验证执行行为是否符合预期，完全不需要为了凑断言强行给业务逻辑加不必要的异常。

1. 先做依赖解耦，为行为测试铺路 #

你提供的方法包含三个外部逻辑：读取数据库员工数据、判断考勤周期、调用邮件服务发信。必须把数据库、邮件服务这两个外部依赖通过接口注入到业务类中，不要硬编码在方法内部，才能用测试替身（Mock）验证行为是否符合预期。
参考改造后的业务代码：

// 定义依赖接口
public interface IEmployeeRepository
{
    IEnumerable<Employee> GetEmployeesWithCheckInRecords();
}
public interface IMailService
{
    void SendReminderEmail(Employee employee);
}

// 业务类注入依赖
public class ReminderService
{
    private readonly IEmployeeRepository _employeeRepo;
    private readonly IMailService _mailService;

    public ReminderService(IEmployeeRepository employeeRepo, IMailService mailService)
    {
        _employeeRepo = employeeRepo;
        _mailService = mailService;
    }

    public void SendCheckInReminder()
    {
        var employees = _employeeRepo.GetEmployeesWithCheckInRecords();
        if (employees == null) return;
        foreach (var emp in employees)
        {
            if (emp.LastCheckIn < DateTime.Now.AddMonths(-3))
            {
                _mailService.SendReminderEmail(emp);
            }
        }
    }
}

2. 用Mock框架验证依赖调用行为 #

常用Mock库可以选Moq、NSubstitute，不需要真实连接数据库、发送邮件，只要验证对应依赖的方法是否被调用、调用参数是否符合要求即可。以下是用NSubstitute写的测试示例：

using Xunit;
using NSubstitute;

public class ReminderServiceTests
{
    private readonly IEmployeeRepository _mockEmployeeRepo;
    private readonly IMailService _mockMailService;
    private readonly ReminderService _reminderService;

    public ReminderServiceTests()
    {
        // 初始化测试替身
        _mockEmployeeRepo = Substitute.For<IEmployeeRepository>();
        _mockMailService = Substitute.For<IMailService>();
        _reminderService = new ReminderService(_mockEmployeeRepo, _mockMailService);
    }

    [Fact]
    public void SendCheckInReminder_员工上次考勤超过3个月_发送提醒邮件()
    {
        // Arrange 构造测试数据
        var validEmployee = new Employee 
        { 
            Id = 1, 
            LastCheckIn = DateTime.Now.AddMonths(-4) 
        };
        _mockEmployeeRepo.GetEmployeesWithCheckInRecords()
            .Returns(new List<Employee> { validEmployee });

        // Act 调用待测void方法
        _reminderService.SendCheckInReminder();

        // Assert 验证邮件服务被调用1次，参数符合要求
        _mockMailService.Received(1).SendReminderEmail(Arg.Is<Employee>(e => e.Id == 1));
    }

    [Fact]
    public void SendCheckInReminder_员工上次考勤不足3个月_不发送邮件()
    {
        // Arrange
        var invalidEmployee = new Employee 
        { 
            Id = 2, 
            LastCheckIn = DateTime.Now.AddMonths(-2) 
        };
        _mockEmployeeRepo.GetEmployeesWithCheckInRecords()
            .Returns(new List<Employee> { invalidEmployee });

        // Act
        _reminderService.SendCheckInReminder();

        // Assert 验证邮件服务从未被调用
        _mockMailService.DidNotReceive().SendReminderEmail(Arg.Any<Employee>());
    }

    [Fact]
    public void SendCheckInReminder_员工列表为空_无异常且不发邮件()
    {
        // Arrange
        _mockEmployeeRepo.GetEmployeesWithCheckInRecords().Returns((IEnumerable<Employee>)null);

        // Act
        _reminderService.SendCheckInReminder();

        // Assert
        _mockMailService.DidNotReceive().SendReminderEmail(Arg.Any<Employee>());
    }
}

3. 通用最佳实践 #

• 仅当业务逻辑本身预期会抛出异常时，才使用Assert.Throws做断言，不要为了测试强行加无意义的异常
• 每个测试仅覆盖一个场景：符合条件发信、不符合条件不发信、边界值情况分开写测试，不要把多个不同逻辑的断言塞到同一个测试用例里
• 测试命名规则清晰：推荐用「待测方法名_触发场景_预期行为」的格式，看测试名就能知道验证逻辑
• 如果待测void方法会修改类内部的可访问状态，也可以直接读取类的公开属性/状态做断言，不一定都要验证依赖调用

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