google_distance_matrix 0.4.0 → 0.5.0

data/lib/google_distance_matrix/matrix.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module GoogleDistanceMatrix
   # Public: Represents a distance matrix.
   #
@@ -25,23 +27,25 @@ module GoogleDistanceMatrix
   #     config.mode = "walking"
   #   end
   #
-  # You can set default configuration by doing: GoogleDistanceMatrix.configure_defaults { |c| c.sensor = true }
+  # You can set default configuration by doing:
+  # GoogleDistanceMatrix.configure_defaults { |c| c.sensor = true }
   #
   #
   # Query API and get the matrix back
   #
   #   matrix.data   # Returns a two dimensional array.
   #                 # Rows are ordered according to the values in the origins.
-  #                 # Each row corresponds to an origin, and each element within that row corresponds to
+  #                 # Each row corresponds to an origin, and each element
+  #                 # within that row corresponds to
   #                 # a pairing of the origin with a destination.
   #
   #
   class Matrix
     include ActiveModel::Validations
 
-    validates :origins, length: {minimum: 1, too_short: "must have at least one origin"}
-    validates :destinations, length: {minimum: 1, too_short: "must have at least one destination"}
-    validate { errors.add(:configuration, "is invalid") if configuration.invalid? }
+    validates :origins, length: { minimum: 1, too_short: 'must have at least one origin' }
+    validates :destinations, length: { minimum: 1, too_short: 'must have at least one destination' }
+    validate { errors.add(:configuration, 'is invalid') if configuration.invalid? }
 
     attr_reader :origins, :destinations, :configuration
 
@@ -53,6 +57,7 @@ module GoogleDistanceMatrix
       @configuration = attributes[:configuration] || GoogleDistanceMatrix.default_configuration.dup
     end
 
+    delegate :sensitive_url, :filtered_url, to: :url_builder
 
     delegate :route_for,  :routes_for,  to: :routes_finder
     delegate :route_for!, :routes_for!, to: :routes_finder
@@ -61,7 +66,6 @@ module GoogleDistanceMatrix
     delegate :shortest_route_by_duration_in_traffic_to,  to: :routes_finder
     delegate :shortest_route_by_duration_in_traffic_to!, to: :routes_finder
 
-
     # Public: The data for this matrix.
     #
     # Returns a two dimensional array, the matrix's data
@@ -83,40 +87,59 @@ module GoogleDistanceMatrix
       @data.present?
     end
 
-
-
     def configure
       yield configuration
     end
 
-    def url
-      UrlBuilder.new(self).url
-    end
-
-
     def inspect
       attributes = %w[origins destinations]
-      attributes << "data" if loaded?
+      attributes << 'data' if loaded?
       inspection = attributes.map { |a| "#{a}: #{public_send(a).inspect}" }.join ', '
 
       "#<#{self.class} #{inspection}>"
     end
 
-
     private
 
+    def url_builder
+      # We do not keep url builder as an instance variable as origins and destinations
+      # may be added after URL is being built for the first time. We should either
+      # make our matrix immutable or reset if origins/destinations are added after data (and
+      # the url) first being built and data fetched.
+      UrlBuilder.new self
+    end
+
     def routes_finder
       @routes_finder ||= RoutesFinder.new self
     end
 
     def load_matrix
-      parsed = JSON.parse client.get(url, instrumentation: {elements: origins.length * destinations.length}).body
+      body = client.get(
+        sensitive_url,
+        instrumentation: instrumentation_for_api_request,
+        configuration: configuration
+      ).body
+
+      parsed = JSON.parse(body)
+      create_route_objects_for_parsed_data parsed
+    end
+
+    def instrumentation_for_api_request
+      {
+        elements: origins.length * destinations.length,
+        sensitive_url: sensitive_url,
+        filtered_url: filtered_url
+      }
+    end
 
-      parsed["rows"].each_with_index.map do |row, origin_index|
+    def create_route_objects_for_parsed_data(parsed)
+      parsed['rows'].each_with_index.map do |row, origin_index|
         origin = origins[origin_index]
 
-        row["elements"].each_with_index.map do |element, destination_index|
-          route_attributes = element.merge(origin: origin, destination: destinations[destination_index])
+        row['elements'].each_with_index.map do |element, destination_index|
+          route_attributes = element.merge(
+            origin: origin, destination: destinations[destination_index]
+          )
           Route.new route_attributes
         end
       end
@@ -133,9 +156,9 @@ module GoogleDistanceMatrix
     end
 
     def clear_from_cache!
