Правильно ли я делаю модульные тесты?

@kuznets · Регистрация: 22.11.2013

Author24 — интернет-сервис помощи студентам

Привет, начал смотреть как делать модульные тесты, мой вопрос не о том как пофиксить что то, а просьба оценить написанное и дать рекомендации.

Есть класс MessageController ( ссылка на гитхаб )

Java

@RestController
public class MessageController {
 
    @Autowired
    private MessageService messageService;
 
    @RequestMapping(
            value = "/api/messages",
            method = RequestMethod.GET,
            produces = MediaType.APPLICATION_JSON_VALUE)
    public List<Message> getAllMessages(){
        return messageService.findAll();
    }
 
    @RequestMapping(
            value = "/api/message/{id}",
            method = RequestMethod.GET,
            produces = MediaType.APPLICATION_JSON_VALUE)
    public ResponseEntity<Message> getMessage(@PathVariable Long id){
        return new ResponseEntity<>(messageService.findOne(id), HttpStatus.OK);
    }
 
    @RequestMapping(
            value = "/api/message",
            method = RequestMethod.POST,
            consumes = MediaType.APPLICATION_JSON_VALUE,
            produces = MediaType.APPLICATION_JSON_VALUE)
    public ResponseEntity<?> addMessage(@RequestBody Message message) {
        return new ResponseEntity<>(messageService.save(message), HttpStatus.CREATED);
    }

И сами тесты

Java

@RunWith(MockitoJUnitRunner.class)
public class MessageControllerTest {
 
    private final long ID = 1L;
    private final Date CREATED = new Date();
    private final Message  MESSAGE = new Message(ID, "text mesage", "author name", CREATED);
    private final List<Message> MESSAGE_LIST = new ArrayList<>();
 
    @Mock
    private MessageService messageService;
    @InjectMocks
    MessageController messageController;
 
    @Test
    public void getAllMessages() throws Exception {
        //prepare
        when(messageService.findAll()).thenReturn(ImmutableList.of());
        //testing
        List<Message> messages = messageController.getAllMessages();
        //validate
        verify(messageService).findAll();
    }
 
    @Test
    public void getMessage() throws Exception {
        //prepare
        when(messageService.findOne(ID)).thenReturn(MESSAGE);
        //testing
        ResponseEntity<Message> testMessage = messageController.getMessage(ID);
        //validate
        verify(messageService).findOne(ID);
    }
 
    @Test
    public void addMessage() throws Exception {
        //prepare
        when(messageService.save(MESSAGE)).thenReturn(MESSAGE);
        //testing
        ResponseEntity<Message> testMessage = (ResponseEntity<Message>) messageController.addMessage(MESSAGE);
        //validate
        verify(messageService).save(MESSAGE);
    }
 
    @Test
    public void addMessages() throws Exception {
        //prepare
        MESSAGE_LIST.add(MESSAGE);
        when(messageService.saveArray(MESSAGE_LIST)).thenReturn(MESSAGE_LIST);
        //testing
        ResponseEntity<Object> testMessages = (ResponseEntity<Object>) messageController.addMessages(MESSAGE_LIST);
        //validate
        verify(messageService).saveArray(MESSAGE_LIST);
    }

Заранее спасибо.

@Fene4ka_ · 12.12.2016, 01:03

kuznets, также стоит проверять на кол-во вызовов методы и проверять на возникновение исключительных ситуаций

@kuznets · 19.12.2016, 00:06 **[ТС]**

Fene4ka_,
Спасибо за наводку, сделал тест на колличество вызовов

Java

@Test
    public void InvocationCount() throws Exception {
        //prepare
        MESSAGE_LIST.add(MESSAGE);
 
        // call methods in mock
        messageController.getAllMessages();
        messageController.getAllMessages();
 
        messageController.getMessage(ID);
        messageController.getMessage(ID);
 
        messageController.addMessage(MESSAGE);
        messageController.addMessage(MESSAGE);
        messageController.addMessage(MESSAGE);
 
        messageController.addMessages(MESSAGE_LIST);
        messageController.addMessages(MESSAGE_LIST);
 
        messageController.updateMessage(ID, MESSAGE);
        messageController.updateMessage(ID, MESSAGE);
        messageController.updateMessage(ID, MESSAGE);
        
//        messageController.deleteMessage(ID);
 
        // verify method calls
        verify(messageService, times(2)).findAll();
        verify(messageService, times(2)).findOne(ID);
        verify(messageService, times(3)).save(MESSAGE);
        verify(messageService, times(2)).saveArray(MESSAGE_LIST);
        verify(messageService, times(3)).update(ID, MESSAGE);
//        verify(messageController, times(1)).deleteMessage(ID);
    }

Все работает кроме deleteMessage на него ругается, что не может найти такого по ИД, как его проверять?

@Fene4ka_ · 19.12.2016, 08:03

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

@kuznets · 22.12.2016, 22:11 **[ТС]**

Есть у меня метод

Java

        @Override
    public boolean delete(Long id) {
        Message message = messageRepository.findOne(id);
        if (message == null) {
            throw  new DataNotFoundException("Data with id=" + id + " not found.");
        }
        messageRepository.delete(id);
        return true;
    }

и тест к нему

Java

        public void delete() throws Exception {
        //prepare
        when(messageService.delete(ID)).thenReturn(true);
        //testing
        boolean testMessage = messageService.delete(ID);
        //validate
        verify(messageService).delete(ID);

И получаю ошибку java.lang.Exception: No tests found matching Method delete
не понимаю в чем дело, есть же такой метод.

@Fene4ka_ 102 / 102 / 40 Регистрация: 24.01.2014 Сообщений: 1,242
	12.12.2016, 01:03	2
	kuznets, также стоит проверять на кол-во вызовов методы и проверять на возникновение исключительных ситуаций 0

@Fene4ka_ 102 / 102 / 40 Регистрация: 24.01.2014 Сообщений: 1,242
	19.12.2016, 08:03	4
	возможно у вас как раз какая-то проблема в теле метода ну или вы что-то забыли "замокать". Собственно не видя картины целиком трудно предсказать) 0

	22.12.2016, 22:11