td-client 0.8.75 → 0.8.76

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 1ecd6b8aed80e8f53f2d1558d5d3eb7a46139bb4
-  data.tar.gz: cf9e411eb507cc49dbd37cce1a9dfa0b1a9b4338
+  metadata.gz: bed709b18af78da0d5e9be3b5ff11e52cce7b631
+  data.tar.gz: 3280f6e1a3fa77067adf1ef2695eb05b4b0cd30b
 SHA512:
-  metadata.gz: ec2506706f26dce0390218f20dd350b0d0389172ad6b1adf1b047a8fff1e233c45c1cdd0ac74ae9b02cea68414b273e63d0ac2aad582a2857e42d795eea8aa35
-  data.tar.gz: 47094392a7112ef70587f9994a83299cac26da9a86e660ac51046efd429ae998aed4880132dcf2150aff982f0679e19298ac1eac3404c10cb85589623f1e4a36
+  metadata.gz: 3bf28abf67a22673d54c387915701fc9775aeb82f02c1e92e9e73df4009656ba489d1c7089369aa6efa3909ddc50f81c6b45411cb56c190718a8cd8655af94f1
+  data.tar.gz: c438cc9da912217589d01a3debca817d168eb4a7bdb6840c4c20ae17b8e25a8c2d56a87f481276beacadfc5132da65b3ac7b8c95f6161eea790838342ea093b5

data/lib/td/client/api.rb

@@ -103,19 +103,6 @@ class API
     end
 
     @http_proxy = opts[:http_proxy] || ENV['HTTP_PROXY']
-    if @http_proxy
-      http_proxy = if @http_proxy =~ /\Ahttp:\/\/(.*)\z/
-                     $~[1]
-                   else
-                     @http_proxy
-                   end
-      proxy_host, proxy_port = http_proxy.split(':', 2)
-      proxy_port = (proxy_port ? proxy_port.to_i : 80)
-      @http_class = Net::HTTP::Proxy(proxy_host, proxy_port)
-    else
-      @http_class = Net::HTTP
-    end
-
     @headers = opts[:headers] || {}
     @api = api_client("#{@ssl ? 'https' : 'http'}://#{@host}:#{@port}")
   end
@@ -222,7 +209,7 @@ class API
     when /double/i
       "double"
     else
-      raise "Type name must eather of int, long, string float or double"
+      raise "Type name must either of int, long, string float or double"
     end
   end
 
@@ -241,55 +228,6 @@ class API
 
 private
 
-  module DeflateReadBodyMixin
-    attr_accessor :gzip
-
-    # @yield [fragment]
-    def each_fragment(&block)
-      if @gzip
-        infl = Zlib::Inflate.new(Zlib::MAX_WBITS + 16)
-      else
-        infl = Zlib::Inflate.new
-      end
-      begin
-        read_body {|fragment|
-          block.call infl.inflate(fragment)
-        }
-      ensure
-        infl.close
-      end
-      nil
-    end
-  end
-
-  module CountReadBodyTotalSize
-    attr_reader :total_fragment_size
-
-    def read_body(&block)
-      return super if @total_fragment_size
-
-      if block_given?
-        @total_fragment_size = 0
-
-        super {|fragment|
-          @total_fragment_size += fragment.size
-          block.call(fragment)
-        }
-      else
-        super().tap {|body|
-          @total_fragment_size = body.size
-        }
-      end
-    end
-  end
-
-  module DirectReadBodyMixin
-    # @yield [fragment]
-    def each_fragment(&block)
-      read_body(&block)
-    end
-  end
-
   # @param [String] url
   # @param [Hash] params
   # @yield [response]
@@ -298,22 +236,17 @@ private
       do_get(url, params, &block)
     end
   end
-
   # @param [String] url
   # @param [Hash] params
   # @yield [response]
   def do_get(url, params=nil, &block)
