Merge pull request #519 from guitsaru/optimize-guide-index

Eager load gardens on guide reindex.

app/jobs/reindex_guides_job.rb

@@ -2,6 +2,8 @@ class ReindexGuidesJob < ActiveJob::Base
   queue_as :default
 
   def perform
-    Guide.reindex
+    Guide.desc(:popularity_score).each do |guide|
+      guide.reindex_async
+    end
   end
 end

app/models/guide.rb

@@ -76,7 +76,7 @@ def compatibilities
 
     @compatibilities = []
 
-    User.each do |user|
+    User.includes(:gardens).each do |user|
       @compatibilities << {
         user_id: user.id.to_s, score: compatibility_score(user).to_i
       }

spec/jobs/reindex_guides_job_spec.rb

@@ -2,7 +2,9 @@
 
 describe ReindexGuidesJob do
   it 'reindexes guides' do
-    expect(Guide).to receive(:reindex)
+    FactoryGirl.create(:guide)
+
+    expect_any_instance_of(Guide).to receive(:reindex_async)
 
     ReindexGuidesJob.new.perform
   end

spec/models/garden_crop_spec.rb

@@ -14,7 +14,9 @@
   end
 
   it 'reindexes guides' do
-    expect(Guide).to receive(:reindex)
+    FactoryGirl.create(:guide)
+
+    expect_any_instance_of(Guide).to receive(:reindex_async)
 
     FactoryGirl.create(:garden, user: FactoryGirl.create(:user))
   end