3

У меня есть геттеры и сеттеры, которые используются только для юнит-тестов.
Надо ли их оформлять в интерфейсах или можно просто в тестах обращаться не через интерфейс, а напрямую?

6
  • 7
    Как по мне, весьма странно изменять контракт интерфейса из-за тестов. Если в самой программе вам геттеры и сеттеры точно не нужны, то их не должно быть ни в интерфейсе, ни в тестах.
    – Regent
    Commented 5 янв. 2017 в 8:33
  • 2
    геттеров и сеттеров должно быть почти 0 (исключение "глупые" объекты - DTO, VO). добавлять их ради тестирования не рекомендуется. тестируйте поведение объектов, а не состояние. Commented 5 янв. 2017 в 9:34
  • >>тестируйте поведение объектов, а не состояние<<< поведение приводит к изменению состояния вот я и проверяю состояние как результат поведения, тем самым проверяя правильность поведения. Не понял вас, извините не могли бы чуть подробнее объяснить? >>>"глупые" объекты - DTO, VO<<< а как расшифровываются аббревиатуры DTO и VO?
    – Pavel
    Commented 5 янв. 2017 в 16:03
  • Приведите, пожалуйста, пример кода (тестируемого и самого теста).
    – andreycha
    Commented 2 февр. 2017 в 9:36
  • Если ваш класс знает, что его кто-то юнит-тестирует - вы что-то делаете не так
    – vp_arth
    Commented 7 мар. 2017 в 18:49

1 ответ 1

3

Судя по всему у вас не верный подход к юнит-тестированию. Как сказал Mikhail Vaysman в своем комментарии:

тестируйте поведение объектов, а не состояние

Ваш класс служит какой-то цели и имеет набор публичных методов, а значит и нужно тестировать именно поведение этих методов, а не внутреннюю реализацию класса. По большому счету, человек пишущий тесты может вообще не знать о том как реализован класс, ему важно только иметь описание публичных членов класса и оперировать ими при написании теста.

Для примера представим, что у вас есть самописный класс дублирующий StringBuilder. Допустим внутри класса есть приватное поле типа char[] хранящее введенную строку посимвольно.

Вы хотите протестировать метод void append(String s), который должен добавить в конец хранимой на текущей момент строки переданную в качестве аргумента метода строку. Суть тестирования в данном случае в том, что объект класса при вызове метода toString() должен вернуть строку того, что было в объекте плюс то, что вы добавили тестируемым методом append. То есть правильный тест должен оперировать публичными методами класса.

Допустим вы пошли другим путем и каким-то образом достучались до приватных членов класса через рефлексию или еще как-нибудь. Теперь вы тестируете метод append анализируя как изменяется внутреннее хранилище объекта. С виду ничего не изменилось. Если кто-то полез в реализацию метода append и метод стал работать неверно, то ваш тест завалится, как вам и требуется. Но допустим вы сами или кто-то еще решил изменить реализацию внутреннего хранилища строки. Может он переименовал поле, может теперь таких полей несколько, а может он вообще перенес хранилище в облако и теперь оно хранится где-то на других серверах. Не важно. Важно то, что при этом метод toString после вызова append возвращает именно то, что ожидается. Но при этом ваш тест завалится. То есть все работает правильно, а тест завалился! Поэтому, при написании теста, нужно оперировать только публичными методами.

Ваш ответ

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

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