Geocode building addresses as a background job

Author: rsmithlal

app/jobs/better_together/geography/geocoding_job.rb

@@ -0,0 +1,14 @@
+# frozen_string_literal: true
+
+module BetterTogether
+  module Geography
+    class GeocodingJob < ApplicationJob # rubocop:todo Style/Documentation
+      queue_as :geocoding
+
+      def perform(geocodable)
+        geocodable.geocode
+        geocodable.save
+      end
+    end
+  end
+end

app/models/better_together/infrastructure/building.rb

@@ -32,7 +32,8 @@ class Building < Structure
 
       after_create :ensure_floor
 
-      after_validation :geocode, if: ->(obj) { obj.address.present? and (obj.address_changed? || obj.geocoded?) }
+      after_create :schedule_address_geocoding
+      after_update :schedule_address_geocoding
 
       translates :name
       translates :description, backend: :action_text
@@ -57,6 +58,15 @@ def ensure_floor
         floors.create(name: 'Ground')
       end
 
+      def schedule_address_geocoding
+        return unless should_geocode?
+        BetterTogether::Geography::GeocodingJob.perform_later(self)
+      end
+
+      def should_geocode?
+        address.present? and (address_changed? or !geocoded?)
+      end
+
       def select_option_title
         "#{name} (#{slug})"
       end

spec/dummy/config/sidekiq.yml

@@ -3,6 +3,8 @@ default: &default
   queues:
     - default
     - mailers
+    - es_indexing
+    - geocoding
 
 development:
   <<: *default