Heroku Review Apps を coderdojo.jp でも使いたい

[yasulab]

yasslab.jp や railstutorial.jp では導入済み ✅ Review Apps はメッチャ便利で、 例えばこういったテストで見抜きにくいエラーに事前に気づけるし、デザインのレビューでもスクショなどをコメント欄に貼る必要がなくなるので、coderdojo.jp でも導入したい 🚀🚀🚀 ✨

Review Apps | Heroku Dev Center
https://devcenter.heroku.com/articles/github-integration-review-apps

[yasulab]

Heroku Review Apps 設定しました ✅ 🔧 💨
cf. Add Heroku generated app.json 776f00a

[yasulab]

んー、これまだうまく動いていない...? ようなのでひとまず Re-open 💦
cf. #409

[AnaTofuZ]

closeしてしまったので復旧しました。
今のところの知見をここにもまとめておきます。

原因に関しては　#409 のPRで次の様な事かなと発見しました。

• ReviewAppsでアプリケーションが作成、デプロイされる際にDB関連の初期化、seedの設定が行われていなかった
• 現在conderdojoのトップページでは、ログインした際にいくつかDojoのモデルを利用するものがある
• この中で group_by_region メソッドが呼ばれている箇所でエラーが発生している
• その為、 DB関連の設定をデプロイ時に設定すれば解決しそう

[AnaTofuZ]

group_by_region はこれです

coderdojo.jp/app/models/dojo.rb

Lines 38 to 40 in ca2b95c

	def group_by_region
	eager_load(:prefecture).default_order.group_by { |dojo| dojo.prefecture.region }
	end

[AnaTofuZ]

試しにconsoleで使ってみたところ破滅したので、ここが原因であってそうです

+mumei+anatofuz$ heroku console -a coderdojo-japan-pr-446
Running console on ⬢ coderdojo-japan-pr-446... up, run.2370 (Hobby)
Loading staging environment (Rails 5.1.6.2)
irb(main):001:0> Dojo.group_by_region
D, [2019-05-15T20:04:14.804297 #4] DEBUG -- :   SQL (81.4ms)  SELECT "dojos"."id" AS t0_r0, "dojos"."name" AS t0_r1, "dojos"."email" AS t0_r2, "dojos"."order" AS t0_r3, "dojos"."description" AS t0_r4, "dojos"."logo" AS t0_r5, "dojos"."url" AS t0_r6, "dojos"."tags" AS t0_r7, "dojos"."created_at" AS t0_r8, "dojos"."updated_at" AS t0_r9, "dojos"."prefecture_id" AS t0_r10, "dojos"."is_active" AS t0_r11, "dojos"."is_private" AS t0_r12, "prefectures"."id" AS t1_r0, "prefectures"."name" AS t1_r1, "prefectures"."region" AS t1_r2 FROM "dojos" LEFT OUTER JOIN "prefectures" ON "prefectures"."id" = "dojos"."prefecture_id" ORDER BY "dojos"."prefecture_id" ASC, "dojos"."order" ASC
Traceback (most recent call last):
        4: from (irb):1
        3: from app/models/dojo.rb:39:in `group_by_region'
        2: from app/models/dojo.rb:39:in `group_by'
        1: from app/models/dojo.rb:39:in `block in group_by_region'
NoMethodError (undefined method `region' for nil:NilClass)