Only execute update if document attributes is not empty (#862)

* Only execute update if document attributes is not empty

* Adds test for skipping empty elasticsearch update

Author: jonbwhite

elasticsearch-model/lib/elasticsearch/model/indexing.rb

@@ -409,7 +409,7 @@ def update_document(options={})
                 type:  document_type,
                 id:    self.id,
                 body:  { doc: attributes } }.merge(options)
-            )
+            ) unless attributes.empty?
           else
             index_document(options)
           end

elasticsearch-model/spec/elasticsearch/model/indexing_spec.rb

@@ -423,10 +423,10 @@ def changes
       context 'when changes are present' do
 
         before do
-          expect(instance).to receive(:client).and_return(client)
-          expect(instance).to receive(:index_name).and_return('foo')
-          expect(instance).to receive(:document_type).and_return('bar')
-          expect(instance).to receive(:id).and_return('1')
+          allow(instance).to receive(:client).and_return(client)
+          allow(instance).to receive(:index_name).and_return('foo')
+          allow(instance).to receive(:document_type).and_return('bar')
+          allow(instance).to receive(:id).and_return('1')
         end
 
         context 'when the changes are included in the as_indexed_json representation' do
@@ -457,6 +457,21 @@ def changes
           end
         end
 
+        context 'when none of the changes are included in the as_indexed_json representation' do
+
+          let(:instance) do
+            DummyIndexingModelWithCallbacksAndCustomAsIndexedJson.new
+          end
+
+          before do
+            instance.instance_variable_set(:@__changed_model_attributes, {'bar' => 'D' })
+          end
+
+          it 'does not update the document' do
+            expect(instance.update_document).to_not be(true)
+          end
+        end
+
         context 'when there are partial updates' do
 
           let(:instance) do