我的私有方法中的代码行从未被覆盖

Question

下面是动作方法的一段代码：

private static IEnumerable<SelectListItem> GetProductTypes()
{
    System.Text.RegularExpressions.Regex filter = new System.Text.RegularExpressions.Regex("Prodcut1|Prodcut2|Prodcut3");
    var prodtypes = from ProdType e in Enum.GetValues(typeof(ProdType))
                    where filter.IsMatch(e.ToString())
                    select new { Id = (int)e, Name = e.ToString() };

    if (prodtypes  != null)
    {
        return prodtypes .Select(x => new SelectListItem
               {
                   Value = x.Id.ToString(),
                   Text = x.Name
               });
    }
    return new List<SelectListItem>();
}

其中：ProdType 是枚举。其中包含一些产品类型及其值。

我已经为该方法编写了单元测试：-

[TestMethod]
public void GetProductTypes_Test()
{
    //Arrange        
    PrivateType pvtType = new PrivateType(typeof(ProductController));
    //Act
    var actual = (IEnumerable<SelectListItem>)pvtType.InvokeStatic("GetProductTypes");

    //Assert
    Assert.IsNotNull(actual);
    Assert.IsInstanceOfType(actual, typeof(IEnumerable<SelectListItem>));
    Assert.AreEqual(3, actual.ToList().Count); //here : 3 = Product1,Product2,Product3
}

但是当我通过代码覆盖率选项检查它的代码覆盖率时，它没有覆盖下面的代码行：-

return new List<SelectListItem>();

谁能建议我在这里做错了什么？是的，这是我正在测试的私有方法。因为我想测试所有私有方法。

Answer 1

正如 Patryk 已经说过的：prodtypes 永远不会是 null。
要获得 100% 的覆盖率，您可以将该方法调整为以下之一（仅第二部分 - 第一部分保持不变）：

private static IEnumerable<SelectListItem> GetProductTypes()
{
    //select prodtypes
    if (prodtypes != null && prodtypes.Count() > 0)
    {
        return prodtypes.Select(x => new SelectListItem
               {
                    Value = x.Id.ToString(),
                    Text = x.Name
               });
    }
    return new List<SelectListItem>();
}

或更短且性能更高的，因为它不应该是null：

private static IEnumerable<SelectListItem> GetProductTypes()
{
    //select prodtypes
    return prodtypes.Select(x => new SelectListItem
       {
            Value = x.Id.ToString(),
            Text = x.Name
       });
}

但是
正如约翰所说：在大多数情况下，您不应该对私有方法进行单元测试，因为它们应该被一些公共接口/方法所覆盖。如果没有涵盖，请考虑该方法 - 也许甚至不需要 - 我已经发生过几次了。
但我也添加了“在大多数情况下”，因为在某些情况下测试私有方法可能是有意义的。在我的情况下，例如我的硬件有一些错误处理方法，这些方法在正常单元测试期间没有附加，所以我直接调用“私有错误处理程序”来检查它是否行为正确。但是你的方法看起来不像这样……

更新
正如 Episodex 所建议的，该方法的更好解决方案：

private static IEnumerable<SelectListItem> GetProductTypes()
{
    var filter = new System.Text.RegularExpressions.Regex("Prodcut1|Prodcut2|Prodcut3");
    return from ProdType e in Enum.GetValues(typeof(ProdType))
           where filter.IsMatch(e.ToString())
           select new SelectListItem 
               {
                   Value = ((int)e).ToString(), 
                   Text = e.ToString() 
                };
}