ASP.net MVC RTM 测试命名约定

Question

我正在开发一个 asp.net mvc 应用程序并编写我的单元测试 BDD 风格。 例如。

GetResource_WhenResourceFileExists_ShouldReturnResources()

但是当我为控制器编写测试时，我通常有两个同名的方法。一种不带参数用于获取请求，另一种带参数用于帖子。有人有一个很好的命名约定来区分两者吗？

我能想到：

1.
LogIn_WithParameters_ShouldReturnLogInView()
LogIn_WithoutParameters_WhenAuthenticationFailed_ShouldReturnLogInView()
LogIn_WithoutParameters_WhenAuthenticationPassed_ShouldReturnProfileRedirect()

2.
LogIn_Get_ShouldReturnLogInView()
LogIn_Post_WhenAuthenticationFailed_ShouldReturnLogInView()
LogIn_Post_WhenAuthenticationPassed_ShouldReturnProfileRedirect()

3.
LogIn_ShouldReturnLogInView()
LogIn_WhenCalledWithParametersAndAuthenticationFailed_ShouldReturnLogInView()
LogIn_WhenCalledWithParametersAndAuthenticationPassed_ShouldReturnProfileRedirect()

有什么意见吗？

Answer 1

我不太喜欢的一种选择是为控制器操作指定不同的名称，然后使用 ActionName 属性重命名它们：

public ActionResult Login() {
    // ... code ...
    return View();
}

[HttpPost]
[ActionName("Login")]
public ActionResult LoginPost(... some params ...) {
    // ... more code ...
    return View();
}

这本质上是用一个问题（单元测试命名）替换另一个问题（更难读取控制器代码） 。尽管如此，您可能会发现这种模式很有吸引力，因为它确实解决了上述问题。

Answer 2

我使用与您问题中的命名约定类似的命名约定，即 method_scenario_expected
我认为你应该详细说明“场景”部分 - 如果你传递参数 - 让读者知道它们有什么特别之处。

请记住，以这种方式命名您的测试更多的是“TDD 导向”，而不是 BDD - BDD 测试名称应该与规则和“行为有关：”。

如果您觉得当前的命名约定不影响代码的可读性 - 请随意尝试四处寻找适合您的方法。

Answer 3

我认为这是一个完美的例子，说明了为什么单元测试的严格命名约定没有吸引力。

您提出的方案仅在有两种方法重载时才有效：一种带参数，另一种不带参数。它不会扩展到具有多个具有不同参数的重载的情况。

就我个人而言，我更喜欢更宽松的命名约定，可以概括为：

[Action][Will|Should|Is|...][Result]

这使我可以灵活地命名我的测试，

SutIsPathResolutionCommand
ExecuteWithNullEvaluationContextWillThrow
ExecuteWillAddDefaultClaimsTransformationServiceWhenNoConnectionServiceIsAvailable

我必须承认我很少阅读测试的名称。相反，我阅读了它的作用的规范（即测试代码）。名字对我来说并不那么重要。

Answer 4

我可能不会回答你的问题，但我想分享我所做的事情。
我不遵循特定的命名约定，但我尝试给出名称来解释测试方法试图测试的内容。在某些情况下，我需要更多解释，我会添加描述 [Test(“此测试评估特定用户回答了多少问题”)]。

需要确保的一件事是测试更具可读性并且易于理解。

Answer 5

我使用以下格式，这对我来说非常有效：

[TestFixture]    
public class Log_in_with_parameters_should
{
    [Test]
    public void Return_the_log_in_view() {}
}

[TestFixture]    
public class Log_in_without_parameters_should
{
    [Test]
    public void Return_the_log_in_view_when_the_authentication_failed() {}

    [Test]
    public void Redirect_to_the_profile_when_the_authentication_passed() {}
}