Feature/add event item repository

sato-mh / savoten

We want to make a voting system that can be used easily.

MIT License

0 stars 1 forks source link

Closed iamsmith06 closed 4 years ago

iamsmith06 commented 5 years ago

event_item_repositoryとevent_item_repository_interfaceを作りました
- conhumiさんのevent_repositoryの ~パクり~ 構成をコピーしてます。
ざっくりですがtestも作ってみました

iamsmith06 commented 5 years ago

コメントもらったところの修正に加えて、event_idとevent_itemの関係を記録する役割をこのrepositoryに追加しました。(レビュー後に追加してごめんなさい！これ必要なのに後から気付きました)
- event_id_to_event_item_map関連の処理です。
- テーブル設計ではevent_itemテーブルがevent_idを持つのですが、on_memoryなのでその代わりをします。
- .find_by_event_idされた時などにevent_idに紐づくevent_itemを返す処理をするために追加しました。

iamsmith06 commented 5 years ago

pytestの理解が進んだので前処理を全体的に書き直してみました。

基本的にsetup_classでrepository前処理をする
test関数でrepositoryの書き換えを行う場合はsetup_methodでrepository前処理する(test関数実行毎に初期化) こんな感じのポリシーで書いてみました。

conhumi commented 4 years ago

遅ればせながら確認しました！ Python プロダクトのレビューの経験はあんまりないのですが、 LGTM です！

iamsmith06 commented 4 years ago

二人ともレビューありがとう！ F401を.flake8でignoreするよう修正しました。（各ファイルからは#noqa F401消してあります。

sato-mh commented 4 years ago

修正ありがとうございます、良さそうなのでマージします！