Add InfoRequest#move_to_public_body

2 files changed, 116 insertions, 0 deletions

diff --git a/app/models/info_request.rb b/app/models/info_request.rb
index f9f6cffa9..8f464e786 100644
--- a/app/models/info_request.rb
+++ b/app/models/info_request.rb
@@ -1359,6 +1359,30 @@ public
                 order('last_event_time')
     end
 
+    def move_to_public_body(destination_public_body, opts = {})
+        old_body = public_body
+        editor = opts.fetch(:editor)
+
+        attrs = { :public_body => destination_public_body }
+
+        if destination_public_body
+          attrs.merge!({
+            :law_used => destination_public_body.law_only_short.downcase
+          })
+        end
+
+        if update_attributes(attrs)
+            log_event('move_request',
+                      :editor => editor,
+                      :public_body_url_name => public_body.url_name,
+                      :old_public_body_url_name => old_body.url_name)
+
+            reindex_request_events
+
+            public_body
+        end
+    end
+
     private
 
     def set_defaults
diff --git a/spec/models/info_request_spec.rb b/spec/models/info_request_spec.rb
index d8c0c59b1..d18b254fb 100644
--- a/spec/models/info_request_spec.rb
+++ b/spec/models/info_request_spec.rb
@@ -28,6 +28,98 @@ require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
 
 describe InfoRequest do
 
+    describe :move_to_public_body do
+
+        context 'with no options' do
+
+          it 'requires an :editor option' do
+              request = FactoryGirl.create(:info_request)
+              new_body = FactoryGirl.create(:public_body)
+              expect {
+                  request.move_to_public_body(new_body)
+              }.to raise_error IndexError
+          end
+
+        end
+
+        context 'with the :editor option' do
+
+          it 'moves the info request to the new public body' do
+              request = FactoryGirl.create(:info_request)
+              new_body = FactoryGirl.create(:public_body)
+              user = FactoryGirl.create(:user)
+              request.move_to_public_body(new_body, :editor => user)
+              request.reload
+              expect(request.public_body).to eq(new_body)
+          end
+
+          it 'logs the move' do
+              request = FactoryGirl.create(:info_request)
+              old_body = request.public_body
+              new_body = FactoryGirl.create(:public_body)
+              user = FactoryGirl.create(:user)
+              request.move_to_public_body(new_body, :editor => user)
+              request.reload
+              event = request.info_request_events.last
+
+              expect(event.event_type).to eq('move_request')
+              expect(event.params[:editor]).to eq(user)
+              expect(event.params[:public_body_url_name]).to eq(new_body.url_name)
+              expect(event.params[:old_public_body_url_name]).to eq(old_body.url_name)
+          end
+
+          it 'updates the law_used to the new body law' do
+              request = FactoryGirl.create(:info_request)
+              new_body = FactoryGirl.create(:public_body, :tag_string => 'eir_only')
+              user = FactoryGirl.create(:user)
+              request.move_to_public_body(new_body, :editor => user)
+              request.reload
+              expect(request.law_used).to eq('eir')
+          end
+
+          it 'returns the new public body' do
+              request = FactoryGirl.create(:info_request)
+              new_body = FactoryGirl.create(:public_body)
+              user = FactoryGirl.create(:user)
+              expect(request.move_to_public_body(new_body, :editor => user)).to eq(new_body)
+          end
+
+          it 'retains the existing body if the new body does not exist' do
+              request = FactoryGirl.create(:info_request)
+              user = FactoryGirl.create(:user)
+              existing_body = request.public_body
+              request.move_to_public_body(nil, :editor => user)
+              request.reload
+              expect(request.public_body).to eq(existing_body)
+          end
+
+          it 'returns nil if the body cannot be updated' do
+              request = FactoryGirl.create(:info_request)
+              user = FactoryGirl.create(:user)
+              expect(request.move_to_public_body(nil, :editor => user)).to eq(nil)
+          end
+
+          it 'reindexes the info request' do
+              request = FactoryGirl.create(:info_request)
+              new_body = FactoryGirl.create(:public_body)
+              user = FactoryGirl.create(:user)
+              reindex_job = ActsAsXapian::ActsAsXapianJob.
+                where(:model => 'InfoRequestEvent').
+                  delete_all
+
+              request.move_to_public_body(new_body, :editor => user)
+              request.reload
+
+              reindex_job = ActsAsXapian::ActsAsXapianJob.
+                where(:model => 'InfoRequestEvent').
+                  last
+              expect(reindex_job.model_id).to eq(request.info_request_events.last.id)
+          end
+
+        end
+
+    end
+
     describe 'when validating' do
 
         it 'should accept a summary with ascii characters' do