Upgrade Rails to 5.1.4 and Scrivito to 1.11.0

[yasulab]

This PR upgrades Rails, Scrivito, and other related gems to the latest versions:

• Rails 5.1.4
• Scrivito 1.11.0
• And exec $ bundle update to update other related gems

[yasulab]

@nalabjp すみません 💦 だいたい Rails 5.1 に更新できたかなと思ったのですが、このテストが落ちる原因だけがよくわからなくてちょっと手が止まっちゃっています ><

Failures:
  1) Statistics::Aggregation.run should change `EventHistory.count` from 0 to 2
     Failure/Error: Dojo.delete_all
     ActiveRecord::InvalidForeignKey:
       SQLite3::ConstraintException: FOREIGN KEY constraint failed: DELETE FROM "dojos"
     # ./spec/lib/statistics/aggregation_spec.rb:14:in `block (2 levels) in <top (required)>'
     # ------------------
     # --- Caused by: ---
     # SQLite3::ConstraintException:
     #   FOREIGN KEY constraint failed
     #   ./spec/lib/statistics/aggregation_spec.rb:14:in `block (2 levels) in <top (required)>'

cf. https://travis-ci.org/coderdojo-japan/coderdojo.jp/builds/294500786#L664-L676

もし何かアイデアがあればアドバイスもらえるとめっちゃ助かります 🙏 💦 (手元ではテストが全て通っているので、DB周りで何か見逃してるのかなぁと考えています🤔💭)

[yasulab]

んー、db:drop && db:create しても手元のテストが通ってしまう不思議... 😓

$ bundle exec rspec spec/

...............................................

Finished in 13.05 seconds (files took 5.87 seconds to load)
47 examples, 0 failures

[nalabjp]

@yasulab

SQLite3::ConstraintException: FOREIGN KEY constraint failed: DELETE FROM "dojos"

あぁ、ゴメンナサイ、Modelの実装のような気が。

coderdojo.jp/db/migrate/20170730050122_create_event_histories.rb

Line 4 in 688f1f5

t.references :dojo, foreign_key: true, index: true, null: false

coderdojo.jp/db/migrate/20170730114150_create_dojo_event_services.rb

Line 4 in 688f1f5

t.references :dojo, foreign_key: true, index: true, null: false

この2箇所でFOREIGN KEY制約をかけているので、

coderdojo.jp/app/models/dojo.rb

Lines 6 to 7 in 688f1f5

	has_one :dojo_event_service
	has_many :event_histories

ここにdependent: :destroyを追加しないといけないのかな？🤔（未確認）

[yasulab]

オォ! ありがとうございます! 😸 とりあえず feature branch に push して Travis CI 走らせてみます 💨

[yasulab]

Hmm, テストが同じところで落ちているところを見ると、別の何かが原因っぽいですねぇ🤔💭
https://travis-ci.org/coderdojo-japan/coderdojo.jp/builds/294510105#L666-L678

[nalabjp]

あ、Dojo.delete_allで呼んでるからSQLを直接発行していてARのdependentが効いていないのか。
Dojo.destory_allにするとどうでしょうか？

[yasulab]

試してみます! 💨

[yasulab]

アドバイスありがとうございます! 😸 無事にテスト通りました✅ ✨
無事にテスト通ったので、こちらのPRマージしますね 🔧 💨