-    http, header = new_http
-
-    path = @base_path + url
-    if params && !params.empty?
-      path << "?"+params.map {|k,v|
-        "#{k}=#{e v}"
-      }.join('&')
-    end
+    client, header = new_client
+    client.send_timeout = @send_timeout
+    client.receive_timeout = @read_timeout
 
     header['Accept-Encoding'] = 'deflate, gzip'
-    request = Net::HTTP::Get.new(path, header)
+
+    target = build_endpoint(url, @host)
 
     unless ENV['TD_CLIENT_DEBUG'].nil?
       puts "DEBUG: REST GET call:"
@@ -332,20 +265,23 @@ private
     begin # this block is to allow retry (redo) in the begin part of the begin-rescue block
       begin
         if block
-          response = http.request(request) {|res|
-            res.extend(CountReadBodyTotalSize)
-            block.call(res)
+          current_total_chunk_size = 0
+          response = client.get(target, params, header) {|res, chunk|
+            current_total_chunk_size += chunk.size
+            block.call(res, chunk, current_total_chunk_size)
           }
+
+          # XXX ext/openssl raises EOFError in case where underlying connection causes an error,
+          #     and msgpack-ruby that used in block handles it as an end of stream == no exception.
+          #     Therefor, check content size.
+          validate_content_length!(response, current_total_chunk_size) if @ssl
         else
-          response = http.request(request)
-        end
+          response = client.get(target, params, header)
 
-        # XXX ext/openssl raises EOFError in case where underlying connection causes an error,
-        #     and msgpack-ruby that used in block handles it as an end of stream == no exception.
-        #     Therefor, check content size.
-        raise IncompleteError if @ssl && !completed_body?(response)
+          validate_content_length!(response, response.body.size) if @ssl
+        end
 
-        status = response.code.to_i
+        status = response.code
         # retry if the HTTP error code is 500 or higher and we did not run out of retrying attempts
         if !block_given? && status >= 500 && cumul_retry_delay < @max_cumul_retry_delay
           $stderr.puts "Error #{status}: #{get_error(response)}. Retrying after #{retry_delay} seconds..."
@@ -381,37 +317,29 @@ private
       puts "DEBUG:   body:   " + response.body.to_s
     end
 
-    body = response.body
-    unless block
-      if ce = response.header['content-encoding']
-        if ce == 'gzip'
-          infl = Zlib::Inflate.new(Zlib::MAX_WBITS + 16)
-          begin
-            body = infl.inflate(body)
-          ensure
-            infl.close
-          end
-        else
-          body = Zlib::Inflate.inflate(body)
-        end
-      end
-    end
+    body = block ? response.body : inflate_body(response)
 
-    return [response.code, body, response]
+    return [response.code.to_s, body, response]
   end
 
-  def completed_body?(response)
-    # NOTE If response doesn't have content_length, we assume it succeeds.
-    return true unless (content_length = response.header.content_length)
+  def validate_content_length!(response, body_size)
+    content_length = response.header['Content-Length'].first
+    raise IncompleteError if @ssl && content_length && content_length.to_i != body_size
+  end
 
-    if response.body.instance_of? String
-      content_length == response.body.length
-    else
-      if response.respond_to? :total_fragment_size
-        content_length == response.total_fragment_size
-      else
-        true
+  def inflate_body(response)
+    return response.body if (ce = response.header['Content-Encoding']).empty?
+
+    if ce.include?('gzip')
+      infl = Zlib::Inflate.new(Zlib::MAX_WBITS + 16)
+      begin
+        infl.inflate(response.body)
+      ensure
+        infl.close
       end
+    else
+      # NOTE maybe for content-encoding is msgpack.gz ?
+      Zlib::Inflate.inflate(response.body)
     end
   end
 
@@ -426,25 +354,20 @@ private
   # @param [String] url
   # @param [Hash] params
   def do_post(url, params=nil)
-    http, header = new_http
+    target = build_endpoint(url, @host)
 
-    path = @base_path + url
+    client, header = new_client
+    client.send_timeout       = @send_timeout
+    client.receive_timeout    = @read_timeout
+    header['Accept-Encoding'] = 'gzip'
 
     unless ENV['TD_CLIENT_DEBUG'].nil?
       puts "DEBUG: REST POST call:"
       puts "DEBUG:   header: " + header.to_s
-      puts "DEBUG:   path:   " + path.to_s
+      puts "DEBUG:   path:   " + (@base_path + url).to_s
       puts "DEBUG:   params: " + params.to_s
     end
 
-    if params && !params.empty?
-      request = Net::HTTP::Post.new(path, header)
-      request.set_form_data(params)
-    else
-      header['Content-Length'] = 0.to_s
-      request = Net::HTTP::Post.new(path, header)
-    end
-
     # up to 7 retries with exponential (base 2) back-off starting at 'retry_delay'
     retry_delay = @retry_delay
     cumul_retry_delay = 0
@@ -455,7 +378,7 @@ private
     response = nil
     begin # this block is to allow retry (redo) in the begin part of the begin-rescue block
       begin
-        response = http.request(request)
+        response = client.post(target, params || {}, header)
 
         # if the HTTP error code is 500 or higher and the user requested retrying
         # on post request, attempt a retry
@@ -488,14 +411,18 @@ private
       end
     end while false
 
-    unless ENV['TD_CLIENT_DEBUG'].nil?
-      puts "DEBUG: REST POST response:"
-      puts "DEBUG:   header: " + response.header.to_s
-      puts "DEBUG:   status: " + response.code.to_s
-      puts "DEBUG:   body:   <omitted>"
+    begin
+      unless ENV['TD_CLIENT_DEBUG'].nil?
+        puts "DEBUG: REST POST response:"
+        puts "DEBUG:   header: " + response.header.to_s
+        puts "DEBUG:   status: " + response.code.to_s
+        puts "DEBUG:   body:   <omitted>"
+      end
+      return [response.code.to_s, response.body, response]
+    ensure
+      # Disconnect keep-alive connection explicitly here, not by GC.
+      client.reset(target) rescue nil
     end
-
-    return [response.code, response.body, response]
   end
 
   # @param [String] url
@@ -561,38 +488,6 @@ private
     end
   end
 
-  # @param [Hash] opts
-  # @return [http, Hash]
-  def new_http(opts = {})
-    host = opts[:host] || @host
-    http = @http_class.new(host, @port)
-    http.open_timeout = 60
-    if @ssl
-      http.use_ssl = true
-      http.verify_mode = OpenSSL::SSL::VERIFY_PEER
-      #store = OpenSSL::X509::Store.new
-      #http.cert_store = store
-      http.ca_file = File.join(ssl_ca_file)
-      # Disable SSLv3 connection in favor of POODLE Attack protection
-      # ruby 1.8.7 uses own @ssl_context instead of calling
-      # SSLContext#set_params.
-      if ctx = http.instance_eval { @ssl_context }
-        ctx.options = OpenSSL::SSL::OP_ALL | OpenSSL::SSL::OP_NO_SSLv3
-      end
-    end
-
-    header = {}
-    if @apikey
-      header['Authorization'] = "TD1 #{apikey}"
-    end
-    header['Date'] = Time.now.rfc2822
-    header['User-Agent'] = @user_agent
-
-    header.merge!(@headers)
-
-    return http, header
-  end
-
   # @param [Hash] opts
   # @return [HTTPClient, Hash]
   def new_client(opts = {})
@@ -694,11 +589,7 @@ private
     begin
       js = JSON.load(res.body)
       if js.nil?
-        error['message'] = if res.respond_to?(:message)
-                      res.message # Net::HTTP
-                    else
-                      res.reason # HttpClient
-                    end
+        error['message'] = res.reason
       else
         error['message']    = js['message'] || js['error']
         error['stacktrace'] = js['stacktrace']
@@ -747,14 +638,22 @@ private
   if ''.respond_to?(:encode)
     # @param [String] s
     # @return [String]
+    # * ' ' and '+' must be escaped into %20 and %2B because escaped text may be
+    #   used as both URI and query.
+    # * '.' must be escaped as %2E because it may be cunfused with extension.
     def e(s)
-      CGI.escape(s.to_s.encode("UTF-8"))
+      s = s.to_s.encode(Encoding::UTF_8).force_encoding(Encoding::ASCII_8BIT)
+      s.gsub!(/[^\-_!~*'()~0-9A-Z_a-z]/){|x|'%%%02X' % x.ord}
+      s
     end
   else
     # @param [String] s
     # @return [String]
+    # * ' ' and '+' must be escaped into %20 and %2B because escaped text may be
+    #   used as both URI and query.
+    # * '.' must be escaped as %2E because it may be cunfused with extension.
     def e(s)
-      CGI.escape(s.to_s)
+      s.to_s.gsub(/[^\-_!~*'()~0-9A-Z_a-z]/){|x|'%%%02X' % x.ord}
     end
   end
 

data/lib/td/client/api/bulk_load.rb

@@ -149,8 +149,8 @@ module BulkLoad
   def bulk_load_run(name, scheduled_time = nil)
     path = job_path(name)
     opts = {}
-    opts[:scheduled_time] = scheduled_time unless scheduled_time.nil?
-    res = api { post(path, opts) }
+    opts[:scheduled_time] = scheduled_time.to_s unless scheduled_time.nil?
+    res = api { post(path, opts.to_json) }
     unless res.ok?
       raise_error("BulkLoadSession: #{name} job create failed", res)
     end

data/lib/td/client/api/job.rb

@@ -135,38 +135,21 @@ module Job
   # @return [nil, String]
   def job_result_format(job_id, format, io=nil, &block)
     if io
-      code, body, res = get("/v3/job/result/#{e job_id}", {'format'=>format}) {|res|
-        if res.code != "200"
+      infl = nil
+      code, body, res = get("/v3/job/result/#{e job_id}", {'format'=>format}) {|res, chunk, current_total_chunk_size|
+        if res.code != 200
           raise_error("Get job result failed", res)
         end
 
-        if ce = res.header['Content-Encoding']
-          res.extend(DeflateReadBodyMixin)
-          res.gzip = true if ce == 'gzip'
-        else
-          res.extend(DirectReadBodyMixin)
-        end
-
-        res.extend(DirectReadBodyMixin)
-        if ce = res.header['Content-Encoding']
-          if ce == 'gzip'
-            infl = Zlib::Inflate.new(Zlib::MAX_WBITS + 16)
-          else
-            infl = Zlib::Inflate.new
-          end
-        end
+        infl ||= create_inflalte_or_null_inflate(res)
 
-        res.each_fragment {|fragment|
-          # uncompressed if the 'Content-Enconding' header is set in response
-          fragment = infl.inflate(fragment) if ce
-          io.write(fragment)
-          block.call(res.total_fragment_size) if block_given?
-        }
+        io.write infl.inflate(chunk)
+        block.call(current_total_chunk_size) if block_given?
       }
       nil
     else
       code, body, res = get("/v3/job/result/#{e job_id}", {'format'=>format})
-      if res.code != "200"
+      if code != "200"
         raise_error("Get job result failed", res)
       end
       body
@@ -174,56 +157,55 @@ module Job
   end
 
   # block is optional and must accept 1 argument
-  # 
+  #
   # @param [String] job_id
   # @param [Proc] block
   # @return [nil]
   def job_result_each(job_id, &block)
-    get("/v3/job/result/#{e job_id}", {'format'=>'msgpack'}) {|res|
-      if res.code != "200"
+    upkr = MessagePack::Unpacker.new
+    infl = nil
+
+    get("/v3/job/result/#{e job_id}", {'format'=>'msgpack'}) {|res, chunk, current_total_chunk_size|
+      if res.code != 200
         raise_error("Get job result failed", res)
       end
 
       # default to decompressing the response since format is fixed to 'msgpack'
-      res.extend(DeflateReadBodyMixin)
-      res.gzip = (res.header['Content-Encoding'] == 'gzip')
-      upkr = MessagePack::Unpacker.new
-      res.each_fragment {|inflated_fragment|
-        upkr.feed_each(inflated_fragment, &block)
-      }
+      infl ||= create_inflate(res)
+
+      inflated_fragment = infl.inflate(chunk)
+      upkr.feed_each(inflated_fragment, &block)
     }
     nil
+  ensure
+    infl.close if infl
   end
 
   # block is optional and must accept 1 argument
-  # 
+  #
   # @param [String] job_id
   # @param [Proc] block
   # @return [nil]
   def job_result_each_with_compr_size(job_id, &block)
-    get("/v3/job/result/#{e job_id}", {'format'=>'msgpack'}) {|res|
-      if res.code != "200"
+    upkr = MessagePack::Unpacker.new
+    infl = nil
+
+    get("/v3/job/result/#{e job_id}", {'format'=>'msgpack'}) {|res, chunk, current_total_chunk_size|
+      if res.code != 200
         raise_error("Get job result failed", res)
       end
 
-      res.extend(DirectReadBodyMixin)
-      if res.header['Content-Encoding'] == 'gzip'
-        infl = Zlib::Inflate.new(Zlib::MAX_WBITS + 16)
-      else
-        infl = Zlib::Inflate.new
-      end
-      upkr = MessagePack::Unpacker.new
-      begin
-        res.each_fragment {|fragment|
-          upkr.feed_each(infl.inflate(fragment)) {|unpacked|
-            block.call(unpacked, res.total_fragment_size) if block_given?
-          }
-        }
-      ensure
-        infl.close
-      end
+      # default to decompressing the response since format is fixed to 'msgpack'
+      infl ||= create_inflate(res)
+
+      inflated_fragment = infl.inflate(chunk)
+      upkr.feed_each(inflated_fragment) {|unpacked|
+        block.call(unpacked, current_total_chunk_size) if block_given?
+      }
     }
     nil
+  ensure
+    infl.close if infl
   end
 
   # @param [String] job_id
@@ -232,20 +214,20 @@ module Job
   def job_result_raw(job_id, format, io = nil, &block)
     body = nil
 
-    get("/v3/job/result/#{e job_id}", {'format'=>format}) {|res|
-      if res.code != "200"
+    get("/v3/job/result/#{e job_id}", {'format'=>format}) {|res, chunk, current_total_chunk_size|
+      if res.code != 200
         raise_error("Get job result failed", res)
       end
 
       if io
-        res.extend(DirectReadBodyMixin)
-
-        res.each_fragment {|fragment|
-          io.write(fragment)
-          block.call(res.total_fragment_size) if block_given?
-        }
+        io.write(chunk)
+        block.call(current_total_chunk_size) if block_given?
       else
-        body = res.read_body
+        if body
+          body += chunk
+        else
+          body = chunk
+        end
       end
     }
     body
@@ -303,5 +285,31 @@ module Job
     return js['job_id'].to_s
   end
 
+  private
+
+  class NullInflate
+    def inflate(chunk)
+      chunk
+    end
+
+    def close
+    end
+  end
+
+  def create_inflalte_or_null_inflate(response)
+    if response.header['Content-Encoding'].empty?
+      NullInflate.new
+    else
+      create_inflate(response)
+    end
+  end
+
+  def create_inflate(response)
+    if response.header['Content-Encoding'].include?('gzip')
+      Zlib::Inflate.new(Zlib::MAX_WBITS + 16)
+    else
+      Zlib::Inflate.new
+    end
+  end
 end
 end

data/lib/td/client/model.rb

@@ -577,7 +577,7 @@ class ScheduledJob < Job
 
   # @param [TreasureData::Client] client
   # @param [String] scheduled_at
-  # @param [...] args for Job#initialize
+  # @param [...] super_args for Job#initialize
   def initialize(client, scheduled_at, *super_args)
     super(client, *super_args)
     if scheduled_at.to_s.empty?
@@ -739,4 +739,3 @@ end
 
 
 end
-

data/lib/td/client/version.rb

@@ -1,5 +1,5 @@
 module TreasureData
   class Client
-    VERSION = '0.8.75'
+    VERSION = '0.8.76'
   end
 end

data/spec/spec_helper.rb

@@ -58,7 +58,6 @@ shared_context 'common helper' do
   end
 
   def e(s)
-    require 'cgi'
-    CGI.escape(s.to_s)
+    s.to_s.gsub(/[^*\-0-9A-Z_a-z]/){|x|'%%%02X' % x.ord}
   end
 end

data/spec/td/client/bulk_load_spec.rb

@@ -359,7 +359,7 @@ describe 'BulkImport API' do
   describe 'run' do
     it 'returns job_id' do
       stub_api_request(:post, '/v3/bulk_loads/nahi_test_1/jobs').
-        with(:body => '').
+        with(:body => '{}').
         to_return(:body => {'job_id' => 12345}.to_json)
       api.bulk_load_run('nahi_test_1').should == '12345'
     end
@@ -367,7 +367,7 @@ describe 'BulkImport API' do
     it 'accepts scheduled_time' do
       now = Time.now.to_i
       stub_api_request(:post, '/v3/bulk_loads/nahi_test_1/jobs').
-        with(:body => "scheduled_time=#{now}").
+        with(:body => {scheduled_time: now.to_s}.to_json).
         to_return(:body => {'job_id' => 12345}.to_json)
       api.bulk_load_run('nahi_test_1', now).should == '12345'
     end

data/spec/td/client/job_api_spec.rb

@@ -191,26 +191,45 @@ describe 'Job API' do
       s.string
     end
 
-    it 'returns formatted job result' do
-      stub_api_request(:get, '/v3/job/result/12345').
-        with(:query => {'format' => 'json'}).
-        to_return(
-          :headers => {'Content-Encoding' => 'gzip'},
-          :body => packed
-        )
-      api.job_result_format(12345, 'json').should == ['hello', 'world'].to_json
+    context 'Content-Encoding is empty' do
+      let(:io) { StringIO.new }
+      let(:json) { ['hello', 'world'].to_json }
+
+      it 'retrunes json' do
+        stub_api_request(:get, '/v3/job/result/12345').
+          with(:query => {'format' => 'json'}).
+          to_return(:body => json)
+
+        total_size = 0
+        api.job_result_format(12345, 'json', io) {|size| total_size += size }
+
+        io.string.should  == json
+        total_size.should == json.size
+      end
     end
 
-    it 'writes formatted job result' do
-      stub_api_request(:get, '/v3/job/result/12345').
-        with(:query => {'format' => 'json'}).
-        to_return(
-          :headers => {'Content-Encoding' => 'gzip'},
-          :body => packed
-        )
-      s = StringIO.new
-      api.job_result_format(12345, 'json', s)
-      s.string.should == ['hello', 'world'].to_json
+    context 'Content-Encoding is gzip' do
+      it 'returns formatted job result' do
+        stub_api_request(:get, '/v3/job/result/12345').
+          with(:query => {'format' => 'json'}).
+          to_return(
+            :headers => {'Content-Encoding' => 'gzip'},
+            :body => packed
+          )
+        api.job_result_format(12345, 'json').should == ['hello', 'world'].to_json
+      end
+
+      it 'writes formatted job result' do
+        stub_api_request(:get, '/v3/job/result/12345').
+          with(:query => {'format' => 'json'}).
+          to_return(
+            :headers => {'Content-Encoding' => 'gzip'},
+            :body => packed
+          )
+        s = StringIO.new
+        api.job_result_format(12345, 'json', s)
+        s.string.should == ['hello', 'world'].to_json
+      end
     end
   end
 

data/spec/td/client/spec_resources.rb

@@ -15,7 +15,7 @@ shared_context 'spec symbols' do
   end
 
   let :sched_name do
-    'sched_test'
+    'sched test'
   end
 
   let :result_name do
@@ -51,7 +51,7 @@ end
 shared_context 'job resources' do
   include_context 'database resources'
 
-  MAX_JOB = 20
+  MAX_JOB ||= 20
 
   let :job_types do
     [

data/spec/td/client/user_api_spec.rb

@@ -53,8 +53,8 @@ describe 'User API' do
 
     # TODO
     it 'does not escape sp but it must be a bug' do
-      stub_api_request(:post, "/v3/user/add/!++++@%23$%25%5E&*()_+%7C~").to_return(:body => {}.to_json)
-      api.add_user('!    @#$%^&*()_+|~', "org", 'name+suffix@example.com', 'password').should == true
+      stub_api_request(:post, "/v3/user/add/!%20%20%20%20@%23$%25%5E&*()_%2B%7C~%2Ecom").to_return(:body => {}.to_json)
+      api.add_user('!    @#$%^&*()_+|~.com', "org", 'name+suffix@example.com', 'password').should == true
     end
   end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: td-client
 version: !ruby/object:Gem::Version
-  version: 0.8.75
+  version: 0.8.76
 platform: ruby
 authors:
 - Treasure Data, Inc.
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-08-11 00:00:00.000000000 Z
+date: 2015-11-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: msgpack
@@ -232,7 +232,6 @@ files:
 - spec/spec_helper.rb
 - spec/td/client/access_control_api_spec.rb
 - spec/td/client/account_api_spec.rb
-- spec/td/client/api_http_access_spec.rb
 - spec/td/client/api_spec.rb
 - spec/td/client/api_ssl_connection_spec.rb
 - spec/td/client/bulk_import_spec.rb
@@ -263,7 +262,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 1.8.7
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
@@ -271,14 +270,13 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.5
+rubygems_version: 2.4.5.1
 signing_key: 
 specification_version: 4
 summary: Treasure Data API library for Ruby
 test_files:
 - spec/td/client/access_control_api_spec.rb
 - spec/td/client/account_api_spec.rb
-- spec/td/client/api_http_access_spec.rb
 - spec/td/client/api_spec.rb
 - spec/td/client/api_ssl_connection_spec.rb
 - spec/td/client/bulk_import_spec.rb

data/spec/td/client/api_http_access_spec.rb

@@ -1,50 +0,0 @@
-require 'spec_helper'
-
-describe API do
-  describe '#completed_body?' do
-    let(:api) { TreasureData::API.new('')  }
-    let(:response) { double(:response) }
-
-    subject { api.__send__(:completed_body?, response) }
-
-    context 'response has no content length' do
-      before do
-        response.stub_chain(:header, :content_length).and_return(nil)
-      end
-
-      it { is_expected.to be }
-    end
-
-    context 'response has content length' do
-      let(:content_length) { 10 }
-
-      before do
-        response.stub_chain(:header, :content_length).and_return(content_length)
-      end
-
-      context 'content length equal body size' do
-        before do
-          response.stub(:body).and_return('a' * content_length)
-        end
-
-        it { is_expected.to be }
-      end
-
-      context 'content length lager than body size' do
-        before do
-          response.stub(:body).and_return('a' * (content_length - 1))
-        end
-
-        it { is_expected.not_to be }
-      end
-
-      context 'content length less than body size' do
-        before do
-          response.stub(:body).and_return('a' * (content_length + 1))
-        end
-
-        it { is_expected.not_to be }
-      end
-    end
-  end
-end