NSubstitute, используя FOrPartsOf, чтобы настроить метод void, чтобы ничего не делать - PullRequest
Новая
       52

NSubstitute, используя FOrPartsOf, чтобы настроить метод void, чтобы ничего не делать

0 голосов
/ 22 марта 2020

У меня есть простой пример, в котором я хотел бы проверить, вызывается ли метод в том же классе, что и вызывающий метод: 

    public class MyClass
    {
        public void SomeMethod()
        {
            SomeSubMethod();
        }

        public virtual void SomeSubMethod()
        {
            // do a lot of weird stuff
        }

    }

    public class UnitTest1
    {
        [Fact]
        public void Test1()
        {
            var target = Substitute.ForPartsOf<MyClass>();
            target.Configure().SomeSubMethod(); // <---- please just do nothing

            target.SomeMethod();

            target.Received(1).SomeSubMethod();
        }
    }

Моя проблема в том, что SomeSubMethod фактически вызывается в модульное тестирование, и в моем реальном коде я бы хотел этого избежать.

Простой обходной путь - позволить SomeSubMethod вернуть что-то, но теперь я загрязняю свой реальный код 

    public class MyClass
    {
        public void SomeMethod()
        {
            SomeSubMethod();
        }

        public virtual int SomeSubMethod()
        {
            // do a lot of weird stuff
            return 0;
        }

    }

    public class UnitTest1
    {
        [Fact]
        public void Test1()
        {
            var target = Substitute.ForPartsOf<MyClass>();
            target.Configure().SomeSubMethod().Returns(0); // <--- Now the real SomeSubMethod won't be invoked

            target.SomeMethod();

            target.Received(1).SomeSubMethod();
        }
    }

Есть ли способ настроить метод void, чтобы он ничего не делал?

Ваш / Петр

Ответы [ 2 ]

2 голосов
/ 23 марта 2020

Вы можете использовать синтаксис When..Do для работы с void методами: 

public void Test1() {
    var target = Substitute.ForPartsOf<MyClass>();
    target.When(x => x.SomeSubMethod()).DoNotCallBase(); // <- do not invoke real code

    target.SomeMethod();

    target.Received(1).SomeSubMethod();
}

Другой вариант, предполагая, что мы делаем SomeMethod virtual, состоит в замене всех членов, использующих стандартный Substitute.For<T> и разрешить вызов только для того метода, который вы хотите проверить. 

public void Test2() {
    var target = Substitute.For<MyClass>(); // <- substitute all members
    target.When(x => x.SomeMethod()).CallBase(); // <- except this one, call the real base implementation for SomeMethod 

    target.SomeMethod();
    target.Received(1).SomeSubMethod();

    Assert.Equal(0, target.counter);
}

Ссылки на документацию:

1 голос
/ 22 марта 2020

Согласно вашему описанию, я переписываю код: 

Public class MyClass
{
    public int x = 0;   // to verify whether the unit test really execute SomeSubMethod()

    public void SomeMethod()
    {
        SomeSubMethod();
    }

    public virtual void SomeSubMethod()
    {
        // do a lot of weird stuff
        int x = 10; 
    }
}

И тест проводится (обратите внимание, что я использую NUnit framework): 

    [TestFixture]
    public class UnitTest1
    {
        [Test]
        public void Test2()
        {
            var myClass = Substitute.For<MyClass>();
            myClass.SomeMethod();
            myClass.Received(1).SomeSubMethod();
            Assert.That(myClass.x, Is.EqualTo(0));
        }
    }

Поскольку целевой метод теста SomeSubMethod () - это виртуальный метод, вам не нужно макетировать часть класса MyClass.

Это то, что вы хотите?

...