rspecモデル仕様vs工場

Question

私はテストを開始していますが、letを使用するときは正確にはわかりません。

遅延ロードのモデルテストではletを使用する必要がありますか、またはすべてのテストでデータが少し異なるため、そのまま使用できますか？いくつかの例で見たように、コントローラーテストの方が重要です。具体的には、:taskはそれぞれのアクションテストで同じです。

モデルの仕様

require 'rails_helper' 

RSpec.describe Task, type: :model do 

    describe "model validations" do 

    it "has a valid factory" do 
     expect(build(:task)).to be_valid 
    end 

    it "is invalid without executor" do 
     expect(build(:task, executor_id: nil)).not_to be_valid 
    end 

    it "is invalid without assigner" do 
     expect(build(:task, assigner_id: nil)).not_to be_valid 
    end 

    it "is invalid without content" do 
     expect(build(:task, content: nil)).not_to be_valid 
    end 

    it "is invalid without deadline" do 
     expect(build(:task, deadline: nil)).not_to be_valid 
    end 

    it "is invalid with deadline in the past" do 
     expect(build(:task, deadline: Faker::Time.between(DateTime.now - 1, DateTime.now - 2))).not_to be_valid 
    end 
    end 
end 

工場

FactoryGirl.define do 
    factory :task do 
    content { Faker::Lorem.sentence } 
    deadline { Faker::Time.between(DateTime.now + 2, DateTime.now + 3) } 
    association :executor, factory: :user 
    association :assigner, factory: :user 
    end 
end 

Answer 1

メリット：何が価値がある、あなたがそこに持っているものを達成するためにshoulda-matchers宝石を使用することができるため

。このグループを想像してみてください。

context "completing tasks" do 
    let(:completing_a_task){ task.complete } 
    context "that aren't due yet" do 
    let(:task){ create(:not_due_task) } 
    it "should not send an email" do 
     expect(TaskMailer).not_to receive(:deliver_message) 
     expect{ completing_a_task }.not_to raise_error 
    end 
    end 
    context "overdue" do 
    let(:task){ create(:overdue_task) } 
    it "should send an email" do 
     expect(TaskMailer).to receive(:deliver_message) 
     expect{ completing_a_task }.not_to raise_error 
    end 
    end 
end 

遅延バインディングを使用すると、変更を最小限に抑えることができますが、最大限のカバレッジが得られます。テストに必要な適切な動作を設定するために必要な共同作業者が多いほど、letの恩恵を受ける可能性が高くなります。特にテストスイートでDRYを実行する必要はありませんが、テスト用の巨大なセットアップブロックは匂いです。letテクニックは、ドメインが複雑になっても明快でシンプルに戦うのに役立つ素晴らしいツールです。自分の例にはまだ協力者はいませんが、そのコンセプトはまだ十分明確です。

Answer 2

私はletを使用して、あなたがそれを持ってどのようにそれを維持していないことをお勧めしたいです。あなたのテストのDRYを心配しないでください。それらはお互いに対話しないので、アプリケーションロジックでコードの重複が発生することはありません。あなたは上記の使用の形のものではないテストから来letのhttps://github.com/thoughtbot/shoulda-matchers

describe Task do 
    describe "validations" do 
    it { is_expected.to validate_presence_of(:content) } 
    it { is_expected.to validate_presence_of(:deadline) } 
    end 
end