0

Тестируемый метод

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;
}

Тест:

@Test
public void delete() throws Exception {
    //prepare
    MessageServiceImp spy = spy(new MessageServiceImp(messageRepository));
    messageServiceImp.save(MESSAGE); //создаю объект, пробовал и не создавать результат одинаков
    doReturn(true).when(spy).delete(ID);
    //testing
    boolean testMessage = messageServiceImp.delete(ID);
    //validate
    verify(messageRepository).delete(ID);
}

Тест не проходит получаю:

.exception.DataNotFoundException: Data with id=1 not found.

Что делаю не так?


Добавление MessageRepository

public interface MessageRepository extends JpaRepository<Message, Long>{
}
2
  • а что вы пытаетесь протестировать? опишите словами ваш тест. 2 янв 2017 в 7:45
  • Я хочу проверить, что метод delete получив id выполнит удаление и вернет true.
    – Sergei R
    2 янв 2017 в 11:25

1 ответ 1

2

Вот мой вариант.

Тестируемый метод

public void delete(Long id) {
    // так делать плохо
    // добавлено по просьбе вопрошающего
    try {
        // проверку на существование и пр. должен делать MessageRepository
        messageRepository.delete(id);
    } catch(EmptyResultDataAccessException ex) {
        throw new DataNotFoundException("Data with id=" + id + " not found.");
    }
}

Тест

private static final Long ID = 123;

// интерфейс, а не реализация
private MessageRepository messageRepository;

@Test
public void detele_delegateToRepository() throws Exception {
    // подготовка
    messageRepository = mock(MessageRepository.class);
    // не знаю как у вас класс называется - будем считать Subject
    Subject subject = new Subject(messageRepository);

    // действие
    subject.delete(ID);

    // проверка
    verify(messageRepository).delete(ID);
}

@Test(expected = DataNotFoundException.class)
public void detele_throwsException() throws Exception {
    // подготовка
    messageRepository = mock(MessageRepository.class);
    // не знаю как у вас класс называется - будем считать Subject
    Subject subject = new Subject(messageRepository);

    when(messageRepository.delete(ID)).thenThrow(new EmptyResultDataAccessException());

    // действие
    subject.delete(ID);
}

Это все что вам надо тестировать. Остальные тесты должны быть у класса который реализует MessageRepository.

13
  • boolean и return messageRepository.delete(id); == Incompatible types.
    – Sergei R
    4 янв 2017 в 17:47
  • покажите ваш MessageRepository 4 янв 2017 в 17:49
  • Добавил выше в тело вопроса.
    – Sergei R
    4 янв 2017 в 17:53
  • @SergeiR исправил код 4 янв 2017 в 18:06
  • Ну а проверка если ли такой id? У меня она сдесь же до вызова messageRepository.delete(id);
    – Sergei R
    4 янв 2017 в 18:36

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими условиями использования и подтверждаете, что прочитали и поняли наши политику конфиденциальности и нормы поведения.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.