商品出品機能 ０

[kaneko-joei]

What 商品出品機能

Why

テストコード実行後の画像を添付 https://gyazo.com/c95c2214fede158418d29e32c44f9c13 クティブハッシュに関するカラムは数値を扱うカラムなので''は外して1のように記述しましょう。 他のアクティブハッシュに関するレスとも同様です

テストコード実行後の画像を添付 https://gyazo.com/4790bfbbe0529144c6cf4f3e657e6703

numericality: { other_than: 1 }が重複しているため with_options を使用してまとめる

ActiveHashのテストも追加 prefecture蘭が未選択のidの場合は保存できないこと

テストコード実行後の画像を添付 https://gyazo.com/8afcd3c14f6b8954758626c1fc716b75

入力された値に対して1円の誤差が出てしまうため priceの誤差がなくなるように実装を修正。

validates :price, presence: true, numericality: {only_integer: true, greater_than_or_equal_to: 300, less_than_or_equal_to: 9999999} こちらの記述のpresence: trueの部分は削除。 presence: trueについてはwith_optionsを利用して適用されている為

userが紐付いていない場合は保存出来ないこと追加

active_hashには未選択状態のidがあり 未選択の場合でも「未選択のid」がデータベースに保存されてしまう可能性がある為 未選択の時はidを保存できないようにバリデーションを追加。

active_hasnは未選択でも「未選択のid」が送られ 未選択状態のidが送られてきた際にデータを弾くかどうかテストしました。

バリデーションの追加 ・金額は全角数字では保存できないこと ・金額は数字以外が混じっていると保存できないこと

spec/models/item_spec.rb:

• before do
• @item = FactoryBot.build(:item) +end 現在FactoryBotの記入

app/models/item.rb:

• extend ActiveHash::Associations::ActiveRecordExtensions
• belongs_to :prefecture
• belongs_to :category
• belongs_to :shipping_cost
• belongs_to :shipping_days
• belongs_to :status
• belongs_to :user
• has_one_attached :image

• +end バリデーションの記述

def index

• @items = Item.order("created_at DESC") 次回実装内容になるため一旦コメントアウト

  ・出品に成功する動画 https://gyazo.com/c26484fd7cfe43fca7cb7296432e47a5 ・出品に失敗しエラーが表示される動画 https://gyazo.com/cd15792a6ab2a192d1a454a540023c78 ・ログイン状態のユーザーだけが、商品出品ページへ遷移できる動画 https://gyazo.com/38996241dded1918dd1fba9f58e2a99d ・ログアウト状態のユーザーは、商品出品ページへ遷移しようとすると、ログインページへ遷移する動画 https://gyazo.com/df9898b0b5ca0f09987b7ab1f08cac48 ・金額が入力されたと同時に手数料と利益が表示される動画 https://gyazo.com/90e7f539bc5e1c173e9fa18909a6c63f ・テスト結果の画像 https://gyazo.com/e71454e3d88a56d23485f5e22fc11092

  Why 商品出品機能を実装する

     - モデルの単体テストコードを書く

[chobi555]

レビュー依頼ありがとうございます！ LGTMとさせていただきます、実装お疲れさまでした！

マージ後、ローカルのmasterブランチに忘れずにプルしましょう。 その後、新しいブランチを作成し次の実装に取り組んでいただけますと幸いです。 引き続き頑張っていきましょう！