イベント履歴収集スクリプトでプロバイダ指定したとき、成功/失敗のメッセージにプロバイダを含める

[chicaco]

背景

#371 & #375 マージ＆デプロイ後、本番環境で統計スクリプトを実行したが、facebook のイベント履歴収集＋登録処理でエラーになる。
プロバイダ指定で統計スクリプトを実行しても、idobata に通知される正常終了のメッセージからはどのプロバイダを指定しての実行が成功したのか、どのプロバイダで成功していないのか、判断できない。

やりたいこと

1. プロバイダを指定してイベント履歴収集スクリプトを実行したとき、成功/失敗のメッセージに対象プロバイダ名を含め、どのプロバイダで成功/失敗したのか分かるようにする

1. facebook のイベント履歴収集＋登録処理のエラーを解消する → #398
1. 本番環境でのエラー情報が idobata に通知が来るよう修正する (正常終了時の通知は来ている) → #397
cf. #395

このPRでやること

- [ ] facebook のイベント履歴収集＋登録処理のエラーの原因を調査し、解消する
- [ ] 本番環境でのエラー情報が idobata に通知が来るよう修正する

• イベント履歴収集スクリプトをプロバイダ指定で実行したとき、正常/エラーメッセージにプロバイダを含める

やらなかったこと

• facebook のイベント履歴収集＋登録処理のエラーの原因を調査し、解消する
• 本番環境でのエラー情報が idobata に通知が来るよう修正する

レビューポイント

• プロバイダ指定なし(＝すべて)のとき、正常/エラーメッセージへのプロバイダ名の埋め込みは行わない
• プロバイダ指定ありのとき、正常/エラーメッセージへのプロバイダ名の埋め込みを行う

困ってること

特になし

[chicaco]

facebook のイベント履歴収集＋登録処理のエラーの件、本番DBをリストアしたローカル環境で再現しました。
やはり原因は dojo_event_services T に二重登録されているレコードで、#371 & #375 で facebook イベントを yaml から読み込むよう修正したときの対応漏れでした。

対応方法をいくつか考えたのですが、dojo_event_services T を正しいレコードにすること、が恒久的にも一番有効だという結論に至りました。つまり rake dojo_event_services:upsert を修正して、二重登録が起きないように(※)します。
(※1) find_by の一意条件を dojo_id, name, group_id に限定し、url は除外

⇒ 修正対象が異なりますので、小規模ですが別 PR に切り分けます。

また、2018 年のイベント履歴収集を優先し、「本番環境でのエラー情報が idobata に通知が来るよう修正する」も難航しそう(※2)なので別 issue に変更します。
(※2) ローカル環境からテスト用 idobata 部屋に curl したら成功してしまいました。

欲張って「やりたいこと」を積んだこと...、反省です。

[chicaco]

間違ったボタンをクリックしてしまいました... 🙇‍♀️
これからレビュ依頼ですっ。 🙏

[chicaco]

この PR でのスコープを見直して絞りました。
レビューお願いします。 🙏 > @nalabjp

[nalabjp]

LGTM✨