-      if configuration.cache
-        configuration.cache.delete ClientCache.key(url)
-      end
+      return if configuration.cache.nil?
+
+      configuration.cache.delete ClientCache.key(sensitive_url, configuration)
     end
   end
 end

data/lib/google_distance_matrix/place.rb

@@ -1,20 +1,22 @@
-# Public: Represents a place and knows how to convert it to param.
-#
-# Examples
-#
-#   GoogleDistanceMatrix::Place.new address: "My address"
-#   GoogleDistanceMatrix::Place.new lat: 1, lng: 3
-#
-#   You may also build places by other objects responding to lat and lng or address.
-#   If your object responds to all of the attributes we'll use lat and lng as data
-#   for the Place.
-#
-#   GoogleDistanceMatrix::Place.new object
+# frozen_string_literal: true
+
 module GoogleDistanceMatrix
+  # Public: Represents a place and knows how to convert it to param.
+  #
+  # Examples
+  #
+  #   GoogleDistanceMatrix::Place.new address: "My address"
+  #   GoogleDistanceMatrix::Place.new lat: 1, lng: 3
+  #
+  #   You may also build places by other objects responding to lat and lng or address.
+  #   If your object responds to all of the attributes we'll use lat and lng as data
+  #   for the Place.
+  #
+  #   GoogleDistanceMatrix::Place.new object
   class Place
-    ATTRIBUTES = %w[address lat lng]
+    ATTRIBUTES = %w[address lat lng].freeze
 
-    attr_reader *ATTRIBUTES, :extracted_attributes_from
+    attr_reader(*ATTRIBUTES, :extracted_attributes_from)
 
     def initialize(attributes_or_object)
       if respond_to_needed_attributes? attributes_or_object
@@ -22,7 +24,7 @@ module GoogleDistanceMatrix
       elsif attributes_or_object.is_a? Hash
         assign_attributes attributes_or_object
       else
-        fail ArgumentError, "Must be either hash or object responding to lat, lng or address. "
+        raise ArgumentError, 'Must be either hash or object responding to lat, lng or address. '
       end
 
       validate_attributes
@@ -33,7 +35,6 @@ module GoogleDistanceMatrix
       address.present? ? address : lat_lng(options[:lat_lng_scale]).join(',')
     end
 
-
     def eql?(other)
       if address.present?
         address == other.address
@@ -58,7 +59,9 @@ module GoogleDistanceMatrix
     end
 
     def inspect
-      inspection = (ATTRIBUTES | [:extracted_attributes_from]).reject { |a| public_send(a).blank? }.map { |a| "#{a}: #{public_send(a).inspect}" }.join ', '
+      inspection =  (ATTRIBUTES | [:extracted_attributes_from])
+                    .reject { |a| public_send(a).blank? }
+                    .map { |a| "#{a}: #{public_send(a).inspect}" }.join ', '
 
       "#<#{self.class} #{inspection}>"
     end
@@ -66,19 +69,17 @@ module GoogleDistanceMatrix
     private
 
     def respond_to_needed_attributes?(object)
-      (object.respond_to?(:lat) &&  object.respond_to?(:lng)) || object.respond_to?(:address)
+      (object.respond_to?(:lat) && object.respond_to?(:lng)) || object.respond_to?(:address)
     end
 
     def extract_and_assign_attributes_from_object(object)
