45

我有一个类，它有一个私有成员，该成员为 type System.Windows.Forms.Timer。每次我的计时器滴答作响时，都会调用一个私有方法。

  1. 是否值得测试该方法？（因为它是私人的）
  2. 我该如何测试它？（我知道我可以让我的测试类继承我想要测试的类......）
  3. 我应该嘲笑我的计时器吗？因为如果我必须测试一个使用内部计时器的类，我的测试可能需要很长时间才能完成，对吧？

编辑：

实际上，该方法依赖于时间，这里是代码：

private void alertTick(object sender, EventArgs e) {
    if (getRemainingTime().Seconds <= 0) {
        Display.execute(Name, WarningState.Ending, null);
        AlertTimer.Stop();
    }
    else {
        var warning = _warnings.First(x => x == getRemainingTime());

        if (warning.TotalSeconds > 0)
            Display.execute(Name, WarningState.Running, warning);
    }
}

如您所见，如果计时器正在运行，它会Display.execute()使用与结束时不同的参数进行调用（当剩余时间等于 0 时）。那会是设计的问题吗？

4

2 回答 2

46
  1. 您不是在测试方法（私有或公共） - 您正在验证班级的行为。而且，如果您还没有验证某些行为，那么您就无法判断它是否已实施。可以通过多种方式调用此行为 - 类的公共接口，或某些依赖事件。行为调用也不一定会改变公共接口所达到的内容，与依赖项的交互也很重要。
  2. 请参见下面的示例 - 它显示了如何测试这种“隐藏”行为。
  3. 请参见下面的示例 - 它展示了如何拆分职责、注入依赖项和模拟它们。

实际上你的班级有太多的责任 - 一个是安排一些任务，另一个 - 执行一些动作。尝试将您的班级分成两个具有单一职责的单独班级。

因此，调度转到调度程序 :) 调度程序的 API 可能类似于：

public interface IScheduler
{
    event EventHandler<SchedulerEventArgs> Alarm;
    void Start();
    void Stop();
}

现在忘记调度程序。返回并实现你的第二个类，这将显示一些警告。让我们先测试一下（使用起订量）：

[Test]
public void ShouldStopDisplayingWarningsWhenTimeIsOut()
{
    Mock<IDisplay> display = new Mock<IDisplay>();
    Mock<IScheduler> scheduler = new Mock<IScheduler>();                      

    Foo foo = new Foo("Bar", scheduler.Object, display.Object);
    scheduler.Raise(s => s.Alarm += null, new SchedulerEventArgs(0));

    display.Verify(d => d.Execute("Bar", WarningState.Ending, null));
    scheduler.Verify(s => s.Stop());
}

编写实现：

public class Foo
{
    private readonly IScheduler _scheduler;
    private readonly IDisplay _display;
    private readonly string _name;

    public Foo(string name, IScheduler scheduler, IDisplay display)
    {
        _name = name;
        _display = display;
        _scheduler = scheduler;
        _scheduler.Alarm += Scheduler_Alarm;
        _scheduler.Start();
    }

    private void Scheduler_Alarm(object sender, SchedulerEventArgs e)
    {
        _display.Execute(_name, WarningState.Ending, null);
        _scheduler.Stop();
    }
}

测试通过。再写一个：

[Test]
public void ShouldNotStopDisplayingWarningsWhenTimeRemains()
{
    Mock<IDisplay> display = new Mock<IDisplay>(MockBehavior.Strict);
    Mock<IScheduler> scheduler = new Mock<IScheduler>(MockBehavior.Strict);
    scheduler.Setup(s => s.Start());

    Foo foo = new Foo("Bar", scheduler.Object, display.Object);
    scheduler.Raise(s => s.Alarm += null, new SchedulerEventArgs(1));
}

测试失败。啊，你需要剩余时间的条件：

private void Scheduler_Alarm(object sender, SchedulerEventArgs e)
{
    if (e.RemainingTime > 0)
        return;

    _display.Execute(_name, WarningState.Ending, null);
    _scheduler.Stop();
}

您可以继续为您的类编写测试，该类负责处理调度程序警报并在显示上执行一些警告。完成后，您可以为您的IScheduler接口编写实现。无论您将如何实现调度 - 通过 System.Windows.Forms.Timer 或通过 System.ThreadingTimer 或其他方式。

于 2012-07-15T23:10:58.927 回答
27

是否值得测试该方法？（因为它是私人的）

您的目的是决定您的代码是否有效。即使它是一个私有方法，它也应该生成一个公共接口可以访问的输出。您应该以用户可以知道它是否工作的方式设计您的课程。

此外，当您进行单元测试时，如果您可以模拟计时器，则可以访问分配给计时器的 Elapsed 事件的回调。

我该如何测试它？（我知道我可以让我的测试类继承我想要测试的类......）

您可以在此处使用适配器类。首先，您必须定义一个抽象，因为 Timer 类不提供。

public interface ITimer
{
    void Start();
    void Stop();
    double Interval { get; set; }
    event ElapsedEventHandler Elapsed;
    //and other members you need
}

然后你可以在一个适配器类中实现这个接口，只是继承自 Timer 类。

public class TimerAdaper : Timer, ITimer { }

您应该在构造函数（或作为属性）中注入抽象，以便在测试中模拟它。

public class MyClass
{
    private readonly ITimer _timer;

    public MyClass(ITimer timer)
    {
        _timer = timer
    }
}

我应该嘲笑我的计时器吗？因为如果我必须测试一个使用内部计时器的类，我的测试可能需要很长时间才能完成，对吧？

当然，你应该嘲笑你的计时器。您的单元测试不能依赖于系统时间。您应该通过模拟来引发事件并查看代码的行为方式。

于 2012-07-15T21:52:50.987 回答