Как тестировать private
и protected
методы, может есть какие-то стандартные средства, позволяющие сделать это?
В интернете видел примеры, что подобные методы вызываются через рефлексию, или у них тупо меняют модификатор доступа на public
на время теста.
5 ответов
У класса есть интерфейс - это совокупность его открытых методов. Тем самым он подобен черному ящику - вы что-то даете на вход, внутри ящика(класса) происходят некие таинственные процессы, а затем он выдает вам результат этих загадочных процессов. Поэтому любому пользователю класса важно чтобы должным образом работал именно этот интерфейс, и неважно, за счет чего он работает - за счет хитрых колдунств, безупречной логики или чего-то еще. О наличии приватных методов он ничего не знает и знать не должен. Соответственно, тестирование имеет смысл только для публичных методов. Если они работают должным образом, то и класс работает должным образом. А его внутренности - это его личное дело.
-
2@sp7: не, это какая-то странная логика. Так если тестировать любой метод, можно пропустить ошибки по невнимательности. Единственным правильным методом тестирования является проверка выполнения ТЗ на метод. Приватные методы имеют право появляться и исчезать, они имеют право крешить программу, будучи вызванными через reflection в неподходящий момент. Вы ж не собираетесь менять тесты с каждым изменением внутренней структуры класса, правильно?– VladD19 мая 2014 в 15:20
-
2> на время тестирования меняю у приватных методов модификатор доступа на публичный, тестирую их. Вот это точно не уруру. Половина всей сути тестов в том, чтобы выполнять их регулярно.– etki19 мая 2014 в 19:51
-
2@sp7: Если же у вас много логики реализовано в приватных методах, и этот код имеет свой смысл — скорее всего, ваш класс делает слишком много, его надо разбить на мелкие классы, имеющие самостоятельный смысл. Если в вашем классе более ста строк, стоит задуматься. --- Вот ещё хороший ответ по теме: hashcode.ru/questions/208710#208816– VladD19 мая 2014 в 21:17
-
2Согласен с ответом. Если класс становится таким сложным, что вам уже не обойтись без тестирования приватных методов, то это значит, что пора часть логики переносить в отдельный вспомогательный класс.– dzhioev20 мая 2014 в 0:06
-
1Есть ещё вариант воспользоваться "покрытием кода". Многие инструменты для юнит тестирования его делают. А оно покрывает все, в том числе и приватные методы. Если какой то приватный метод не покрыт (или его часть), то значит нужно улучшить тесты (или переписать класс детальнее, вынести логику, и тому подобное).– KoVadim20 мая 2014 в 10:26
Никак.
private
- и protected
-методы не принадлежат официальному «фасаду» класса, на них нет специфицированных пред- и постусловий.
Тестируйте публичные методы, на то они и публичные. Подглядывать в грязное бельё имплементации — некрасиво.
-
1@avp: на мой взгляд, с C ситуация не вполне аналогична: большие, осмысленные внутренние функции есть скорее аналог внутренних классов, а не приватных методов. Классы обычно разрабатываются (ну или должны разрабатываться) в соответствии с single responsibility principle: у класса есть одна, не раскладывающаяся на более мелкие, обязанность. Если приватные методы имеют семантику, скорее всего, этот принцип нарушен и класс чересчур сложен.– VladD20 мая 2014 в 7:53
-
1@VladD, в таком случае получается, что вся логика внутреннего устройства класса должна меняться только public методами (и ни в коим случае не private), хотя бы мы и хотели спрятать часть этой кухни. Поясняю -- если мы захотим изменить внутреннюю реализацию класса, а тестировать можно только вызывая public, то неразумно возлагать что-то нетривиальное на принципиально не тестируемые private методы. IMHO излишне строгий подход, проще отказаться от неудобных академических принципов. -- А вот как организовать это в шарпе? That is the question.– avp20 мая 2014 в 9:00
-
1@avp: Почему же? Логика внутреннего устройства может быть сколько угодно сложной, но эта сложность достигается не при помощи больших внутренних процедур, а при помощи композиции с другими объектами, реализующими части функциональности. Это очень похоже на unix way написания утилит: каждый класс делает маленькое задание, и их можно/нужно комбинировать. Пример: класс может запрашивать нужные данные, когда его об этом попросят, а может и закешировать их. Как именно он делает — невидимые миру подробности реализации. Кэш, понятно, имплементируется отдельным готовым классом, а не вручную.– VladD20 мая 2014 в 9:30
-
2@VladD, я вот еще раз перечитал Ваш и @DreamChild ответы. Такое ощущение, что вы оба говорите о тестировании с позиции программиста, использующего некий класс, а не пишущего (и потом поддерживающего) его. Или я ошибаюсь? -- А что по поводу реализации путем композиции нескольких классов, так это замечательно, но только если классы, включаемые в нее, самодостаточны и часто применимы для разных задач. Если же несколько таких классов могут использоваться только все вместе (да еще и только в рамках данной задачи), то это и будет вытаскиванием наружу нижнего белья (тоже м.б. неплохо :)).– avp20 мая 2014 в 11:57
-
5@avp: «Зелёный» (т. е., пройденный) тест — это гарантия (насколько полная — отдельный вопрос) того, что класс работает правильно с точки зрения пользователя класса. Пользователи класса вызывают лишь публичные методы, поэтому если даже какой-то приватный метод в каких-то условиях работает неправильно, это на самом деле никому не интересно. Если эта неправильная работа мешает правильной работе всего класса, тесты публичных методов должны не проходить. Если она никому не мешает, этот метод просто никому не нужен.– VladD20 мая 2014 в 12:43
Если в приватных методах достаточно большое количество логики - подумайте о выносе их в отдельный класс, в котором они будут простыми, публичными и тестируемыми.
Пример:
Надо протестировать класс построителя навмеша. Грубо говоря, у него на входе 3D модель местности, а на выходе 3D модель навмеша. Внутренняя логика же достаточно сложна (десятки тысяч строк кода).
Соответственно для юнит-тестирования, надо разбивать монолит на множество простых/примитивных классов (геометрические методы, построитель высот, вектора, триангуляторы и прочие) и тестировать их по отдельности юнит-тестами, и вместе - интеграционными.
Можно создать новую публичную функцию с префиксом "test" и с теми же аргументами. Использование таких функций легко отследить. Их легко найти (и, следовательно, удалить) в нужный момент.
Примечание: В Python для обозначения приватности используется "__". Но там можно получить доступ даже к приватному методу, прописав этот спецификатор. Возможно, я ошибся в деталях, но суть такая.