为什么以下测试使用 Mbunit 和 Rhino 模拟失败？

Question

我正在玩 MbUnit 和 Rhino Mocks 并做了一个简单的测试。它可能是设计不佳的代码，但我更专注于看看我是否能让测试通过。基本上，当汽车的发动机灯亮起时，汽车应该换油。这是代码：

public interface ICar
{
    bool EngineLight { get; set; }
    void GetOilChange();
    bool CheckEngineLight(ICar car);
}

public class Car : ICar
{
    public bool EngineLight { get; set; }

    public void GetOilChange()
    {
    }

    public bool CheckEngineLight(ICar car)
    {
        if (car.EngineLight)
            GetOilChange();
            return true;

        return false;
    } 
}

[TestFixture]
public class CarTests
{
[Test]
    public void WhenEngineLightIsOnGetOilChange()
    {
        var carMock = MockRepository.GenerateMock<ICar>();
        carMock.Stub(x => x.EngineLight).Return(true);

        Assert.AreEqual(true, new Car().CheckEngineLight(carMock)); //This passes

        carMock.AssertWasCalled(x => x.GetOilChange()); //This fails
    }
 }

Answer 1

在这个方法中：

public bool CheckEngineLight(ICar car)
{
    if (car.EngineLight)
        GetOilChange();
        return true;

    return false;
} 

您正在调用 GetOilChange();在您正在测试的对象上，而不是在您作为参数传递的模拟上。因此，确实没有在模拟上调用此方法。我猜你想要这个：

public bool CheckEngineLight(ICar car)
{
    if (car.EngineLight)
    {
        car.GetOilChange();
        return true;
    }
    return false;
} 

Answer 2

首先，这段代码有一个错误。在 IF 周围添加 {}：

public bool CheckEngineLight(ICar car)
    {
        if (car.EngineLight)
        {
            car.GetOilChange();
            return true;
        }

        return false;
    } 

它失败的原因是因为您正在调用 new Car().CheckEngineLight ... 而 new Car() 正在调用 GetOilChange ... carMock 没有调用 GetOilChange。将代码更改为car.GetOilChange()（参见上面的代码）。

您将 Car 对象传递给 Car 对象的方法这一事实非常令人困惑。为什么不把代码改成这样：

public class Car : ICar
{
    public bool EngineLight { get; set; }

    public void GetOilChange()
    {
    }

    public bool CheckEngineLight()
    {
        if (EngineLight)
        {
            GetOilChange();
            return true;
        }

        return false;
    } 
}

将您的测试更改为：

[Test]
    public void WhenEngineLightIsOnGetOilChange()
    {
        var carMock = MockRepository.GenerateMock<ICar>();
        carMock.Stub(x => x.EngineLight).Return(true);

        Assert.AreEqual(true, carMock.CheckEngineLight()); 

        carMock.AssertWasCalled(x => x.GetOilChange()); 
    }
 }

Answer 3

您的测试有问题，仅此而已。改变

GetOilChange(); // calls the car instance, not the mock, your
                // AssertMethodCalled will never be true

到

car.GetOilChange(); // calls the mock you passed in

顺便说一句，这对 Rhino 来说可能更惯用：

public interface ICar
{
    bool EngineLight { get; set; }
    void GetOilChange();
    bool CheckEngineLight();
}

public class Car : ICar
{
    public bool EngineLight { get; set; }

    public virtual void GetOilChange()
    {
    }

    public virtual bool CheckEngineLight()
    {
        if (EngineLight)
        {
            GetOilChange();
            return true;
        }

        return false;
    }
}

[TestFixture]
public class CarTests
{
    [Test]
    public void WhenEngineLightIsOnGetOilChange()
    {
        MockRepository mocks = new MockRepository();

        Car car;
        using (mocks.Record())
        {
            car = mocks.PartialMock<Car>();
            car.EngineLight = true;

            car.Expect(x => x.GetOilChange())
                .Repeat.Once()
                .Message("Should have called GetOilChange");

        }

        using (mocks.Playback())
        {
            var res = car.CheckEngineLight();
            Assert.IsTrue(res);
        }
    }
}