Форум программистов, компьютерный форум, киберфорум
C# для начинающих
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.50/6: Рейтинг темы: голосов - 6, средняя оценка - 4.50
1 / 1 / 0
Регистрация: 23.04.2021
Сообщений: 127

Тестирование MOCK

09.07.2022, 00:14. Показов 1212. Ответов 7
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
Есть интерфейс по "постройке объекта". В данном случае как пример это ручка.
C#
1
2
3
4
5
6
7
8
public interface IPenBuilder
    {
        IPenBuilder Create(); 
        IPenBuilder SetColor(string color);
        IPenBuilder SetPrice(double price);
        IPenBuilder SetBrand(Brand brand);
        Pen Build();
    }
Её реализация

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
public class PenBuilder : IPenBuilder
    {
        private Pen _pen;
 
        public IPenBuilder Create()
        {
            _pen = new Pen();
            return this;
        }
        public IPenBuilder SetBrand(Brand brand)
        {
            _pen.Brand = brand;
            return this;
        }
 
        public IPenBuilder SetColor(string color)
        {
            _pen.Color = color;
            return this;
        }
 
        public IPenBuilder SetPrice(double price)
        {
            _pen.Price = price;
            return this;
        }
 
        public Pen Build()
        {
            return _pen;
        }
    }
Есть метод который делает эту ручку и далее записывает в бд и тд.
C#
1
2
3
4
5
6
7
8
9
10
public void PostTodoItem(Pen pen)
        {
            var newPen = _penBuilder
               .Create()
               .SetBrand(new Brand(pen.Brand.Name))
               .SetColor(pen.Color)
               .SetPrice(pen.Price)
               .Build();
            // запись в БД и тд.
        }
Пишу тесты к нему такие: чтобы проверял вызванные методы в PostTodoItem (такое задание)
C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
// Arrange
            private Mock<IPenBuilder> _penBuilder;
            private ManagementApiController _target;
            private readonly Fixture _fixture = new Fixture();
         //в TestInitialize
            _penBuilder = new Mock<IPenBuilder>(MockBehavior.Strict);
            _target = new ManagementApiController(_penBuilder.Object);
         // далее проверка метода
            var pen = _fixture.Create<Pen>();//заполнение рандомными данными
            _penBuilder.Setup(z => z.Create()).Returns(_penBuilder.Object);
            _penBuilder.Setup(z => z.SetBrand(pen.Brand)).Returns(_penBuilder.Object);
            _penBuilder.Setup(z => z.SetBrand(new Brand(pen.Brand.Name))).Returns(_penBuilder.Object);
            _penBuilder.Setup(z => z.SetColor(pen.Color)).Returns(_penBuilder.Object);
            _penBuilder.Setup(z => z.SetPrice(pen.Price)).Returns(_penBuilder.Object);
            _penBuilder.Setup(z => z.Build()).Returns(pen);
// Act
            _target.PostTodoItem(pen);
// Assert
            _penBuilder.Verify(z => z.Create(), Times.Once);
            _penBuilder.Verify(z => z.SetBrand(pen.Brand), Times.Once);
            _penBuilder.Verify(z => z.SetBrand(new Brand(pen.Brand.Name)), Times.Once);
            _penBuilder.Verify(z => z.SetColor(pen.Color), Times.Once);
            _penBuilder.Verify(z => z.SetPrice(pen.Price), Times.Once);
            _penBuilder.Verify(z => z.Build(), Times.Once);
Он выдает ошибку:
Moq.MockException: IPenBuilder.SetBrand(Brand) invocation failed with mock behavior Strict.
All invocations on the mock must have a corresponding setup.
Почему? Условие что MockBehavior.Strict должен быть!
0
Лучшие ответы (1)
Programming
Эксперт
39485 / 9562 / 3019
Регистрация: 12.04.2006
Сообщений: 41,671
Блог
09.07.2022, 00:14
Ответы с готовыми решениями:

Нагрузочное тестирование и стрессовое тестирование
Разработать компилятор простых арифметических выражений, например 2+ +(-5)*(7-8). Разработать тестовый сценарий нагрузочного...

