Recommended
More Related Content
What's hot
What's hot (16)
Recently uploaded
Recently uploaded (9)
Rails devs bb lt
- 2. 自己紹介 ・中村 雅行（なかむら まさゆき） ・Sier in 東京（６年くらい） ・Webエンジニア in London（９年くらい） ・市役所職員 in 茨城県某市（２年半くらい） ・Webエンジニア@五反田（１年ちょっと） ← 今ここ
- 4. 🍻cheers!
- 19. 手動でやるとまぢで大変 😩
- 20. 開発が忙しくて忘れる 😫
- 21. 自動化しよう！
- 26. Rails6 アップデート成功！
- 27. いや、ちょっと嘘。
- 28. テストが盛大に落ちた 😢
- 30. enum 😭
- 34. ご静聴ありがとうございました。
Editor's Notes
- みなさん、こんばんわ。 いこーよチケットサービスをRails6にアップデートしましたということで、LTは始めたいと思います。まずは、 https://www.slideshare.net/sinsoku/gem-122656455 https://speakerdeck.com/masayoshi2018/wen-xi-karaban-nian-deruby-sinatra-rails-woxue-ndararen-sheng-bian-watuta-at-chong-nawa-rubyhui-yi-02?slide=25
- 自己紹介です。 中村雅行といいます。 新卒で都内にあるSier企業に６年くらいお世話になりました。 そのあと何を思ったか、会社をやめて渡英します。 そこで、オンラインショップの梱包作業のアルバイトに申し込んだら、Railsっていういけてるミドルウェアがあるから、一緒にやらない？って誘われて、エンジニアとして働くことになりました。で、９年くらいロンドンで働いてたんですけど、５年くらい前に帰国して、一回、市役所職員として働いてます。 でも、やっぱり、先生、コーディングがしたいです。ということになり。もう１度、エンジニアとして働くことになりました。 そんなブランクありのエンジニアを採用してくれたのが、
- アクトインディ株式会社という素晴らしい会社です。
- Cheers！ はい、
- で、アクトインディでは、主に子育て世代向けに３つの情報サイトを運営していて、
- 今回、僕のいるチームではお出かけ情報サイトいこーよで、Webチケットが購入できるようにサービスリリースしました。その時点での環境が、
- rubyが2.6.3、railsが5.2.3でリリースしてます。ただ、GW周辺で、
- Rails6がそろそろリリースされるかもよ？的な話で周りがざわついていたので、Rails6でいっちゃうか、5.2にしておこうか、チーム内でもかなりざわざわしてました。 結果的に、Rails5.2でリリースしたのですが、Rails6が正式リリースされたらサクッとあげれるように準備しておこうということをサービスリリース後の目標の１つにしました。 今回は、その中で、やってきてよかったことを２つほど紹介したいと思います。
- まずは、テストをちゃんと書く。ということ。
- 僕らのチームはRSpecでテスコードを書いていて、SimpleCovでカバレッジを計測しています。 カバレッジの指標値は90%以上としています。
- まず、担当者がレビュー依頼を出す際にGithub上で「自動テスト対象」というラベルを貼ると、Codebuidが自動テストを回してくれます。
- テスト以外にも、rubocop、rails_best_practice、brakemanを実行させて、チーム内でコード規約が自動で保たれるような仕組みも導入しています。
- 規約に違反しているところが見つかると、github上にコメントを残してくれます。
- あまりにぬるいコードを書いていると、容赦ないツッコミが入ります。だいだい、50を越えると、ページを開くのも大変。みたいな結果になったりして苦労したことがありました。
- 結果はスラック上にも通知されるので、revieweeもreviewerもこの値を確認して、カバレッジが維持されているか確認しています。 前回よりも大きく下がっていたりすると、詳細を確認して実際ちゃんとテストコードが書かれて、パスしているかを確認します。
- 全体図はこんな感じです。まず、GithubからAmazonSNSへイベントを通知し、それをトリガーにしてLambdaがCodeBuildを起動します。 CodeBuildが終了すると、CloudWatchEventが、イベントを受けてLambdaがSlackに通知するといった流れになっています。この辺の詳しい設定などは後ほど紹介するアクトインディ開発者ブログに詳しく書かれているので興味のある方はのぞいてみてください。次に、
- サービスで使っているRails以外のGemのアップデートです。これは、
- 各Gemの更新確認。更新されてたら、作業チケットを切る、担当者がコミットログ、リリースノートを確認する、テストする、落ちたら通過するように修正す。動作確認を実行する。これを、
- 手動でやるとまぢで大変です。大変だし、
- 開発が忙しくて、更新するのを忘れます。なので、
- 自動化しよう！という結論に至りました。そこで、
- ぼくたちのチームは、dependabotというサービスを採用しました。dependabotは更新のあったgemをチェックしてくれて、更新があったら自動でプルリクを作成してくれます。実は、このdependabotの導入を検証してくれたのが、社内で一番新しいエンジニアさんなんですが、弊社では社歴とかに関係なく新しいことにチャレンジできる文化と、それに快くおっけーを出してくれるチームメーンバーなので、その辺も手前味噌ながら、いい会社だなぁとおもっています。で、dependabotなんですが、
- 僕らのチームは毎週、金曜日にアップデートチェックが走るような設定にしていて、そのタイミングで更新があったGemは自動でプルリクが作成され、自動テストが実行されるようになっています。
- 全体イメージはこんな感じです。 問題がなかったブランチに関しては、一旦、動作確認用のブランチに自動マージするようになっていて、問題があったブランチに関しては、リリースノートや変更内容のコミットログを確認し、テストが通過するように修正を行いプッシュし、動作検証ブランチにマージします。
- その後、動作確認用のブランチでrevieweeがまとめて動作確認を行い問題がなければ、レビュー依頼を出し、reviewerがレビューします。 こういったことを毎週繰り返すことで、
- 結果的に、他Gemと同様のプロセスで特に問題なくRails6へのアップデートに成功しました。
- とはいっても、やっぱり、少し引っかかったところがありました。まず、やはりRails6のアップデートブランチでは、
- テストが盛大に落ちました。でも、これは、
- rspec-railsのアップデートで解決してます。次に、
- enumのnegative scopeです。これは、
- Rails6から、negative scopeが採用されているので、該当箇所を修正しています。enumは以外と他でも使っていたのですが、僕らのチームでは影響範囲が少なかったので、助かりました。最後に、
- まとめです。 テストはやっぱりちゃんと書いておくと、「多分動くと思うからリリースしようぜ」的な思い切りが必要なときの１つの指標にもなるし、安心材料にもなり得るということ。 あとは、なるべく効率的な仕組み化を検討していくことはとても大事だなと実感しました。
- 今日、話した内容の具体的な設定やコードなどは、開発者ブログがあるので興味のある方はこちらも参考にしてみてください。 https://tech.actindi.net/2017/10/23/codebuild-rspec.html https://tech.actindi.net/2018/08/31/120906 https://tech.actindi.net/2019/06/21/181709