api-ai-ruby 1.2.1 → 1.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 8d8eb3cb912b9b4ea97e7cf1e228e222fe55c79b
-  data.tar.gz: a841d4b046350758e633366b427f5a04c501d63d
+  metadata.gz: 522e5912efc69543bb84c18c85e7be17a3858640
+  data.tar.gz: bae85fbe0f16379fef9ceeb70a0b2035a85f26d8
 SHA512:
-  metadata.gz: a00820e65bed040550cbc68e87219b03a1974d8ddcbf61e216a721879d63d6cf5cc45a4ffdd0cf6ebd742da7a9a5171d45afb02abcc795e397bd343ecbe9fe8c
-  data.tar.gz: 2ee7eecfa121825412a5de7f9030c02c86986ea7c36dc89cdd7f4fcbf774eca85c5986b3043ea7c69bf745e3654f55a1b0c0095cfc0ef47675d0e091e7b67b79
+  metadata.gz: 33292054e5c042f63fa248b53bedd68d2d009e1eeb16d79aae63cf798829f9aeb500b031189a530887c2be37c7827867269270630866a56f7eba1f4cff823628
+  data.tar.gz: 1358d1cf73154909c6be7e903828e349fd9bad9342b010344ce92f0d2c825672dd6c867bb4edb86c1a9e23ceff665cc3f043f5dedd4b74ba74020f90a2ae2777

data/README.md

