Как стать автором
Обновить

Комментарии 6

Всё, что не с underscore начинается, публичное же! Че сразу грязный хак, нормальное решение

Ну так то да. С другой стороны, этот класс в штатной доке Django не упоминается, соответственно никаких гарантий по обратной совместимости при выходе новых версий Django не дает.

Вот именно в таких случаях и пишут тесты ;)

Тесты ценны сами по себе, безотносительно любых случаев.

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

И coverage 100% хоть и не является "серебряной пулей", но очень на нее похож.

Разве код clist = ChangeList...

Hidden text
    clist = ChangeList(
      request,
      self.model,
      self.list_display,
      self.list_display_links,
      self.list_filter,
      self.date_hierarchy,
      self.search_fields,
      self.list_select_related,
      self.list_per_page,
      self.list_max_show_all,
      self.list_editable,
      self,
      self.sortable_by
    )

нельзя заменить кодом

clist = self.get_changelist_instance(request)

?

Наверное можно и нужно.

Шансов на то, что сигнатура вызова со временем не изменится, существенно больше, чем у вызова конструктора класса в коде из моей статьи.

Спасибо за подсказку.

Зарегистрируйтесь на Хабре, чтобы оставить комментарий

Публикации

Истории