Adds spec and basic solution for dropped document bug

Author: Brian Kelly

lib/geo_combine/indexer.rb

@@ -41,7 +41,7 @@ def index(docs)
           batch << [doc, path]
         else
           total_indexed += index_batch(batch)
-          batch = []
+          batch = [[doc, path]]
         end
       end
       total_indexed += index_batch(batch) unless batch.empty?

spec/lib/geo_combine/indexer_spec.rb

@@ -96,6 +96,18 @@ def self.connection; end
       )
     end
 
+    context 'when the number of docs is greater than batch size' do
+      before do
+        stub_const('ENV', 'SOLR_BATCH_SIZE' => 10)
+      end
+
+      let(:docs) { (1..40).map { |n| [{ 'id' => n }, "path/to/record#{n}.json"] } }
+
+      it 'indexes the correct number of documents' do
+        expect(indexer.index(docs)).to eq 40
+      end
+    end
+
     it 'commits changes to solr after indexing' do
       indexer.index(docs)
       expect(solr).to have_received(:commit).once