@@ -140,6 +140,20 @@ uer.retrieve('contacts') # will return current state of user entity
 uer.delete('contacts') # will remove user entities for given session    
        
 ```
+#Timeouts
+**ApiAiRuby::Client** uses the [http gem](https://github.com/httprb/http) under the hood.  You can use ```timeout_options``` on the client to set these.
+```ruby
+ApiAiRuby::Client.new(
+    client_access_token: 'YOUR_ACCESS_TOKEN',
+    api_lang: 'FR',
+    api_base_url: 'http://example.com/v1/',
+    api_version: 'YYYYMMDD',
+    api_session_id: 'some_uuid_or_whatever',
+    timeout_options: [:global, { write: 1, connect: 1, read: 1 }]
+)
+```
+
+Please see the [httprb wiki on timeouts](https://github.com/httprb/http/wiki/Timeouts) for more information.
 
 
 #Error handling
@@ -148,6 +162,7 @@ uer.delete('contacts') # will remove user entities for given session
 
 #Changelog
 
+* 1.2.2 - added configurable timeouts for requests (thanks [bramski](https://github.com/bramski))
 * 1.2.1 - fixed UTF-8 in text-requests
 * 1.2.0 - added configurable session_id and full userEntities support
 * 1.1.4 - removed unused dependency and updated default API version 

data/lib/api-ai-ruby/client.rb

@@ -2,7 +2,7 @@ require 'securerandom'
 
 module ApiAiRuby
   class Client
-    attr_accessor :client_access_token, :subscription_key
+    attr_accessor :client_access_token, :subscription_key, :timeout_options
     attr_writer :user_agent, :api_version, :api_lang, :api_base_url, :api_session_id
 
     # Initializes a new Client object

data/lib/api-ai-ruby/constants.rb

@@ -1,6 +1,6 @@
 module ApiAiRuby
   class Constants
-    VERSION = '1.2.1'
+    VERSION = '1.2.2'
     DEFAULT_BASE_URL = 'https://api.api.ai/v1/'
     DEFAULT_API_VERSION = '20150910'
     DEFAULT_CLIENT_LANG = 'en'

data/lib/api-ai-ruby/request/request_query.rb

@@ -3,7 +3,7 @@ require 'http'
 module ApiAiRuby
   class RequestQuery
 
-    attr_accessor :client, :headers, :options,  :request_method, :uri
+    attr_accessor :client, :headers, :options, :request_method, :uri
 
     # @param client [ApiAiRuby::Client]
     # @param options [Hash]
@@ -18,20 +18,21 @@ module ApiAiRuby
       @headers = {
           Authorization: 'Bearer ' + client.client_access_token,
       }
+      @timeout_options = client.timeout_options || options[:timeout_options]
     end
 
     # @return [Array, Hash]
     def perform
-
       if @options.has_key?(:voiceData)
         options_key = :form
       else
         options_key = (@request_method === :get) ? :params : :json
       end
 
-      response = HTTP.with(@headers).public_send(@request_method, @uri.to_s, options_key => @options)
+      request = HTTP.with(@headers)
+      request = request.timeout(*@timeout_options) if @timeout_options
+      response = request.public_send(@request_method, @uri.to_s, options_key => @options)
       response_body = symbolize_keys!(response.parse)
-      response_headers = response.headers
       fail_or_return_response_body(response.code, response_body)
     end
 

data/spec/api-ai-ruby/client_spec.rb

@@ -48,13 +48,15 @@ describe ApiAiRuby::Client do
           api_lang: 'RU',
           api_base_url: 'http://localhost',
           api_version: '1234',
-          api_session_id: '555'
+          api_session_id: '555',
+          timeout_options: [:global, { write: 1, connect: 1, read: 1}]
       )
 
       expect(client.api_base_url).to eq 'http://localhost'
       expect(client.api_version).to eq '1234'
       expect(client.api_lang).to eq 'RU'
       expect(client.api_session_id).to eq '555'
+      expect(client.timeout_options).to eq [:global, { write: 1, connect: 1, read: 1}]
     end
 
 

data/spec/api-ai-ruby/text_request_spec.rb

@@ -0,0 +1,59 @@
+require 'helper'
+
+describe ApiAiRuby::TextRequest do
+  let(:client) do
+    ApiAiRuby::Client.new(
+      client_access_token: 'CS',
+      api_lang: 'EN',
+      api_session_id: '555',
+      timeout_options: [:global, { write: 1, connect: 1, read: 1 }]
+    )
+  end
+
+  subject(:text_request) { described_class.new(client, options) }
+  let(:options) { { query: "hello" } }
+
+  around do |test|
+    begin
+      WebMock.disable_net_connect!(allow: 'coveralls.io')
+      test.run
+    ensure
+      WebMock.allow_net_connect!
+    end
+  end
+
+  let(:response_headers) do
+    { "Content-Type" => "application/json;charset=UTF-8", "Content-Length" => "437", "Connection" => "close", "Access-Control-Allow-Credentials" => "true", "Cache-Control" => "no-cache=\"set-cookie\"", "Date" => "Wed, 12 Oct 2016 20:07:54 GMT", "Server" => "nginx/1.9.7", "Set-Cookie" => "AWSELB=9D5B4D210CCFFAF1BE1E0CD7C7E6FCBD7B46140CAAF64A202A005B9079598B549F7A5EC269DD0FF88508DA57410EFC7882B7860453691E7ACC870186C9D1589D2A332B51EC;PATH=/", "X-Cache" => "Miss from cloudfront", "Via" => "1.1 978198446b6fdba8a499c04f84a3a7e6.cloudfront.net (CloudFront)", "X-Amz-Cf-Id" => "ilwhpG75Ea4iXumklw7484nYt2jbx-L6ZaeiO9naUOstx45ia_nuaQ==" }
+  end
+  let(:body) do
+    { :id => "94613973-930f-4a53-9286-6e9efcbb5c57", :timestamp => "2016-10-12T20:07:54.876Z", :result => { :source => "domains", :resolvedQuery => "hello", :action => "smalltalk.greetings", :parameters => { :simplified => "hello" }, :metadata => {}, :fulfillment => { :speech => "Good day!" }, :score => 0.0 }, :status => { :code => 200, :errorType => "success" }, :sessionId => "555" }
+  end
+  let(:expected_headers) do
+    { 'Authorization' => 'Bearer CS', 'Connection' => 'close', 'Content-Type' => 'application/json; charset=UTF-8', 'Host' => 'api.api.ai' }
+  end
+  let(:expected_url) { "https://api.api.ai/v1/query?v=20150910" }
+  let(:expected_body) { '{"query":"hello","lang":"EN","sessionId":"555"}' }
+
+  describe "#perform" do
+    it "performs a request" do
+      stub = stub_request(:post, expected_url).
+        with(:body => expected_body,
+             :headers => expected_headers).
+        to_return(:status => 200, :body => body.to_json, :headers => response_headers)
+      subject.perform
+      expect(stub).to have_been_requested
+    end
+
+    context "it times out" do
+      it "times out properly" do
+        stub = stub_request(:post, expected_url).
+          with(:body => expected_body,
+               :headers => expected_headers).to_timeout
+        expect {
+          subject.perform
+        }.to raise_error(Errno::ETIMEDOUT)
+        expect(stub).to have_been_requested
+      end
+    end
+  end
+end

data/spec/helper.rb

@@ -12,9 +12,10 @@ end
 
 require 'api-ai-ruby'
 require 'rspec'
-#require 'webmock/rspec'
+require 'webmock'
+require 'webmock/rspec'
 
-#WebMock.disable_net_connect!(allow: 'coveralls.io')
+WebMock.allow_net_connect!
 
 RSpec.configure do |config|
   config.expect_with :rspec do |c|

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: api-ai-ruby
 version: !ruby/object:Gem::Version
-  version: 1.2.1
+  version: 1.2.2
 platform: ruby
 authors:
 - api.ai
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-09-22 00:00:00.000000000 Z
+date: 2016-10-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler