From ddbc9243e2ea3bcb260c15f209ca041817ccceb7 Mon Sep 17 00:00:00 2001 From: David Gil Date: Tue, 15 Sep 2015 21:40:50 +0200 Subject: [PATCH] speeds 10s the test suite by avoiding the creation a few unneeded objects --- spec/features/moderation/bulk_spec.rb | 2 +- spec/features/moderation/proposals_spec.rb | 4 ++-- spec/models/proposal_spec.rb | 10 +++++----- 3 files changed, 8 insertions(+), 8 deletions(-) diff --git a/spec/features/moderation/bulk_spec.rb b/spec/features/moderation/bulk_spec.rb index 5d6163f7b..fd3a408b3 100644 --- a/spec/features/moderation/bulk_spec.rb +++ b/spec/features/moderation/bulk_spec.rb @@ -34,7 +34,7 @@ feature 'Moderate in bulk' do end scenario "select all/none", :js do - create_list(:debate, 20) + create_list(:debate, 2) visit moderation_bulk_path diff --git a/spec/features/moderation/proposals_spec.rb b/spec/features/moderation/proposals_spec.rb index 50572c49e..977c44483 100644 --- a/spec/features/moderation/proposals_spec.rb +++ b/spec/features/moderation/proposals_spec.rb @@ -80,7 +80,7 @@ feature 'Moderate proposals' do end scenario "select all/none", :js do - create_list(:proposal, 20) + create_list(:proposal, 2) visit moderation_proposals_path @@ -98,7 +98,7 @@ feature 'Moderate proposals' do end scenario "remembering page, filter and order" do - create_list(:proposal, 55) + create_list(:proposal, 52) visit moderation_proposals_path(filter: 'all', page: '2', order: 'created_at') diff --git a/spec/models/proposal_spec.rb b/spec/models/proposal_spec.rb index f713f3ce5..2a5d1f9e5 100644 --- a/spec/models/proposal_spec.rb +++ b/spec/models/proposal_spec.rb @@ -79,7 +79,7 @@ describe Proposal do describe "#editable?" do let(:proposal) { create(:proposal) } - before(:each) {Setting.find_by(key: "max_votes_for_proposal_edit").update(value: 100)} + before(:each) {Setting.find_by(key: "max_votes_for_proposal_edit").update(value: 5)} it "should be true if proposal has no votes yet" do expect(proposal.total_votes).to eq(0) @@ -87,14 +87,14 @@ describe Proposal do end it "should be true if proposal has less than limit votes" do - create_list(:vote, 91, votable: proposal) - expect(proposal.total_votes).to eq(91) + create_list(:vote, 4, votable: proposal) + expect(proposal.total_votes).to eq(4) expect(proposal.editable?).to be true end it "should be false if proposal has more than limit votes" do - create_list(:vote, 102, votable: proposal) - expect(proposal.total_votes).to eq(102) + create_list(:vote, 6, votable: proposal) + expect(proposal.total_votes).to eq(6) expect(proposal.editable?).to be false end end