0gajun / 2018-newbies

1 stars 1 forks source link

残高管理用のBalanceモデルの追加 #43

Closed 0gajun closed 6 years ago

0gajun commented 6 years ago

ユーザの残高管理を行うための Balance モデルの作成と、バリデーションのテストを追加した

(Rspecの書き方っていいのかこれで 🤔

一応テーブル情報

mysql> describe balances;
+------------+------------+------+-----+---------+----------------+
| Field      | Type       | Null | Key | Default | Extra          |
+------------+------------+------+-----+---------+----------------+
| id         | bigint(20) | NO   | PRI | NULL    | auto_increment |
| user_id    | bigint(20) | NO   | MUL | NULL    |                |
| amount     | bigint(20) | NO   |     | NULL    |                |
| created_at | datetime   | NO   |     | NULL    |                |
| updated_at | datetime   | NO   |     | NULL    |                |
+------------+------------+------+-----+---------+----------------+
5 rows in set (0.00 sec)

mysql> show index from balances;
+----------+------------+---------------------------+--------------+-------------+-----------+-------------+----------+--------+------+------------+---------+---------------+
| Table    | Non_unique | Key_name                  | Seq_in_index | Column_name | Collation | Cardinality | Sub_part | Packed | Null | Index_type | Comment | Index_comment |
+----------+------------+---------------------------+--------------+-------------+-----------+-------------+----------+--------+------+------------+---------+---------------+
| balances |          0 | PRIMARY                   |            1 | id          | A         |           0 |     NULL | NULL   |      | BTREE      |         |               |
| balances |          1 | index_balances_on_user_id |            1 | user_id     | A         |           0 |     NULL | NULL   |      | BTREE      |         |               |
+----------+------------+---------------------------+--------------+-------------+-----------+-------------+----------+--------+------+------------+---------+---------------+
0gajun commented 6 years ago

Chargesのbalance対応も平行してやるために、一旦モデルだけmasetrにマージすることに

アプリケーションに影響ないのでおkという判断 🙆

ShuheiGoto commented 6 years ago

LGTM

specに関しては,テストケースが十分であれば良いという判断でとりあえず進める

takuyan commented 6 years ago

だんだんレビュア側は「このテストは本当にGreeeenなのかな?」っていう不安が出てきますね!

そんなとき! そんなときは!?

0gajun commented 6 years ago

let's CI :gogo:

0gajun commented 6 years ago

circle ciでも明日ちょろっと入れようか

takuyan commented 6 years ago

公開リポジトリだし無料枠でいけるよね!やったぜ!

ryoff commented 6 years ago

よげ 👍