-      attrs =  Hash[ATTRIBUTES.map do |attr_name|
+      attrs = Hash[ATTRIBUTES.map do |attr_name|
         if object.respond_to? attr_name
           [attr_name, object.public_send(attr_name)]
         end
       end.compact]
 
-      if attrs.has_key?('lat') || attrs.has_key?('lng')
-        attrs.delete 'address'
-      end
+      attrs.delete 'address' if attrs.key?('lat') || attrs.key?('lng')
 
       @extracted_attributes_from = object
       assign_attributes attrs
@@ -92,14 +93,20 @@ module GoogleDistanceMatrix
       @lng = attributes[:lng]
     end
 
+    # rubocop:disable Metrics/AbcSize
+    # rubocop:disable Metrics/CyclomaticComplexity
+    # rubocop:disable Style/GuardClause
     def validate_attributes
-     unless address.present? || (lat.present? && lng.present?)
-        fail ArgumentError, "Must provide an address, or lat and lng."
+      unless address.present? || (lat.present? && lng.present?)
+        raise ArgumentError, 'Must provide an address, or lat and lng.'
       end
 
       if address.present? && lat.present? && lng.present?
-        fail ArgumentError, "Cannot provide address, lat and lng."
+        raise ArgumentError, 'Cannot provide address, lat and lng.'
       end
     end
+    # rubocop:enable Metrics/AbcSize
+    # rubocop:enable Metrics/CyclomaticComplexity
+    # rubocop:enable Style/GuardClause
   end
 end

data/lib/google_distance_matrix/places.rb

@@ -1,4 +1,7 @@
+# frozen_string_literal: true
+
 module GoogleDistanceMatrix
+  # Represents a collection of places
   class Places
     include Enumerable
 
@@ -7,10 +10,9 @@ module GoogleDistanceMatrix
       concat Array.wrap(places)
     end
 
-
     delegate :each, :[], :length, :index, :pop, :shift, :delete_at, :compact, :inspect, to: :places
 
-    [:<<, :push, :unshift].each do |method|
+    %i[<< push unshift].each do |method|
       define_method method do |*args|
         args = ensure_args_are_places args
 
@@ -22,10 +24,9 @@ module GoogleDistanceMatrix
     end
 
     def concat(other)
-      push *other
+      push(*other)
     end
 
-
     private
 
     attr_reader :places

data/lib/google_distance_matrix/polyline_encoder/delta.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module GoogleDistanceMatrix
   class PolylineEncoder
     # Calculates deltas between lat_lng values, internal helper class for PolylineEncoder.
@@ -20,7 +22,6 @@ module GoogleDistanceMatrix
         calculate_deltas rounded
       end
 
-
       private
 
       def round_to_precision(array_of_lat_lng_pairs)
@@ -42,7 +43,8 @@ module GoogleDistanceMatrix
           deltas << lat - delta_lat
           deltas << lng - delta_lng
 
-          delta_lat, delta_lng = lat, lng
+          delta_lat = lat
+          delta_lng = lng
         end
 
         deltas

data/lib/google_distance_matrix/polyline_encoder/value_encoder.rb

@@ -1,3 +1,6 @@
+# frozen_string_literal: true
+
+# rubocop:disable Style/NumericPredicate
 module GoogleDistanceMatrix
   class PolylineEncoder
     # Encodes a single value, like 17998321, in to encoded polyline value,
@@ -9,6 +12,8 @@ module GoogleDistanceMatrix
     #
     # @see GoogleDistanceMatrix::PolylineEncoder
     class ValueEncoder
+      # rubocop:disable Metrics/MethodLength
+      # rubocop:disable Metrics/AbcSize
       def encode(value)
         negative = value < 0
         value = value.abs
@@ -27,7 +32,7 @@ module GoogleDistanceMatrix
         # Right shift bits to get rid of the ones we just put on the array.
         # Bits will end up in reverse order.
         chunks_of_5_bits = []
-        while value > 0 do
+        while value > 0
           chunks_of_5_bits.push(value & 0x1f)
           value >>= 5
         end
@@ -44,6 +49,8 @@ module GoogleDistanceMatrix
         # step 11: Convert to ASCII
         chunks_of_5_bits.map(&:chr)
       end
+      # rubocop:enable Metrics/MethodLength
+      # rubocop:enable Metrics/AbcSize
 
       private
 
@@ -52,9 +59,9 @@ module GoogleDistanceMatrix
       # Example of usage
       #   p d 17998321 # => "00000001 00010010 10100001 11110001"
       def d(v, bits = 32, chunk_size = 8)
-        (bits - 1).downto(0).
-          map { |n| v[n] }.
-          each_slice(chunk_size).map { |chunk| chunk.join }.join ' '
+        (bits - 1).downto(0)
+                  .map { |n| v[n] }
+                  .each_slice(chunk_size).map(&:join).join ' '
       end
     end
   end

data/lib/google_distance_matrix/polyline_encoder.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative 'polyline_encoder/delta'
 require_relative 'polyline_encoder/value_encoder'
 
@@ -7,7 +9,6 @@ module GoogleDistanceMatrix
   #
   # See https://developers.google.com/maps/documentation/utilities/polylinealgorithm
   class PolylineEncoder
-
     # Encodes a set of lat/lng pairs
     #
     # Example
@@ -16,7 +17,6 @@ module GoogleDistanceMatrix
       new(array_of_lat_lng_pairs).encode
     end
 
-
     # Initialize a new encoder
     #
     # Arguments

data/lib/google_distance_matrix/railtie.rb

@@ -1,6 +1,9 @@
+# frozen_string_literal: true
+
 module GoogleDistanceMatrix
+  # Class integrating with rails
   class Railtie < Rails::Railtie
-    initializer "google_distance_matrix.logger_setup" do
+    initializer 'google_distance_matrix.logger_setup' do
       GoogleDistanceMatrix.configure_defaults do |config|
         config.logger = Rails.logger
       end

data/lib/google_distance_matrix/route.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module GoogleDistanceMatrix
   # Public: Thin wrapper class for an element in the matrix.
   #
@@ -15,10 +17,9 @@ module GoogleDistanceMatrix
       duration_in_traffic_text duration_in_traffic_in_seconds
     ].freeze
 
