search

jurgens / flickbar

http://flickbar.com
7 stars 2 forks source link

Specs and some updates #7

Closed alg closed 13 years ago

alg commented 13 years ago

Привет. Посидел сегодня часок за проектом. Классно сделано.

Во view не ходил.

Вроде как все. Если что понравится, тяни. :)

jurgens commented 13 years ago

Спасибо, посмотрю

насчет Заменил User.find_or_create на стандартный User.find_or_create_by_title я думаю, что в будущем этот метод может расшириться (например, будет название и год) поэтому сразу сделал отдельный метод. Ты скажешь что заранее этого делать не надо, и скорее всего будешь прав :)

Ю

2011/3/31 alg < reply@reply.github.com>

Привет. Посидел сегодня часок за проектом. Классно сделано.

  • Сделал database.yml.sample и вынес .rspec в .rspec.sample
  • Немного поменял спеки
  • Добавил ZenTest в gembundle для любителей autotest
  • Добавил .watchr для любителей Watchr
  • Написал небольшую доку по инсталяции и тестированию
  • Вынес custom-code из spec_helper.rb в /spec/support (при обновлении rspec ничего не теряется)
  • Заменил User.find_or_create на стандартный User.find_or_create_by_title
  • Переименовал User.in_friendship_with в User.friends_with? (пишите тесты сначала и будет все хорошо)
  • Посмотри логику Watch.recently_watched

Во view не ходил.

Вроде как все. Если что понравится, тяни. :)

Reply to this email directly or view it on GitHub: https://github.com/jurgens/flickbar/pull/7

jurgens commented 13 years ago

Отлично, все понравилось :)