如何测试依赖于当前日期的逻辑[重复]

Question

我有这种方法，它取决于当前日期。它检查今天是周日、周一、周二还是周三，然后给出 5 天的交货时间来运送物品。如果是周四、周五或周六，那么它会提供 6 天的提前期来考虑周末。

private DateTime GetEstimatedArrivalDate()
{
    DateTime estimatedDate; 
    if (DateTime.Now.DayOfWeek >= DayOfWeek.Thursday)
    {
        estimatedDate = DateTime.Now.Date.AddDays(6);
    }
    else
    {
        estimatedDate = DateTime.Now.Date.AddDays(5);
    }
    return estimatedDate; 
}

实际的估计逻辑更复杂。为了这个问题，我已经简化了它。我的问题是如何为这样的事情编写单元测试，这取决于今天的日期？

Answer 1

您需要将当前日期作为参数传入：

private DateTime GetEstimatedArrivalDate(DateTime currentDate)
{
    DateTime estimatedDate; 
    if (currentDate.DayOfWeek >= DayOfWeek.Thursday)
    {
        estimatedDate = currentDate.AddDays(6);
    }
    else
    {
        estimatedDate = currentDate.AddDays(5);
    }
    return estimatedDate; 
}

在实际代码中，您可以这样称呼它：

DateTime estimatedDate = GetEstimatedArrivalDate(DateTime.Now.Date);

那么你可以如下测试：

DateTime actual = GetEstimatedArrivalDate(new DateTime(2010, 2, 10));
DateTime expected = ...;
// etc...

请注意，这还修复了程序中的一个潜在错误，即在连续调用 DateTime.Now 之间日期会发生变化。

Answer 2

一般来说，你会想在一个接口后面抽象出获取当前日期和时间的方法，例如：

public interface IDateTimeProvider
{
    DateTime Now { get; }
}

真正的服务是：

public class DateTimeProvider: IDateTimeProvider
{
    public DateTime Now
    {
        get
        {
            return DateTime.Now;
        }
    }
}

测试服务是：

public class TestDateTimeProvider: IDateTimeProvider
{
    private DateTime timeToProvide;
    public TestDateTimeProvider(DateTime timeToProvide)
    {
        this.timeToProvide = timeToProvide;
    }

    public DateTime Now
    {
        get
        {
            return timeToProvide;
        }
    }
}

对于需要当前时间的服务，让它们将 IDateTimeProvider 作为依赖项。对于真实的东西，传递一个新的 DateTimeProvider();当你是一个组件时，传入一个新的 TestDateTimeProvider(timeToTestFor)。

Answer 3

让您的类采用IClock 参数（通过构造函数或属性）

interface IClock
{
    DateTime Now { get; }
}

然后您可以使用假实现进行测试

class FakeClock : IClock
{
    DateTime Now { get; set }
}

其余时间都是真正的实现。

class SystemClock : IClock
{
    DateTime Now { get { return DateTime.Now; } }
}

Answer 4

我建议以Mark suggests 的形式执行此操作，但添加了一个用于生产用途的重载调用，该调用不带参数并使用 DateTime.Now

private DateTime GetEstimatedArrivalDate()
{
    return GetEstimatedArrivalDate(DateTime.Now);
}

private DateTime GetEstimatedArrivalDate(DateTime currentDate)
{
    DateTime estimatedDate; 
    if (currentDate.DayOfWeek >= DayOfWeek.Thursday)
    {
        estimatedDate = currentDate.AddDays(6);
    }
    else
    {
        estimatedDate = currentDate.AddDays(5);
    }
    return estimatedDate; 
}

Answer 5

这样做的一种“常见”方式是“伪造”当前系统日期（可以通过多种方式完成），然后在“已知”日期测试您的代码。

另一种有趣的方法是将您的实现稍微更改为：

private DateTime GetEstimatedArrivalDate()
{
    return GetEstimatedArrivalDate(DateTime.Now);
}

private DateTime GetEstimatedArrivalDate(DateTime forDate)
{
    DateTime estimatedDate; 
    if (forDate.DayOfWeek >= DayOfWeek.Thursday)
    {
        estimatedDate = forDate.Date.AddDays(6);
    }
    else
    {
        estimatedDate = forDate.Date.AddDays(5);
    }
    return estimatedDate; 
}

然后使用带有参数的方法来测试“立即”日期。

Answer 6

似乎有足够数量的案例，您可以明确地测试它们。该方法取决于今天的日期，但输出仅取决于星期几，并且每个日期都有星期几。

Answer 7

您可以在正常执行期间传入一个返回 DateTime.Now 的委托，然后在您的测试中传入另一个返回固定日期的委托，并基于此断言您的结果。

Answer 8

我会给出有争议的答案，不要测试它。

逻辑是微不足道的，它具有零依赖性，我相信良好的代码覆盖率，但当它增加复杂性而没有真正的收益时则不然。