#143: added checkPasswordByLogin(User user, String password) in UserService

JujaLabs / itevents

Resource to subscribe on it events

Apache License 2.0

7 stars 5 forks source link

#143: added checkPasswordByLogin(User user, String password) in UserService #145

Closed IgorMaksymov closed 8 years ago

IgorMaksymov commented 8 years ago

connect to #143

vaa25 commented 8 years ago

Нет теста дао.

vaa25 commented 8 years ago

имя getUserPasswordByLogin подразумевает, что на вход подается логин а не юзер и не понятно в каком виде ожидается пароль. Рекомендую убрать из названия метода ByLogin и добавить Encoded
в том же тесте задействован реальный PasswordEncoder, хотя его тоже следовало бы замокать

IgorMaksymov commented 8 years ago

сделал

vaa25 commented 8 years ago

when(userService.matchPasswordByLogin(testUser, password)).thenReturn(true); boolean isCorrect = userService.matchPasswordByLogin(testUser, password); verify(userDao, atLeastOnce()).getEncodedUserPassword(testUser); assertTrue(isCorrect);

Теперь тест проверяет иcключительно Mockito

atLeastOnce()

указывает на неопределенность количества вызовов метода, а нам это число достоверно известно

IgorMaksymov commented 8 years ago

А как ты мне предлагаешь в мокеpasswordEncoder.match() проверить правильность пароля? или здесь надо только verify(userDao, atLeastOnce()).getEncodedUserPassword(testUser); ?

vaa25 commented 8 years ago

Замокать все, что не проверяемый метод

romach commented 8 years ago

Почему ты проверяешь, что getEncodedUserPassword(testUser) вызывается не больше двух раз? Он вызывается один раз.

IgorMaksymov commented 8 years ago

Действовал по старой памяти

vaa25 commented 8 years ago

PASSED

witjem commented 8 years ago

passed