测试私有方法调用：LoginPresenter的OnError测试合理性探讨

现有代码 #

LoginPresenter 类 #

public class LoginPresenter
{
    private ILoginView view;
    private APIWrapper api;
    // 省略其他代码
    public virtual IEnumerator Login(string email, string password)
    {
        return api.Login(email, password, OnSuccess, OnError);
    }

    private void OnError(HttpError error)
    {
        switch (error.statusCode)
        {
            case 0:
                view.ShowMessage("Check your Internet Connection");
                break;
            default:
                view.ShowMessage("Invalid Credentials");
                break;
        }
    }
    // 省略其他代码
}

当前测试代码 #

public class LoginPresenterTests
{
    private LoginPresenter presenter;
    private ILoginView view;
    // 省略其他代码
    [Test]
    public void _03_Test_LoginOnError()
    {
        //Arrange
        Dictionary<int, string> statusCodeMessages = new Dictionary<int, string>()
        {
            {0, "Check your Internet Connection"},
            {401, "Invalid Credentials"}
        };

        //Act
        foreach (var statusCodeMessage in statusCodeMessages)
        {
            object[] args = { new HttpError(statusCodeMessage.Key, "", "") };
            ReflectionUtils.Invoke(presenter, "OnError", args);
            
            //Assert
            view.Received().ShowMessage(statusCodeMessage.Value);
        }
    }
    // 省略其他代码
}

核心疑问 #

我需要编写登录失败场景的测试，但不确定正确的实现方式。目前的测试代码存在明显问题：测试逻辑完全复刻了原方法的分支判断，一旦原代码里的提示字符串被修改，测试就会直接失败。这显然不符合测试的本质。

我现在有两个困惑：

1. 这种复刻原逻辑的测试是否合理？
2. 是否只需要验证登录失败时OnError被调用即可？但OnError是私有回调方法，我不知道该如何验证它是否被触发。

问题分析与优化方案 #

1. 当前测试的问题本质 #

你说的没错，当前测试完全复刻业务逻辑，属于过度耦合的测试。测试的核心应该是验证「输入对应预期输出」，而不是重复实现一遍业务规则。一旦业务逻辑（比如提示文案、状态码分支）变动，测试就必须同步修改，失去了测试的稳定性和价值。

2. 正确的测试方向：分层验证 #

我们应该把测试拆分为两个层面，分别验证不同的职责：

层面一：验证Login方法是否正确传递回调

不需要直接测试私有方法OnError，而是通过模拟APIWrapper的行为，验证当API登录失败时，LoginPresenter是否正确触发了对应的错误处理流程。

用Moq这类 mocking 框架可以实现：

[Test]
public void Login_WhenApiReturnsError_ShouldTriggerCorrectViewMessage()
{
    // Arrange
    var mockApi = new Mock<APIWrapper>();
    var mockView = new Mock<ILoginView>();
    var presenter = new LoginPresenter(mockView.Object, mockApi.Object);
    var networkError = new HttpError(0, "", "");

    // 模拟API登录失败时，调用传入的OnError回调
    mockApi.Setup(api => api.Login(It.IsAny<string>(), It.IsAny<string>(), It.IsAny<Action>(), It.IsAny<Action<HttpError>>()))
           .Callback((string email, string password, Action onSuccess, Action<HttpError> onError) =>
           {
               onError(networkError); // 触发错误回调
           });

    // Act
    presenter.Login("test@example.com", "password").MoveNext();

    // Assert
    // 验证View是否显示了对应错误提示（关注最终业务结果，而非复刻逻辑）
    mockView.Verify(v => v.ShowMessage("Check your Internet Connection"), Times.Once);
}

层面二：单独验证错误处理逻辑（可选）

如果确实需要验证OnError的分支逻辑，不要用反射调用私有方法，而是把错误处理逻辑提取到独立的可测试组件中，比如单独的LoginErrorHandler类：

// 提取后的错误处理类
public class LoginErrorHandler
{
    public string GetErrorMessage(HttpError error)
    {
        return error.statusCode switch
        {
            0 => "Check your Internet Connection",
            _ => "Invalid Credentials"
        };
    }
}

// 修改LoginPresenter，依赖这个处理类
public class LoginPresenter
{
    private ILoginView view;
    private APIWrapper api;
    private LoginErrorHandler errorHandler;

    public LoginPresenter(ILoginView view, APIWrapper api, LoginErrorHandler errorHandler)
    {
        this.view = view;
        this.api = api;
        this.errorHandler = errorHandler;
    }

    private void OnError(HttpError error)
    {
        var message = errorHandler.GetErrorMessage(error);
        view.ShowMessage(message);
    }
}

这样就可以直接测试LoginErrorHandler的逻辑，不需要依赖反射，也不会和LoginPresenter的其他逻辑耦合：

[Test]
public void GetErrorMessage_StatusCode0_ReturnsConnectionMessage()
{
    var handler = new LoginErrorHandler();
    var error = new HttpError(0, "", "");
    
    var message = handler.GetErrorMessage(error);
    
    Assert.That(message, Is.EqualTo("Check your Internet Connection"));
}

[Test]
public void GetErrorMessage_NonZeroStatusCode_ReturnsInvalidCredentials()
{
    var handler = new LoginErrorHandler();
    var error = new HttpError(401, "", "");
    
    var message = handler.GetErrorMessage(error);
    
    Assert.That(message, Is.EqualTo("Invalid Credentials"));
}

3. 关于「是否只需验证OnError被调用」 #

如果你的测试目标是验证LoginPresenter正确转发了API的错误回调，只验证OnError被触发是可行的，但这属于行为测试，而非结果测试。不过通常更有价值的是验证最终的用户可见行为（比如View显示了正确提示），因为这才是业务真正关心的结果。

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