google_maps_service 0.4.0 → 0.4.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: fb00f53441211719da0e59680a1186a109615495
-  data.tar.gz: 6075646a6a52b3cd64abce441479b1f80e967cbc
+  metadata.gz: 49c25953305b497d054ac7ae74894e1dbb904d1f
+  data.tar.gz: 749c75ddf3ca5b13fac1c5eb24cc403661edfc04
 SHA512:
-  metadata.gz: a0007fe257d673c7011097aefc8436664e3a450c7cb9a26d772b5ef72b6818ac52342614572fcf014f5f433ee0ef6ee17a7c2fed98b307d685d3b1807ca8662e
-  data.tar.gz: 24a6947fc3683cd5e58886376bc293a3c3c395952cb021435e001b2c2c4dab3131cace9191c99b999a353fd391667e92cda0b053b4b2dd06d73125a38790acda
+  metadata.gz: 38e4aa2429aecaeca3852c527cb256f10160a46984b449752846a8f1f3da3ef6fae1ec5eb2f4b0c816506ae00d7ed270570b7b6986e82d25447e21c3b88e87ec
+  data.tar.gz: 3f667efd574b1e5a200dfdc0f2a58b5976e46ae808d75cd34bc3daf5686ce9220f639ca464888255ef4f265ec53502019745b18148ecf26352252f3d722b939e

data/.travis.yml

@@ -1,5 +1,8 @@
 language: ruby
 rvm:
+  - ruby-head
   - 2.2
   - 2.1
-  - 2.0.0
+  - 2.0.0
+  - jruby-head
+  - jruby-9000

data/CHANGELOG.md

@@ -1,5 +1,10 @@
 # Changelog
 
+## 0.4.1
+
+* Support JRuby 9.0.0.0
+* Refactoring and more test coverage
+
 ## 0.4.0
 
 * Use required positional and optional named parameters (_breaking changes_)

data/Gemfile

@@ -2,3 +2,23 @@ source 'https://rubygems.org'
 
 # Specify your gem's dependencies in google_maps_service.gemspec
 gemspec
+
+group :development do
+  gem 'bundler', '~> 1.6'
+  gem 'rake', '~> 10.0'
+  gem 'rspec', '~> 3.3'
+  gem 'simplecov', '~> 0.10'
+  gem 'coveralls', '~> 0.8.2'
+  gem 'webmock', '~> 1.21'
+end
+
+platforms :ruby do
+  group :development do
+    gem 'yard', '~> 0.8'
+    gem 'redcarpet', '~> 3.2'
+  end
+end
+
+if ENV['RAILS_VERSION']
+  gem 'rails', ENV['RAILS_VERSION']
+end

data/Rakefile

@@ -1,12 +1,15 @@
-require "bundler/gem_tasks"
-require "rspec/core/rake_task"
-require "yard"
+require 'bundler/gem_tasks'
+require 'rspec/core/rake_task'
 
 RSpec::Core::RakeTask.new
 
 task :default => :spec
 task :test => :spec
 
-YARD::Rake::YardocTask.new do |t|
-  t.files   = ['lib/**/*.rb']
+unless defined?(JRUBY_VERSION)
+  require 'yard'
+
+  YARD::Rake::YardocTask.new do |t|
+    t.files   = ['lib/**/*.rb']
+  end
 end

data/google_maps_service.gemspec

@@ -20,13 +20,5 @@ Gem::Specification.new do |spec|
 
   spec.add_runtime_dependency 'multi_json', '~> 1.11'
   spec.add_runtime_dependency 'hurley', '~> 0.1'
-  spec.add_runtime_dependency 'retriable', '~> 2.0', '>= 2.0.2'
-  spec.add_development_dependency 'bundler', '~> 1.7'
-  spec.add_development_dependency 'rake', '~> 10.0'
-  spec.add_development_dependency 'yard', '~> 0.8.7.6'
-  spec.add_development_dependency 'redcarpet', '~> 3.3'
-  spec.add_development_dependency 'rspec', '~> 3.3'
-  spec.add_development_dependency 'simplecov', '~> 0.10.0'
-  spec.add_development_dependency 'coveralls', '~> 0.8.2'
-  spec.add_development_dependency 'webmock', '~> 1.21', '>= 1.21.0'
+  spec.add_runtime_dependency 'retriable', '~> 2.0'
 end

data/lib/google_maps_service.rb

@@ -42,6 +42,8 @@ module GoogleMapsService
     # @return [Object]
     attr_accessor :connection
 
+    # Configure global parameters.
+    # @yield [config]
     def configure
       yield self
       true

data/lib/google_maps_service/apis/roads.rb

@@ -126,7 +126,7 @@ module GoogleMapsService::Apis
 
       # Check response body for error status.
       #
-      # @param [Hurley::Response] body Response object.
+      # @param [Hurley::Response] response Response object.
       # @param [Hash] body Response body.
       def check_roads_body_error(response, body)
         error = body[:error]

data/lib/google_maps_service/client.rb

@@ -94,8 +94,6 @@ module GoogleMapsService
     #      connection: Hurley::HttpCache.new(HurleyExcon::Connection.new)
     #  )
     #
-    #
-    #
     # @option options [String] :key Secret key for accessing Google Maps Web Service.
     #   Can be obtained at https://developers.google.com/maps/documentation/geocoding/get-api-key#key.
     # @option options [String] :client_id Client id for using Maps API for Work services.
@@ -111,17 +109,13 @@ module GoogleMapsService
     #     By default, the default Hurley's HTTP client connection (Net::Http) will be used.
     #     See https://github.com/lostisland/hurley/blob/master/README.md#connections.
     def initialize(**options)
-      @key = options[:key] || GoogleMapsService.key
-      @client_id = options[:client_id] || GoogleMapsService.client_id
-      @client_secret = options[:client_secret] || GoogleMapsService.client_secret
-      @retry_timeout = options[:retry_timeout] || GoogleMapsService.retry_timeout || 60
-      @queries_per_second = options[:queries_per_second] || GoogleMapsService.queries_per_second
-      @request_options = options[:request_options] || GoogleMapsService.request_options
-      @ssl_options = options[:ssl_options] || GoogleMapsService.ssl_options
-      @connection = options[:connection] || GoogleMapsService.connection
-
-      #
-      initialize_qps if @queries_per_second
+      [:key, :client_id, :client_secret,
+          :retry_timeout, :queries_per_second,
+          :request_options, :ssl_options, :connection].each do |key|
+        self.instance_variable_set("@#{key}".to_sym, options[key] || GoogleMapsService.instance_variable_get("@#{key}"))
+      end
+
+      initialize_query_tickets
     end
 
     # Get the current HTTP client.
@@ -133,10 +127,12 @@ module GoogleMapsService
     protected
 
     # Initialize QPS queue. QPS queue is a "tickets" for calling API
-    def initialize_qps
-      @qps_queue = SizedQueue.new @queries_per_second
-      @queries_per_second.times do
-        @qps_queue << 0
+    def initialize_query_tickets
+      if @queries_per_second
+        @qps_queue = SizedQueue.new @queries_per_second
+        @queries_per_second.times do
+          @qps_queue << 0
+        end
       end
     end
 
@@ -144,13 +140,14 @@ module GoogleMapsService
     # @return [Hurley::Client]
     def new_client
       client = Hurley::Client.new
+      client.request_options.query_class = Hurley::Query::Flat
+      client.request_options.redirection_limit = 0
+      client.header[:user_agent] = user_agent
 
       client.connection = @connection if @connection
       @request_options.each_pair {|key, value| client.request_options[key] = value } if @request_options
       @ssl_options.each_pair {|key, value| client.ssl_options[key] = value } if @ssl_options
 
-      client.request_options.query_class = Hurley::Query::Flat
-      client.header[:user_agent] = user_agent
       client
     end
 
@@ -175,18 +172,11 @@ module GoogleMapsService
       url = base_url + generate_auth_url(path, params, accepts_client_id)
 
       Retriable.retriable timeout: @retry_timeout, on: RETRIABLE_ERRORS do |try|
-        # Get/wait the request "ticket" if QPS is configured
-        # Check for previous request time, it must be more than a second ago before calling new request
-        if @qps_queue
-          elapsed_since_earliest = Time.now - @qps_queue.pop
-          sleep(1 - elapsed_since_earliest) if elapsed_since_earliest.to_f < 1
-        end
-
         begin
+          request_query_ticket
           response = client.get url
         ensure
-          # Release request "ticket"
-          @qps_queue << Time.now if @qps_queue
+          release_query_ticket
         end
 
         return custom_response_decoder.call(response) if custom_response_decoder
@@ -194,11 +184,30 @@ module GoogleMapsService
       end
     end
 
+    # Get/wait the request "ticket" if QPS is configured.
+    # Check for previous request time, it must be more than a second ago before calling new request.
+    #
+    # @return [void]
+    def request_query_ticket
+      if @qps_queue
+        elapsed_since_earliest = Time.now - @qps_queue.pop
+        sleep(1 - elapsed_since_earliest) if elapsed_since_earliest.to_f < 1
+      end
+    end
+
+    # Release request "ticket".
+    #
+    # @return [void]
+    def release_query_ticket
+      @qps_queue << Time.now if @qps_queue
+    end
+
     # Returns the path and query string portion of the request URL,
     # first adding any necessary parameters.
     #
     # @param [String] path The path portion of the URL.
     # @param [Hash] params URL parameters.
+    # @param [Boolean] accepts_client_id Sign the request using API {#keys} instead of {#client_id}.
     #
     # @return [String]
     def generate_auth_url(path, params, accepts_client_id)
@@ -253,15 +262,15 @@ module GoogleMapsService
         raise GoogleMapsService::Error::ClientError.new(response), 'Invalid request'
       when 500..600
         raise GoogleMapsService::Error::ServerError.new(response), 'Server error'
-      else
-        raise ArgumentError, 'Invalid response status code'
       end
     end
 
     # Check response body for error status.
     #
-    # @param [Hurley::Response] body Response object.
+    # @param [Hurley::Response] response Response object.
     # @param [Hash] body Response body.
+    #
+    # @return [void]
     def check_body_error(response, body)
       case body[:status]
       when 'OK', 'ZERO_RESULTS'

data/lib/google_maps_service/errors.rb

@@ -7,6 +7,9 @@ module GoogleMapsService
       # @return [Hurley::Response]
       attr_reader :response
 
+      # Initialize error
+      #
+      # @param [Hurley::Response] response HTTP response.
       def initialize(response = nil)
         @response = response
       end

data/lib/google_maps_service/version.rb

@@ -1,6 +1,6 @@
 module GoogleMapsService
   # GoogleMapsService gem version
-  VERSION = '0.4.0'
+  VERSION = '0.4.1'
 
   # Current operating system
   # @private

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: google_maps_service
 version: !ruby/object:Gem::Version
-  version: 0.4.0
+  version: 0.4.1
 platform: ruby
 authors:
 - Edward Samuel Pasaribu
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-09-22 00:00:00.000000000 Z
+date: 2015-09-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: multi_json
@@ -45,9 +45,6 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '2.0'
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 2.0.2
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -55,127 +52,6 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '2.0'
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 2.0.2
-- !ruby/object:Gem::Dependency
-  name: bundler
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '1.7'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '1.7'
-- !ruby/object:Gem::Dependency
-  name: rake
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '10.0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '10.0'
-- !ruby/object:Gem::Dependency
-  name: yard
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 0.8.7.6
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 0.8.7.6
-- !ruby/object:Gem::Dependency
-  name: redcarpet
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '3.3'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '3.3'
-- !ruby/object:Gem::Dependency
-  name: rspec
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '3.3'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '3.3'
-- !ruby/object:Gem::Dependency
-  name: simplecov
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 0.10.0
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 0.10.0
-- !ruby/object:Gem::Dependency
-  name: coveralls
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 0.8.2
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 0.8.2
-- !ruby/object:Gem::Dependency
-  name: webmock
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '1.21'
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 1.21.0
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '1.21'
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 1.21.0
 description: 
 email:
 - edwardsamuel92@gmail.com