aboutsummaryrefslogtreecommitdiffstats
path: root/spec/models/info_request_batch_spec.rb
diff options
context:
space:
mode:
authorLouise Crow <louise.crow@gmail.com>2013-10-24 17:03:52 +0100
committerLouise Crow <louise.crow@gmail.com>2013-12-04 09:32:43 +0000
commitce262657761c1c47c47e2f6cc7bf683543ebbdb7 (patch)
tree800fc74aebcbc5dae39800abdf0830f2afa6251e /spec/models/info_request_batch_spec.rb
parent4b2e3ef0b6f5b881597242a4afe1404415256f3b (diff)
Add check for double submission.
Diffstat (limited to 'spec/models/info_request_batch_spec.rb')
-rw-r--r--spec/models/info_request_batch_spec.rb45
1 files changed, 45 insertions, 0 deletions
diff --git a/spec/models/info_request_batch_spec.rb b/spec/models/info_request_batch_spec.rb
index f6d9bea0c..787b812cf 100644
--- a/spec/models/info_request_batch_spec.rb
+++ b/spec/models/info_request_batch_spec.rb
@@ -18,4 +18,49 @@ describe InfoRequestBatch, "when validating" do
@info_request_batch.errors.full_messages.should == ["Title can't be blank"]
end
+ it 'should require a body' do
+ @info_request_batch.body = nil
+ @info_request_batch.valid?.should be_false
+ @info_request_batch.errors.full_messages.should == ["Body can't be blank"]
+ end
+
+end
+
+describe InfoRequestBatch, "when finding an existing batch" do
+
+ before do
+ @info_request_batch = FactoryGirl.create(:info_request_batch, :title => 'Matched title',
+ :body => 'Matched body')
+ @first_request = FactoryGirl.create(:info_request, :info_request_batch => @info_request_batch)
+ @second_request = FactoryGirl.create(:info_request, :info_request_batch => @info_request_batch)
+ end
+
+ it 'should return a batch with the same user, title and body sent to one of the same public bodies' do
+ InfoRequestBatch.find_existing(@info_request_batch.user,
+ @info_request_batch.title,
+ @info_request_batch.body,
+ [@first_request.public_body_id]).should_not be_nil
+ end
+
+ it 'should not return a batch with the same title and body sent to another public body' do
+ InfoRequestBatch.find_existing(@info_request_batch.user,
+ @info_request_batch.title,
+ @info_request_batch.body,
+ [FactoryGirl.create(:public_body).id]).should be_nil
+ end
+
+ it 'should not return a batch sent the same public bodies with a different title and body' do
+ InfoRequestBatch.find_existing(@info_request_batch.user,
+ 'Other title',
+ 'Other body',
+ [@first_request.public_body_id]).should be_nil
+ end
+
+ it 'should not return a batch sent to one of the same public bodies with the same title and body by
+ a different user' do
+ InfoRequestBatch.find_existing(FactoryGirl.create(:user),
+ @info_request_batch.title,
+ @info_request_batch.body,
+ [@first_request.public_body_id]).should be_nil
+ end
end