Как настроить moq для контроллера?

Question

[DD-var]

Решил написать несколько unit тестов для своих контроллеров, нашел либу moq. И не до конца разобрался как правильно с ней работать.
Есть сервис для добавление пользователей в бд. хотел бы узнать на какие ошибки будет реагировать и что будет отсылать. Не могу дописать unit test из-за того что не знаю что нужно возращать.

[Test]
        public void CheckAddUser()
        {
            SetingUsers();
            
            var moc = new Mock<IUserService>();
            moc.Setup(c => c.Create(It.IsAny<RegisterViewModel>()))
                .Returns<IdentityResult>("не знаю что тут возращать");
            

            var userController = new TestController(moc.Object);

            var result = userController.Create(newUser).Result;
        }

сам контроллер :

public async Task<IdentityResult> Create(RegisterViewModel user)
        {
            return await _userService.Create(user);
        }

Answer 1

[Илья]

Вы не до конца разобрались не в либе, а в себе)
Сначала нужно поставить вопрос, что тестируем, если вы на него ответите будете знать что подавать на вход и что проверять на выходе.

У вас написано следующее "при любой модели пользователя на входе проверь что пользователь создается"
Допустим что вы проверите в конце IdentityResult на успех. Из мока вы видимо захотите вернуть успешный IdentityResult. Но это бессмысленно. Потому что вы все замокали и не тестируете никакую логику приложения, а тестируете библиотеку с моками.

Comments

[DD-var]

Хорошо, а как тогда правильно сделать что б я получил с IdentityResult ответ, с ошибкой или с успехом?

[Илья]

DD-var, что вы тестируете, ответьте на этот вопрос? Пока у вас тест ради теста.

[DD-var]

Илья, хочу протестировать что вернет identity result, если я закину пустого юзера. я поставил на некоторые свойства модели [required(ErrorMessage="пустое поле"])

[Илья]

DD-var, давайте я вам еще раз скажу это.
Код метода контроллера у вас состоит из одной строчки
return await _userService.Create(user);
Метод Create вы замокали, то есть исключили его из тестирования. Не осталось методов которые можно тестировать. Если вы хотите проверить валидацию модели, то вам нужно дописать проверку ModelState.IsValid в свой контроллер. Если вы хотите проверить работу сервиса, то не надо его мокать. Мокайте хранилища которые использует сервис, чтобы логика работы самого сервиса сохранялась.

[DD-var]

Илья, Если я делаю ModelState.IsValid, в контроллере, то он уже возращает не IdentityResult a IActionResult, что более правильно будет.. но так же при проверке в контроллер, я должен передать буду параметр IUserSevice, который уже делаю через мок. я правильно понял?

[Илья]

DD-var, скорее всего нет.

[Илья]

Приведите код получившегося теста, тогда будет понятно.

[DD-var]

Илья, сейчас хочу проверить как работает валидация и пишу тест, сервис мокаю. и он должен проверить есть ли ошибки у модели которую я передаю. Передаю пустую модель
тест:

public void CheckAddUser()
        {
            SetingUsers();
            
            var moc = new Mock<IUserService>();
            moc.Setup(c => c.Create(It.IsAny<RegisterViewModel>()) );
            

            var userController = new TestController(moc.Object);

            var result = userController.Create(fakeUser);
//должен вернуть ошибку
           

        }

Контроллер

public async Task<ActionResult> Create(RegisterViewModel user)
        {
           // ModelState.AddModelError("swe","ssssss");
  
            if (ModelState.IsValid)
            {
                var result = await _userService.Create(user);
                return Ok(user);
            }

            else
            {
                return BadRequest(ModelState);
            }
        }

[Илья]

DD-var, ага, осталось только добавить

Assert.IsInstanceOfType(result, typeof(BadRequestObjectResult));

и получится тест

[DD-var]

Илья, не проходит тест, валидатор не выдает ошибки на пустые поля... из-за чего это может быть?

сама модель:

public class RegisterViewModel
    {
        //обязательные поля
        [Required(ErrorMessage = "Пустое поле")]
        public string FirstName { get; set; }


        [Required(ErrorMessage = "Пустое поле")]
        public string FullName { get; set; }
}

Объявление ее и добавление значений:

RegisterViewModel fakeUser = new RegisterViewModel();
            
            fakeUser.Password = null;
            fakeUser.UserName = null;
            fakeUser.Email = null;

[Илья]

А и правда, я как-то не учел этого.
Инфраструктура aspnet генерит валидатор, применяет его и добавляет ошибки в модель на этапе исполнения

[DD-var]

Илья, а как тогда сделать правильно?

[Илья]

Есть какие-то решения в интернете, по конструированию валидаторов, но я их не пробовал.

[DD-var]

Илья, все же, спасибо.

[Илья]

DD-var, на здоровье