mizukmb / awesome_events

Perfect Ruby on Rails
1 stars 1 forks source link

イベント機能の作成 #9

Closed mizukmb closed 8 years ago

mizukmb commented 8 years ago
mizukmb commented 8 years ago
Failures:

  1) Event#name 
     Failure/Error: it {should validate_presence_of(:name) }

     NoMethodError:
       undefined method `validate_presence_of' for #<RSpec::ExampleGroups::Event::Name:0x007f874b7bdd40>
     # ./spec/models/event_spec.rb:5:in `block (3 levels) in <top (required)>'

Finished in 0.00173 seconds (files took 6.06 seconds to load)
1 example, 1 failure

Failed examples:

rspec ./spec/models/event_spec.rb:5 # Event#name

は、 spec/rails_helper.rb

RSpec.configure do |config|
 config.include(Shoulda::Matchers::ActiveModel, type: :model)
 config.include(Shoulda::Matchers::ActiveRecord, type: :model)
end

と記述すると解決する

mizukmb commented 8 years ago

ensure_length_of はv3.0.0より validate_length_of に rename されていた

http://stackoverflow.com/questions/33280731/shoulda-matchers-are-not-seen-by-rails-rspec https://github.com/thoughtbot/shoulda-matchers/blob/55c8d09bf2af886540924efa83c3b518d926a770/NEWS.md

a-know commented 8 years ago

こういう、実際の作業(コードを伴う)をやりながら出てきたものは、 hanocha がやってるように、PR を作ってその中のコメントとして書いてくれたほうが 対応付けがわかりやすくていいかなー。

issue を使うと良さそうなのは、

とかっていったケースだろうか!

mizukmb commented 8 years ago

確かに issue に実装レベルの作業メモを書き残すのは違うように見えますね。 ありがとうございます!