TDD 和受保护的方法 [重复]

Question

我正在尝试通过创建现有 MVC 应用程序的副本来学习 TDD，但我正在使用 TDD 从头开始​​创建它的副本。

在我现有的应用程序中，我有一个 Application_AuthenticateRequest 方法，如下所示。

这是受保护的。我认为这些方法不应该被测试是否正确 - 即你应该只测试公共方法而不是私有和受保护的方法。如果这是真的，那么我是否会直接在下面编写受保护的方法而不编写任何测试？

public class MvcApplication : System.Web.HttpApplication
{
    protected void Application_Start()
    {
        AreaRegistration.RegisterAllAreas();

        WebApiConfig.Register(GlobalConfiguration.Configuration);
        FilterConfig.RegisterGlobalFilters(GlobalFilters.Filters);
        RouteConfig.RegisterRoutes(RouteTable.Routes);
        BundleConfig.RegisterBundles(BundleTable.Bundles);
        StaticDataSeeder.Seed();
    }

    protected void Application_AuthenticateRequest(Object sender, EventArgs e)
    {
        HttpCookie authCookie = Context.Request.Cookies[FormsAuthentication.FormsCookieName];

        if (authCookie == null) return;

        var authTicket = FormsAuthentication.Decrypt(authCookie.Value);

        if (authTicket == null) return;

        var userData = new UserDataModel(authTicket.UserData);

        var userPrincipal = new PaxiumPrincipal(new GenericIdentity(authTicket.Name), null)
        {
            Email = userData.Email,
            Menu = userData.Menu,
            RememberMe = userData.RememberMe
        };

        Context.User = userPrincipal;
    }
}

Answer 1

仅进行测试并不能使其成为 TDD。如果您已经假设私有方法可以做某事，那么开发不是由测试驱动的，是吗？

尝试盲目地先编写测试，您最终可能根本不需要私有方法，甚至将其作为另一个类中的公共方法。

Answer 2

是的，您的测试用例是正确的，您只能测试您可以调用的方法。所以通常公共方法是可测试的。同样，如果您的组件中有私有方法，这意味着它必须由该组件中的至少一个公共方法调用。因此，当您测试公共方法时，您也在同一测试中测试私有方法的功能。您可以将私有方法视为公共方法中的几行，使其远离该公共方法，以使代码更具可读性和可重用性。

Answer 3

受保护的方法在您正在创建的类和它的未来子类之间创建一个契约。如果您不想创建此合同，您会将这些方法定义为私有的，而不是受保护的。因此，我认为您应该测试这些方法的行为是否符合任何派生类的预期。

考虑两个类：

public abstract class BaseMathsClass {
    protected int Mult(int a, int b) {
        return a * b;
    }
}

public class ConcreteMathClass : BaseMathsClass {
    public int Square(int x) {
        return Mult(x, x);
    }
}

如果您乐于在ConcreteMathClass 的测试中测试Mult 方法的功能，那么您不需要为基类编写测试。但是，如果您确实为基类编写测试，这会在编写基类时巩固合同，并且意味着任何派生类都知道它们要注册的内容。你落在栅栏的哪一边，取决于你的意见。 This question 有其他意见。

你还说你相信你不需要测试私有方法。这是真的，因为您不需要编写特定的测试来直接调用私有方法，但是您应该通过针对类的公共方法的测试来隐式地测试任何私有方法的功能。在编写测试时，您不必真正关心您的类是否具有以下任一实现：

实施 1

class MyClass {
    public int GetSquare(int someValue) {
        return someValue * someValue;
    }
}

实施 2

class MyClass {
    public int GetSquare(int someValue) {
        return Mult(someValue, someValue);
    }

    private int Mult(int a, int b) {
        return a * b;
    }
}

使用实现 2，您无需编写专门调用 Mult 的测试，GetSquare 的测试充分锻炼了这段代码。这并不意味着您只需编写另一个私有方法Add，因为目前在您的公共接口上不需要它。

您可能还对this question 的答案感兴趣。