Подтвердить что ты не робот

Rhino Mocks - AssertWasCalled: как улучшить нечеткое диагностическое сообщение при неправильных аргументах

IMHO, Rhino Mocks создает нечеткое диагностическое сообщение, когда AssertWasCalled используется для проверки того, что метод был вызван с определенным аргументом.

Пример:

interface ISomeInterface
{
    void Write(string s);
}

[TestFixture]
public class SomeTests
{
    [Test]
    public void WriteShouldBeCalledWithCorrectArguments()
    {
        // Arrange
        var mock = MockRepository.GenerateMock<ISomeInterface>();
        var sut = new SomeClass(mock);

        // Act
        sut.DoSomething();

        // Assert
        mock.AssertWasCalled(x => x.Write(Arg<string>.Is.Equal("hello")));
    }
}

Теперь, если тест завершился с этим сообщением...

Rhino.Mocks.Exceptions.ExpectationViolationException: ISomeInterface.Write(равно hello); Ожидаемое # 1, фактическое # 0.

... вы не можете знать, не сработает ли он, потому что

а. "Запись" никогда не вызывается - или -
B. 'Write' на самом деле вызывается, но с неправильным аргументом

Если бы B был причиной сбоя, тогда было бы намного яснее, если бы сообщение прочитало что-то вроде этого:

Rhino.Mocks.Exceptions.ExpectationViolationException: ISomeInterface.Write(string arg): был вызван метод, но с неправильными аргументами: Ожидаемое: привет, Actual: bye

Могу ли я исправить этот недостаток самостоятельно (путем написания пользовательских совпадений для Rhino в некотором роде), или просто мне нужно написать макет руководства для этого?

4b9b3361

Ответ 1

Я нашел простое решение, используя синтаксис "Матчи", предоставленный Rhino:

[Test]
public void WriteShouldBeCalledWithCorrectArguments()
{
    // Arrange
    var mock = MockRepository.GenerateMock<ISomeInterface>();
    var sut = new SomeClass(mock);

    // Act
    sut.DoSomething();

    // Assert
    mock.AssertWasCalled(x => x.Write(Arg<string>.Matches(s => Equal(s, "hello"))));
}

private static bool Equal(string s1, string s2)
{
    Assert.That(s1, Is.EqualTo(s2), "Unexpected argument");
    return true;
}

Конечно, это немного неуклюже, но он выполняет свою работу. Если есть лучший способ сделать это, пожалуйста, дайте мне знать.