-    attr_reader *ATTRIBUTES
-
-    delegate *(STATUSES.map { |s| s + '?' }), to: :status, allow_nil: true
+    attr_reader(*ATTRIBUTES)
 
+    delegate(*(STATUSES.map { |s| s + '?' }), to: :status, allow_nil: true)
 
     def initialize(attributes = {})
       attributes = attributes.with_indifferent_access
@@ -28,25 +29,31 @@ module GoogleDistanceMatrix
 
       @status = ActiveSupport::StringInquirer.new attributes[:status].downcase
 
-      if ok?
-        @distance_text = attributes[:distance][:text]
-        @distance_in_meters = attributes[:distance][:value]
-        @duration_text = attributes[:duration][:text]
-        @duration_in_seconds = attributes[:duration][:value]
+      assign attributes if ok?
+    end
 
-        if attributes.key? :duration_in_traffic
-          @duration_in_traffic_text = attributes[:duration_in_traffic][:text]
-          @duration_in_traffic_in_seconds = attributes[:duration_in_traffic][:value]
-        end
+    def inspect
+      attrs = ATTRIBUTES.reject do |a|
+        public_send(a).blank?
       end
+
+      inspection = attrs.map { |a| "#{a}: #{public_send(a).inspect}" }.join ', '
+
+      "#<#{self.class} #{inspection}>"
     end
 
+    private
 
+    def assign(attributes)
+      @distance_text = attributes[:distance][:text]
+      @distance_in_meters = attributes[:distance][:value]
+      @duration_text = attributes[:duration][:text]
+      @duration_in_seconds = attributes[:duration][:value]
 
-    def inspect
-      inspection = ATTRIBUTES.reject { |a| public_send(a).blank? }.map { |a| "#{a}: #{public_send(a).inspect}" }.join ', '
+      return unless  attributes.key? :duration_in_traffic
 
-      "#<#{self.class} #{inspection}>"
+      @duration_in_traffic_text = attributes[:duration_in_traffic][:text]
+      @duration_in_traffic_in_seconds = attributes[:duration_in_traffic][:value]
     end
   end
 end

data/lib/google_distance_matrix/routes_finder.rb

@@ -1,18 +1,17 @@
+# frozen_string_literal: true
+
 module GoogleDistanceMatrix
   # Public: Has logic for doing finder operations on a matrix.
+  #
+  # rubocop:disable Metrics/ClassLength
   class RoutesFinder
-
     attr_reader :matrix
     delegate :data, :origins, :destinations, :configuration, to: :matrix
 
-
     def initialize(matrix)
       @matrix = matrix
-    end
+    end # Public: Finds routes for given place.
 
-
-
-    # Public: Finds routes for given place.
     #
     # place   -  Either an origin or destination, or an object which you built the place from
     #
@@ -25,7 +24,7 @@ module GoogleDistanceMatrix
       elsif destinations.include? place
         routes_for_destination place
       else
-        fail ArgumentError, "Given place not an origin nor destination."
+        raise ArgumentError, 'Given place not an origin nor destination.'
       end
     end
 
@@ -41,11 +40,11 @@ module GoogleDistanceMatrix
       end
     end
 
-
     # Public: Finds a route for you based on one origin and destination
     #
     # origin        - A place representing the origin, or an object which you built the origin from
-    # destination   - A place representing the destination, or an object which you built the destination from
+    # destination   - A place representing the destination, or an object which you built the
+    #                   destination from
     #
     # A Route for given origin and destination
     def route_for(options = {})
@@ -55,7 +54,7 @@ module GoogleDistanceMatrix
       destination = ensure_place options[:destination]
 
       if origin.nil? || destination.nil?
-        fail ArgumentError, "Must provide origin and destination"
+        raise ArgumentError, 'Must provide origin and destination'
       end
 
       routes_for(origin).detect { |route| route.destination == destination }
@@ -71,7 +70,6 @@ module GoogleDistanceMatrix
       end
     end
 
-
     # Public: Finds shortes route by distance to a place.
     #
     # place - The place, or object place was built from, you want the shortest route to
@@ -79,7 +77,7 @@ module GoogleDistanceMatrix
     # Returns shortest route, or nil if no routes had status ok
     def shortest_route_by_distance_to(place_or_object_place_was_built_from)
       routes = routes_for place_or_object_place_was_built_from
-      select_ok_routes(routes).min_by &:distance_in_meters
+      select_ok_routes(routes).min_by(&:distance_in_meters)
     end
 
     # Public: Finds shortes route by distance to a place.
@@ -88,7 +86,7 @@ module GoogleDistanceMatrix
     #
     # Returns shortest route, fails if any of the routes are not ok
     def shortest_route_by_distance_to!(place_or_object_place_was_built_from)
-      routes_for!(place_or_object_place_was_built_from).min_by &:distance_in_meters
+      routes_for!(place_or_object_place_was_built_from).min_by(&:distance_in_meters)
     end
 
     # Public: Finds shortes route by duration to a place.
@@ -98,7 +96,7 @@ module GoogleDistanceMatrix
     # Returns shortest route, or nil if no routes had status ok
     def shortest_route_by_duration_to(place_or_object_place_was_built_from)
       routes = routes_for place_or_object_place_was_built_from
-      select_ok_routes(routes).min_by &:duration_in_seconds
+      select_ok_routes(routes).min_by(&:duration_in_seconds)
     end
 
     # Public: Finds shortes route by duration to a place.
@@ -107,7 +105,7 @@ module GoogleDistanceMatrix
     #
     # Returns shortest route, fails if any of the routes are not ok
     def shortest_route_by_duration_to!(place_or_object_place_was_built_from)
-      routes_for!(place_or_object_place_was_built_from).min_by &:duration_in_seconds
+      routes_for!(place_or_object_place_was_built_from).min_by(&:duration_in_seconds)
     end
 
     # Public: Finds shortes route by duration in traffic to a place.
@@ -122,7 +120,7 @@ module GoogleDistanceMatrix
       ensure_driving_and_departure_time_or_fail!
 
       routes = routes_for place_or_object_place_was_built_from
-      select_ok_routes(routes).min_by &:duration_in_traffic_in_seconds
+      select_ok_routes(routes).min_by(&:duration_in_traffic_in_seconds)
     end
 
     # Public: Finds shortes route by duration in traffic to a place.
@@ -136,11 +134,9 @@ module GoogleDistanceMatrix
     def shortest_route_by_duration_in_traffic_to!(place_or_object_place_was_built_from)
       ensure_driving_and_departure_time_or_fail!
 
-      routes_for!(place_or_object_place_was_built_from).min_by &:duration_in_traffic_in_seconds
+      routes_for!(place_or_object_place_was_built_from).min_by(&:duration_in_traffic_in_seconds)
     end
 
-
-
     private
 
     def ensure_place(object)
@@ -148,27 +144,27 @@ module GoogleDistanceMatrix
         object
       else
         find_place_for_object(origins, object) ||
-        find_place_for_object(destinations, object)
+          find_place_for_object(destinations, object)
       end
     end
 
     def find_place_for_object(collection, object)
       collection.detect do |place|
         place.extracted_attributes_from.present? &&
-        place.extracted_attributes_from == object
+          place.extracted_attributes_from == object
       end
     end
 
     def routes_for_origin(origin)
       index = origins.index origin
-      fail ArgumentError, "Given origin is not i matrix."if index.nil?
+      raise ArgumentError, 'Given origin is not i matrix.' if index.nil?
 
       data[index]
     end
 
     def routes_for_destination(destination)
       index = destinations.index destination
-      fail ArgumentError, "Given destination is not i matrix." if index.nil?
+      raise ArgumentError, 'Given destination is not i matrix.' if index.nil?
 
       [].tap do |routes|
         data.each { |row| routes << row[index] }
@@ -180,24 +176,24 @@ module GoogleDistanceMatrix
       destination_index = destinations.index destination
 
       if origin_index.nil? || destination_index.nil?
-        fail ArgumentError, "Given origin or destination is not i matrix."
+        raise ArgumentError, 'Given origin or destination is not i matrix.'
       end
 
       [data[origin_index][destination_index]]
     end
 
     def fail_unless_route_is_ok(route)
-      fail InvalidRoute.new route unless route.ok?
+      raise InvalidRoute, route unless route.ok?
     end
 
     def select_ok_routes(routes)
-      routes.select &:ok?
+      routes.select(&:ok?)
     end
 
     def ensure_driving_and_departure_time_or_fail!
-      if configuration.mode != 'driving' || configuration.departure_time.nil?
-        fail InvalidQuery, "Matrix must be in mode driving and a departure_time must be set"
-      end
+      return if configuration.mode == 'driving' && configuration.departure_time.present?
+
+      raise InvalidQuery, 'Matrix must be in mode driving and a departure_time must be set'
     end
   end
 end