Invalid setup on a non-virtual overridable in vb member: mock => mock[It.IsAny<string>()]
Вот такой код: public interface IGate { string Name { get; } } public interface ICommutator { ...

Mock тесты
public void GenericRepository_Add() { _mockContext.Setup(x =&gt; x.Add(_pen)); ...

7
 Аватар для IamRain
4694 / 2702 / 734
Регистрация: 02.08.2011
Сообщений: 7,228
09.07.2022, 11:30
Цитата Сообщение от Kurbanov Посмотреть сообщение
Есть метод который делает эту ручку и далее записывает в бд и тд.
Зачем в методе PostTodoItem создавать новую ручку, если на вход подается уже существующая в виде параметра?
Ok, если это просто пример использования Builder-а, просто выглядит не очень.

Цитата Сообщение от Kurbanov Посмотреть сообщение
IPenBuilder Create();
Метода Create не должно быть в Builder-е, так как Builder, по определению, строит объект по частям, а фабричный метод просто создает объект.
Объект Pen должен создаваться только после вызова метода Build.
Вы же смешали паттерны AbstractFactory (на фабричных методах) и Builder в кучу.
Сделайте, например, так:
Sample

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
public interface IPenBuilder
{
    IPenBuilder SetColor(string color);
    IPenBuilder SetPrice(decimal price);
    IPenBuilder SetBrand(Brand brand);
    Pen Build();
}
 
 
public class PenBuilder : IPenBuilder
{
    private List<Action<Pen>> buildPipeline = new();
 
    public IPenBuilder SetColor(string color)
    {
        if (string.IsNullOrEmpty(color))
            throw new ArgumentNullException(nameof(color));
 
        buildPipeline.Add(p => p.Color = color);
        return this;
    }
 
    public IPenBuilder SetPrice(decimal price)
    {
        if (price <= 0) throw new ArgumentOutOfRangeException(nameof(price));
 
        buildPipeline.Add(p => p.Price = price);
        return this;
    }
 
    public IPenBuilder SetBrand(Brand brand)
    {
        if (string.IsNullOrEmpty(brand?.Name))
            throw new ArgumentNullException(nameof(brand));
 
        buildPipeline.Add(p => p.Brand = brand);
        return this;
    }
 
    public Pen Build()
    {
        var pen = new Pen();
        foreach (var buildPart in buildPipeline)
            buildPart(pen);
        return pen;
    }
}



По сабжу:
Вы тестируете ваш IPenBuilder - значит Mock-ать его не надо, надо проверять работу реального builder-а.
Mock тут вообще не нужен.
0
1 / 1 / 0
Регистрация: 23.04.2021
Сообщений: 127
09.07.2022, 13:34  [ТС]
Я тестирую контроллер (target). А мокаю интерфейсы которые передаются в контроллер соответственно
Поэтому вопрос "почему пишет, что не все методы сетаплены" открытый.
0
 Аватар для IamRain
4694 / 2702 / 734
Регистрация: 02.08.2011
Сообщений: 7,228
09.07.2022, 17:53
Цитата Сообщение от Kurbanov Посмотреть сообщение
Поэтому вопрос "почему пишет, что не все методы сетаплены" открытый.
Попробуйте убрать второй Setup на метод SetBrand. Зачем устанавливать Setup два раза?

Цитата Сообщение от Kurbanov Посмотреть сообщение
Я тестирую контроллер (target).
Значит вы должны проверить результат работы контроллера, то есть убедиться в том, что все то, о чем вы упомянули в комментарии:
C#
1
// запись в БД и тд.
было произведено корректно.
Что запись оказалась в БД (используя, например, InMemoryDatabase, если работа с БД идет через EF) и т.д.


Цитата Сообщение от Kurbanov Посмотреть сообщение
Пишу тесты к нему такие: чтобы проверял вызванные методы в PostTodoItem (такое задание)
Какое-то мутное задание, либо вы его неправильно поняли.
Обычно проверяют какой-то специфический кусок логики, который приводит к изменению состояния, и это состояние уже Assert-ят в конце.
А те объекты, которые участвуют в работе метода, и которые достаточно сложно получить как реальную зависимость (например объекты, делающие запросы на сторонний сервис, или запросы в БД) - их уже Mock-ают (если речь идет о Unit-тестах). И эти Mock-и не валидируют.
Вот ответьте на вопросы:
1. Зачем проверять работу вами же созданного Mock-а в тестах?
2. В чем назначение Mock-а?
3. Зачем устанавливать Setup два раза?
0
1 / 1 / 0
Регистрация: 23.04.2021
Сообщений: 127
09.07.2022, 18:40  [ТС]
На первые два вопроса отвечаю - это задание. Я также тестировал другие методы например добавление в базу и удаление. А теперь я тестирую интерфейс IPenBuilder.
Отвечаю на 3ий вопрос - хоть сетаплю 1 раз выходит эта же ошибка. Поэтому я хочу понять, что еще "незасетаплено". Ведь ошибка в том, что не все методы "засетапили", а не "пересетапили"
0
 Аватар для IamRain
4694 / 2702 / 734
Регистрация: 02.08.2011
Сообщений: 7,228
09.07.2022, 19:24
Цитата Сообщение от Kurbanov Посмотреть сообщение
На первые два вопроса отвечаю - это задание.
Ок, повторюсь - мутное задание.

Попробуйте Mock-ать другим способом:
C#
1
_penBuilder.Setup(z => z.SetBrand(It.IsAny<Brand>())).Returns(_penBuilder.Object);
То есть не указывая конкретных параметров, возможно, в этом и есть проблема.
0
1 / 1 / 0
Регистрация: 23.04.2021
Сообщений: 127
09.07.2022, 21:56  [ТС]
Попробовал но пишет, что метод SetBrands не вызывается. Я убрал этот метод из контроллера и из теста, тогда все успешно проходит. Но почему то он зацепился за SetBrand.

Модели выглядят так со связью 1 ко многим
C#
1
2
3
4
5
6
7
8
public class Pen
    {
        public int Id { get; set; }
        public double Price { get; set; }
        public string Color { get; set; }
        public int BrandId { get; set; }
        public Brand Brand { get; set; }
    }
C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 public class Brand
    {
        public int Id { get; set; }
        public string Name { get; set; }
        public ICollection<Pen> Pens { get; set; }
 
        public Brand(string name)
        {
            Name = name;
        }
        public Brand()
        {
            
        }
    }
Сразу скажу конструкторы нужны. Видимо из за них?
0
 Аватар для IamRain
4694 / 2702 / 734
Регистрация: 02.08.2011
Сообщений: 7,228
09.07.2022, 23:34
Лучший ответ Сообщение было отмечено Kurbanov как решение

Решение

Kurbanov, по всей видимости - это детали работы Mock-а, в определении того, был вызван метод или нет, используется отслеживание ссылок на параметры.
Если создавать на лету новый объект Brand-а, то ничего не работает:
C#
1
penBuilder.Verify(z => z.SetBrand(new Brand() { Name = pen.Brand.Name}), Times.Once);
=>
Code
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
Moq.MockException : 
Expected invocation on the mock once, but was 0 times: z => z.SetBrand(Brand)
 
Performed invocations:
 
   Mock<IPenBuilder:1> (z):
 
      IPenBuilder.Create()  => Mock<IPenBuilder:1>
      IPenBuilder.SetBrand(Brand)  => Mock<IPenBuilder:1>
      IPenBuilder.SetColor("Color8a35941a-b220-45f3-8f9c-3ce057f26303")  => Mock<IPenBuilder:1>
      IPenBuilder.SetPrice(239)  => Mock<IPenBuilder:1>
      IPenBuilder.Build()
 
   at Moq.Mock.Verify(Mock mock, LambdaExpression expression, Times times, String failMessage) in C:\projects\moq4\src\Moq\Mock.cs:line 330
   at Moq.Mock`1.Verify[TResult](Expression`1 expression, Func`1 times) in C:\projects\moq4\src\Moq\Mock`1.cs:line 840
   at Mocking.Tests.PenBuilderTests.PenBuilder_ShouldWork() in D:\Projects\Rider\Mocking\Mocking\Tests\PenBuilderTests.cs:line 32
Если глянуть на список Performed invocations, то видно что Mock сам себе противоречит, => в списке вызовов указан метод SetBrand. Но далее уже вываливается MockException, мол не был вызван ни разу.
Видимо, недоработка/несовершенство Mock-а.

Для такого кода:
C#
1
penBuilder.Verify(z => z.SetBrand(pen.Brand), Times.Once);
Все работает.

Добавлено через 3 минуты
И builder у вас криво написан.

Но это все тесты ради тестов, все это мало связано с реальными enterprise-проектами.

Добавлено через 18 минут
Цитата Сообщение от IamRain Посмотреть сообщение
Если создавать на лету новый объект Brand-а, то ничего не работает:
Я не совсем точно выразился - с Mock тут все в порядке, метод так и называется - Verify.
То есть мы проверяем, был ли вызван метод с конкретным набором параметров.
Поскольку на лету создается новый параметр:
C#
1
penBuilder.Verify(z => z.SetBrand(new Brand() { Name = pen.Brand.Name}), Times.Once);
То и это вбросит исключение, так как конкретно данный параметр и не использовался при вызове. По всей видимости, это by design в Mock.
1
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
inter-admin
Эксперт
29715 / 6470 / 2152
Регистрация: 06.03.2009
Сообщений: 28,500
Блог
09.07.2022, 23:34
Помогаю со студенческими работами здесь

Mock контекста при юнит тестировании
Проект ASP под .Net Framework Есть класс теста, тестирует методы HomeController : BaseController HomeController использует поле...

Mock тестирование
jmock 2.6.0 (Java 6) это последняя библиотека которую я нашел на их офф.сайте. Я использую Java 8, поэтому у меня практически весь...

Юнит тестирование С++ (Google Mock)
Добрый день. Я только начинаю знакомиться с Юнит тестированием и фреймворком Google. может ли мне кто-нибудь подсказать, почему данный тест...

No Last call on a Mock available
Столкнулся с такой проблемой при тестировании сервиса. @Test public void testExecuteCommand() { IMocksControl control =...

Mock и Stub
Как пользоваться методами mock и stub для модульного тестирования в JUnit?


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
8
Ответ Создать тему
Новые блоги и статьи
Воспроизведение звукового файла с помощью SDL3_mixer при касании экрана Android
8Observer8 26.01.2026
Содержание блога SDL3_mixer - это библиотека я для воспроизведения аудио. В отличие от инструкции по добавлению текста код по проигрыванию звука уже содержится в шаблоне примера. Нужно только. . .
Установка Android SDK, NDK, JDK, CMake и т.д.
8Observer8 25.01.2026
Содержание блога Перейдите по ссылке: https:/ / developer. android. com/ studio и в самом низу страницы кликните по архиву "commandlinetools-win-xxxxxx_latest. zip" Извлеките архив и вы увидите. . .
Вывод текста со шрифтом TTF на Android с помощью библиотеки SDL3_ttf
8Observer8 25.01.2026
Содержание блога Если у вас не установлены Android SDK, NDK, JDK, и т. д. то сделайте это по следующей инструкции: Установка Android SDK, NDK, JDK, CMake и т. д. Сборка примера Скачайте. . .
Использование SDL3-callbacks вместо функции main() на Android, Desktop и WebAssembly
8Observer8 24.01.2026
Содержание блога Если вы откроете примеры для начинающих на официальном репозитории SDL3 в папке: examples, то вы увидите, что все примеры используют следующие четыре обязательные функции, а. . .
моя боль
iceja 24.01.2026
Выложила интерполяцию кубическими сплайнами www. iceja. net REST сервисы временно не работают, только через Web. Написала за 56 рабочих часов этот сайт с нуля. При помощи perplexity. ai PRO , при. . .
Модель сукцессии микоризы
anaschu 24.01.2026
Решили писать научную статью с неким РОманом
http://iceja.net/ математические сервисы
iceja 20.01.2026
Обновила свой сайт http:/ / iceja. net/ , приделала Fast Fourier Transform экстраполяцию сигналов. Однако предсказывает далеко не каждый сигнал (см ограничения http:/ / iceja. net/ fourier/ docs ). Также. . .
http://iceja.net/ сервер решения полиномов
iceja 18.01.2026
Выкатила http:/ / iceja. net/ сервер решения полиномов (находит действительные корни полиномов методом Штурма). На сайте документация по API, но скажу прямо VPS слабенький и 200 000 полиномов. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru