Rspec：如何重構類似的測試？ （給出的例子）

Question

我有2個例子，但我覺得他們裏面的大部分代碼是相同的。然而，它們有點不同（記錄略有不同，並且在第二個中也有一個額外的說法）。我在測試中仍然是初學者，所以在我前進時尋找一些提示。我正在測試一個耙子任務。這是我的代碼：

it 'leaves one billing info for each order' do 
    order = FactoryGirl.create(:order) 
    FactoryGirl.create_list(:billing_info, 2, order_id: order.id) 

    expect(BillingInfo.all.count).to eq(2) 

    run_rake_task 

    expect(BillingInfo.all.count).to eq(1) 
end 

it 'keeps the billing info with trevance information' do 
    order = FactoryGirl.create(:order) 
    FactoryGirl.create(:billing_info, order_id: order.id, complete_trevance_message: nil, trevance_attempts: nil) 
    FactoryGirl.create(:billing_info, order_id: order.id, complete_trevance_message: "303 -- Processor Decline", trevance_attempts: 1) 

    expect(BillingInfo.all.count).to eq(2) 

    run_rake_task 

    expect(BillingInfo.all.count).to eq(1) 
    expect(BillingInfo.first.complete_trevance_message).to eq("303 -- Processor Decline") 
end 

正如你所看到的，它們非常相似。把這些分成兩部分是可以的嗎？或者，還有更好的方法？

Answer 1

在我看來，DRY並不總是測試中最好的規則。它通常意味着你必須在方法中隱藏一些代碼，所以這很難閱讀等等。所以我沒有太大的改變，但有一些事情可以用更簡單的方式來完成。

context "billing rake task" 
    let(:order) { FactoryGirl.create(:order) } 

    it 'leaves one billing info for each order' do 
    FactoryGirl.create_list(:billing_info, 2, order_id: order.id) 
    expect { run_rake_task }.to change { BillingInfo.count }.by(-1) 
    end 

    it 'keeps the billing info with trevance information' do 
    FactoryGirl.create(:billing_info, order_id: order.id, complete_trevance_message: nil, trevance_attempts: nil) 
    FactoryGirl.create(:billing_info, order_id: order.id, complete_trevance_message: "303 -- Processor Decline", trevance_attempts: 1) 

    expect { run_rake_task }.to change { BillingInfo.count }.by(-1) 
    expect(BillingInfo.first.complete_trevance_message).to eq("303 -- Processor Decline") 
    end 
end 

請注意，這會稍微改變規格，因此您不會檢查計數是否從2正好改變爲1，而是改爲1。我認爲這樣更好，但我不能確定，因爲我不太瞭解你的應用。

Answer 2

我將重構爲這樣的事情：

let(:order) { FactoryGirl.create(:order) } 

describe "description here" do 

    before do 
    FactoryGirl.create_list(:billing_info, 2, order_id: order.id) 
    end 

    it 'leaves one billing info for each order' do 
    expect(BillingInfo.all.count).to eq(2) 

    run_rake_task 

    expect(BillingInfo.all.count).to eq(1) 
    end 

end 

describe "description here" do 

    before do 
    FactoryGirl.create(:billing_info, order_id: order.id, complete_trevance_message: nil, trevance_attempts: nil) 
    FactoryGirl.create(:billing_info, order_id: order.id, complete_trevance_message: "303 -- Processor Decline", trevance_attempts: 1) 
    end 

    it 'keeps the billing info with trevance information' do 
    expect(BillingInfo.all.count).to eq(2) 

    run_rake_task 

    expect(BillingInfo.all.count).to eq(1) 
    expect(BillingInfo.first.complete_trevance_message).to eq("303 -- Processor Decline") 
    end 

end 

的before鉤會給你在未來更多的靈活性，您的需要。