Как правильно использовать Rhino.Mocks AssertWasCalled ()? - PullRequest
18 голосов
/ 25 июня 2010

Я звоню _mocks.ReplayAll(), затем один или несколько _mockedObject.AssertWasCalled(), а затем _mocks.VerifyAll(). Но он говорит мне, что «Это действие недопустимо, когда макет объекта находится в состоянии записи».

[Test]
public void SetStateExecuting_Should_Set_State_To_Pause_And_Not_Change_GlobalState_When_GlobalState_Is_Paused()
{
    var task = new Task { ID = 1, TimeZone = -660, GlobalState = TaskState.Paused };
    _taskDataProvider.Expect(p => p.StateUpdate(task.ID, task.TimeZone, TaskState.Paused));
    _mockRepository.ReplayAll();
    _manager.SetStateExecuting(task);
    _taskDataProvider.AssertWasNotCalled(p => p.GlobalStateUpdate(task.ID, 
                                                                  TaskState.Executing));
    _mockRepository.VerifyAll();
}

Какой правильный порядок вызова для правильной работы этих методов?

Ответы [ 3 ]

17 голосов
/ 26 июня 2010

Сообщение в блоге Джона Крюгера "Как использовать насмешки носорога, документированные с помощью тестов" содержит простые примеры всего, что можно сделать с помощью методов носорога носорога Он также показывает, что вы не можете сделать, что я нашел очень полезным в обучении.

Как упоминалось ранее, использование синтаксиса Arrange, Act, Assert со статическими конструкторами легче для чтения. В блоге приведены примеры обоих методов.

Вот примеры из примера кода Джона:

Новый синтаксис:

 [Test]
    public void You_can_check_to_see_if_a_method_was_called()
    {
        var stub = MockRepository.GenerateStub<ISampleClass>();

        stub.MethodThatReturnsInteger("foo");

        stub.AssertWasCalled(s => s.MethodThatReturnsInteger("foo"));
        stub.AssertWasCalled(s => s.MethodThatReturnsInteger(Arg<string>.Is.Anything));
    }

Старый стиль:

    [Test]
    public void Calling_virtual_methods_will_call_the_actual_method()
    {
        var mockRepository = new MockRepository();
        var sampleClass = mockRepository.PartialMock<SampleClass>();
        sampleClass.Replay();

        sampleClass.VirtualMethod("foo").ShouldEqual(3);
        sampleClass.VirtualMethodWasCalled.ShouldBeTrue();
        sampleClass.AssertWasCalled(c => c.VirtualMethod("foo"));
    }
12 голосов
/ 25 июня 2010

Вы смешиваете старый шаблон записи / воспроизведения и новый AAA шаблон .

Обычный способ установить ожидание «не вызывается с этими аргументами» в режиме записи выглядит следующим образом:

_taskDataProvider.Expect(
    p => p.GlobalStateUpdate(task.ID, TaskState.Executing)).Repeat.Never();

Или же вы можете использовать строгий макет, который просто не допускает неожиданные вызовы.

AssertWasCalled и AssertWasNotCalled предназначены для AAA, где вы ставите свои утверждения в конце. С синтаксисом «Запись / Воспроизведение» поведение и ожидания должны быть установлены при запуске до перехода в режим воспроизведения.

(AssertWasNotCalled на самом деле может работать и с Record / Replay, но я никогда не пробовал, потому что не люблю смешивать элементы обоих подходов. Это без необходимости усложняет вещи.)

0 голосов
/ 25 июня 2010

Похоже, я нашел решение. Это кажется немного странным, но это работает. Оказывается, мне почему-то нужно дважды вызывать ReplayAll () ...

Это работает:

[Test]
public void SetStateExecuting_Should_Set_State_To_Pause_And_Not_Change_GlobalState_When_GlobalState_Is_Paused()
{
    var task = new Task { ID = 1, TimeZone = -660, GlobalState = TaskState.Paused };
    _mockRepository.ReplayAll();
    _manager.SetStateExecuting(task);
    _taskDataProvider.AssertWasCalled(p => p.StateUpdate(task.ID, task.TimeZone, TaskState.Paused));
    _taskDataProvider.AssertWasNotCalled(p => p.GlobalStateUpdate(task.ID, TaskState.Executing));
    _mockRepository.ReplayAll();
}
...