Arion Mar 2 2009 at 13:32

RSpec. Часть #1: создаем тесты для модели

3 min

38K

Ruby on Rails*

+12

Comments 12

mholub Mar 2 2009 at 15:46

Из замечаний:
1-й it совершенно не очевиден, я бы его обозвал «should require name»
2-й it что вообще проверяет? правильную работу ActiveRecord'а? Так он сам отспекан уже, это не ваша задача — его проверять.

Из пожеланий:
Последняя мода в ruby-community не юзать фикстуры (они ж реально неудобные, особенно, когда модель сложная и нужно прописать в ней кучу ассоциаций, поэтому посмотрите в сторону github.com/thoughtbot/factory_girl/tree/master

Arion Mar 2 2009 at 16:04

1 изменил, так наверно и вправду лучше, насчет второго, это просто для примера, что бы понять как работают фикстуры.

mholub Mar 2 2009 at 16:10

ну тогда пример неудачный, кто-то подумает, что так и надо тестить
можно создать две фикстуры: valid и missing_name и на них показать работу фикстур, если уж так хочется

snitko Mar 3 2009 at 00:07

Поддерживаю. Оба теста занимаются проверкой AR, что в 90% случаев не нужно, даже в качестве примера. Я иногда пишу тесты, проверяющие AR, но только чтобы убедиться, что я нигде не ошибся в описании модели (например когда у меня сложные ассоциации). Потом эти тесты я обычно удаляю.

По поводу фикстур зря вы: я бы порекомендовал посмотреть примочки типа nested_scenarios — делает работу с фикстурами удовольствием.

bewhite Mar 2 2009 at 15:56

Может перенести статью в блог Ruby?

Arion Mar 2 2009 at 15:59

Ruby is not Rails

mholub Mar 2 2009 at 16:05

но Rails < Ruby, если нет блога по Rails, логичнее же в Ruby, чем в своем блоге

Somebody32 Mar 2 2009 at 20:13

Rspec вещь универсальная, тут Rails больше для примера, т.к. если справится с тестами на них, то потом уже что угодно тестировать можно, подход мало изменится.

FunFun Mar 2 2009 at 23:55

наконец-то, читаем…

snitko Mar 3 2009 at 00:09

И еще лучше придерживаться какого-то одного стиля. Либо it «should require», либо 3-е лицо единственное число it «requires». А у вас в двух тестах по разному.

Over Mar 3 2009 at 06:16

Почему сразу не autotest настроить?
Не понятно, зачем фикстуры то? Бла-бла-бла и прочую рыбу можно и на лету генерить.

timurv Mar 6 2009 at 14:21

А еще есть тема применить shoulda, они планируют всякие вкусны матчеры запустить под rspec giantrobots.thoughtbot.com/2009/2/3/speculating-with-shoulda