Получение аргументов, передаваемых в FakeItEasy-mock без использования волшебных строк? - PullRequest
11 голосов
/ 20 августа 2011

Я использовал Moq для моих насмешливых нужд в последние годы, но после просмотра FakeItEasy я хотел попробовать.

Я часто хочучтобы проверить, что метод был вызван с правильными параметрами, но я не нашел удовлетворительного способа сделать это с FakeItEasy.

У меня есть следующий код для проверки:

    public class WizardStateEngine : IWizardStateEngine
{
    private readonly IWorkflowInvoker _workflowInvoker;
    private List<CustomBookmark> _history;

    public WizardStateEngine(IWorkflowInvoker workflowInvoker)
    {
        _workflowInvoker = workflowInvoker;
    }

    public void Initialize(List<CustomBookmark> history)
    {
        _history = history;
    }

    public WizardStateContext Execute(Command command, WizardStateContext stateContext, CustomBookmark step)
    {
        Activity workflow = new MyActivity();
        var input = new Dictionary<string, object>();
        input["Action"] = command;
        input["Context"] = stateContext;
        input["BookmarkHistory"] = _history;

        var output = _workflowInvoker.Invoke(workflow, input);

        _history = output["BookmarkHistory"] as List<CustomBookmark>;

        return output["Context"] as WizardStateContext;
    }

    public List<CustomBookmark> GetBookmarkHistory()
    {
        return _history;
    }
}

Я хочунаписать несколько тестов, которые проверяют ввод в _workflowInvoker.Invoke ().Мой метод TestInitialize устанавливает необходимые ресурсы и сохраняет входной словарь в _workflowInvoker.Invoke () как локальное поле _wfInput.

    [TestInitialize]
    public void TestInitialize()
    {
        _wizardStateContext = new WizardStateContext();
        _workflowInvoker = A.Fake<IWorkflowInvoker>();
        _wizardStateEngine = new WizardStateEngine(_workflowInvoker);

        _outputContext = new WizardStateContext();
        _outputHistory = new List<CustomBookmark>();
        _wfOutput = new Dictionary<string, object>
                        {{"Context", _outputContext}, {"BookmarkHistory", _outputHistory}};

        _history = new List<CustomBookmark>();

        A.CallTo(() =>
                 _workflowInvoker.Invoke(A<Activity>.Ignored, A<Dictionary<string, object>>.Ignored))
            .Invokes(x => _wfInput = x.Arguments.Get<Dictionary<string, object>>("input"))
            .Returns(_wfOutput);

        _wizardStateEngine.Initialize(_history);
    }

После настройки у меня есть несколько таких тестов:

    [TestMethod]
    public void Should_invoke_with_correct_command()
    {
        _wizardStateEngine.Execute(Command.Start, null, null);

        ((Command) _wfInput["Action"]).ShouldEqual(Command.Start);
    }

    [TestMethod]
    public void Should_invoke_with_correct_context()
    {
        _wizardStateEngine.Execute(Command.Start, _wizardStateContext, null);

        ((WizardStateContext) _wfInput["Context"]).ShouldEqual(_wizardStateContext);
    }

    [TestMethod]
    public void Should_invoke_with_correct_history()
    {
        _wizardStateEngine.Execute(Command.Start, _wizardStateContext, null);

        ((List<CustomBookmark>) _wfInput["BookmarkHistory"]).ShouldEqual(_history);
    }

Мне не нравится волшебная строка «input» в TestInitialize для получения переданного аргумента (или магического числа).Я могу написать тесты без локального поля, например:

    [TestMethod]
    public void Should_invoke_with_correct_context()
    {
        _wizardStateEngine.Execute(Command.Start, _wizardStateContext, null);

        A.CallTo(() =>
                 _workflowInvoker.Invoke(A<Activity>._,
                                         A<Dictionary<string, object>>.That.Matches(
                                             x => (WizardStateContext) x["Context"] == _wizardStateContext)))
            .MustHaveHappened();
    }

Но я считаю, что тесты с локальным полем более читабельны.

Есть ли способ настроить сохранение ввода какполе в моем тестовом классе без магических чисел или строк?

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

Ответы [ 2 ]

21 голосов
/ 20 августа 2011
A.CallTo(() => service.DoSomething(A<int>.That.Matches(x => x == 100)))
 .MustHaveHappened();
15 голосов
/ 20 августа 2011

Я согласен со всем, что говорит Дарин, кажется плохой практикой делать то, что ты делаешь.Вы говорите, что в этом тривиальном примере это выглядит «глупо», не могли бы вы привести пример, где он выглядит умным?1004 *

...