http 2.2.2 → 3.0.0.pre

data/spec/lib/http/response_spec.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 RSpec.describe HTTP::Response do
   let(:body)          { "Hello world!" }
   let(:uri)           { "http://example.com/" }
@@ -173,4 +174,13 @@ RSpec.describe HTTP::Response do
       expect(response.connection).to eq connection
     end
   end
+
+  describe "#chunked?" do
+    subject { response }
+    context "when encoding is set to chunked" do
+      let(:headers) { {"Transfer-Encoding" => "chunked"} }
+      it { is_expected.to be_chunked }
+    end
+    it { is_expected.not_to be_chunked }
+  end
 end

data/spec/lib/http/uri_spec.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 RSpec.describe HTTP::URI do
   let(:example_http_uri_string)  { "http://example.com" }
   let(:example_https_uri_string) { "https://example.com" }
@@ -18,4 +19,14 @@ RSpec.describe HTTP::URI do
   it "sets default ports for HTTPS URIs" do
     expect(https_uri.port).to eq 443
   end
+
+  describe "#dup" do
+    it "doesn't share internal value between duplicates" do
+      duplicated_uri = http_uri.dup
+      duplicated_uri.host = "example.org"
+
+      expect(duplicated_uri.to_s).to eq("http://example.org")
+      expect(http_uri.to_s).to eq("http://example.com")
+    end
+  end
 end

data/spec/lib/http_spec.rb

@@ -49,7 +49,7 @@ RSpec.describe HTTP do
     end
 
     context "with a large request body" do
-      %w(global null per_operation).each do |timeout|
+      %w[global null per_operation].each do |timeout|
         context "with a #{timeout} timeout" do
           [16_000, 16_500, 17_000, 34_000, 68_000].each do |size|
             [0, rand(0..100), rand(100..1000)].each do |fuzzer|
@@ -382,11 +382,6 @@ RSpec.describe HTTP do
       client = HTTP.headers("Cookie" => "foo=bar").cookies(:baz => :moo)
       expect(client.get(endpoint).to_s).to eq "foo: bar\nbaz: moo"
     end
-
-    it "unifies socket errors into HTTP::ConnectionError" do
-      expect { HTTP.get "http://thishostshouldnotexists.com" }.to raise_error HTTP::ConnectionError
-      expect { HTTP.get "http://127.0.0.1:000" }.to raise_error HTTP::ConnectionError
-    end
   end
 
   describe ".nodelay" do
@@ -434,6 +429,15 @@ RSpec.describe HTTP do
 
         expect(Zlib::GzipReader.new(StringIO.new(encoded)).read).to eq body
       end
+
+      it "sends deflated body" do
+        client   = HTTP.use :auto_deflate => {:method => "deflate"}
+        body     = "Hello!"
+        response = client.post("#{dummy.endpoint}/echo-body", :body => body)
+        encoded  = response.to_s
+
+        expect(Zlib::Inflate.inflate(encoded)).to eq body
+      end
     end
 
     context "with :auto_inflate" do
@@ -461,4 +465,12 @@ RSpec.describe HTTP do
       end
     end
   end
+
+  it "unifies socket errors into HTTP::ConnectionError" do
+    expect { HTTP.get "http://thishostshouldnotexists.com" }.
+      to raise_error HTTP::ConnectionError
+
+    expect { HTTP.get "http://127.0.0.1:000" }.
+      to raise_error HTTP::ConnectionError
+  end
 end

data/spec/regression_specs.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require "spec_helper"
 
 RSpec.describe "Regression testing" do

data/spec/spec_helper.rb

@@ -18,6 +18,7 @@ end
 require "http"
 require "rspec/its"
 require "support/capture_warning"
+require "support/fakeio"
 
 # See http://rubydoc.info/gems/rspec-core/RSpec/Core/Configuration
 RSpec.configure do |config|

data/spec/support/black_hole.rb

@@ -1,6 +1,13 @@
 # frozen_string_literal: true
+
 module BlackHole
-  def self.method_missing(*)
-    self
+  class << self
+    def method_missing(*) # rubocop: disable Style/MethodMissing
+      self
+    end
+
+    def respond_to_missing?(*)
+      true
+    end
   end
 end

data/spec/support/capture_warning.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 def capture_warning
   old_stderr = $stderr
   $stderr = StringIO.new

data/spec/support/dummy_server.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require "webrick"
 require "webrick/ssl"
 
@@ -23,7 +24,7 @@ class DummyServer < WEBrick::HTTPServer
     :SSLStartImmediately  => true
   ).freeze
 
-  def initialize(options = {})
+  def initialize(options = {}) # rubocop:disable Style/OptionHash
     super(options[:ssl] ? SSL_CONFIG : CONFIG)
     mount("/", Servlet)
   end

data/spec/support/dummy_server/servlet.rb

@@ -17,7 +17,7 @@ class DummyServer < WEBrick::HTTPServer
       @handlers ||= {}
     end
 
-    %w(get post head).each do |method|
+    %w[get post head].each do |method|
       class_eval <<-RUBY, __FILE__, __LINE__
         def self.#{method}(path, &block)
           handlers["#{method}:\#{path}"] = block

data/spec/support/fakeio.rb

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+require "stringio"
+
+class FakeIO
+  def initialize(content)
+    @io = StringIO.new(content)
+  end
+
+  def string
+    @io.string
+  end
+
+  def read(*args)
+    @io.read(*args)
+  end
+
+  def size
+    @io.size
+  end
+end

data/spec/support/http_handling_shared.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 RSpec.shared_context "HTTP handling" do
   describe "timeouts" do
     let(:conn_timeout) { 1 }

data/spec/support/proxy_server.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require "webrick/httpproxy"
 
 require "support/black_hole"

data/spec/support/servers/config.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module ServerConfig
   def addr
     config[:BindAddress]

data/spec/support/servers/runner.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 module ServerRunner
   def run_server(name)
     let! name do

data/spec/support/ssl_helper.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require "pathname"
 
 require "certificate_authority"
@@ -7,7 +8,7 @@ module SSLHelper
   CERTS_PATH = Pathname.new File.expand_path("../../../tmp/certs", __FILE__)
 
   class RootCertificate < ::CertificateAuthority::Certificate
-    EXTENSIONS = {"keyUsage" => {"usage" => %w(critical keyCertSign)}}.freeze
+    EXTENSIONS = {"keyUsage" => {"usage" => %w[critical keyCertSign]}}.freeze
 
     def initialize
       super()
@@ -88,7 +89,7 @@ module SSLHelper
       }
     end
 
-    %w(server client).each do |side|
+    %w[server client].each do |side|
       class_eval <<-RUBY, __FILE__, __LINE__
         def #{side}_cert
           @#{side}_cert ||= ChildCertificate.new ca

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: http
 version: !ruby/object:Gem::Version
-  version: 2.2.2
+  version: 3.0.0.pre
 platform: ruby
 authors:
 - Tony Arcieri
@@ -11,7 +11,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-04-27 00:00:00.000000000 Z
+date: 2017-09-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: http_parser.rb
@@ -31,16 +31,22 @@ dependencies:
   name: http-form_data
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 2.0.0.pre.pre2
+    - - "<"
       - !ruby/object:Gem::Version
-        version: 1.0.1
+        version: '3'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 1.0.1
+        version: 2.0.0.pre.pre2
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '3'
 - !ruby/object:Gem::Dependency
   name: http-cookie
   requirement: !ruby/object:Gem::Requirement
@@ -124,6 +130,7 @@ files:
 - lib/http/options.rb
 - lib/http/redirector.rb
 - lib/http/request.rb
+- lib/http/request/body.rb
 - lib/http/request/writer.rb
 - lib/http/response.rb
 - lib/http/response/body.rb
@@ -153,6 +160,7 @@ files:
 - spec/lib/http/options/proxy_spec.rb
 - spec/lib/http/options_spec.rb
 - spec/lib/http/redirector_spec.rb
+- spec/lib/http/request/body_spec.rb
 - spec/lib/http/request/writer_spec.rb
 - spec/lib/http/request_spec.rb
 - spec/lib/http/response/body_spec.rb
@@ -166,6 +174,7 @@ files:
 - spec/support/capture_warning.rb
 - spec/support/dummy_server.rb
 - spec/support/dummy_server/servlet.rb
+- spec/support/fakeio.rb
 - spec/support/http_handling_shared.rb
 - spec/support/proxy_server.rb
 - spec/support/servers/config.rb
@@ -186,12 +195,12 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '2.0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - ">"
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 1.3.1
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.8
+rubygems_version: 2.6.12
 signing_key: 
 specification_version: 4
 summary: HTTP should be easy
@@ -212,6 +221,7 @@ test_files:
 - spec/lib/http/options/proxy_spec.rb
 - spec/lib/http/options_spec.rb
 - spec/lib/http/redirector_spec.rb
+- spec/lib/http/request/body_spec.rb
 - spec/lib/http/request/writer_spec.rb
 - spec/lib/http/request_spec.rb
 - spec/lib/http/response/body_spec.rb
@@ -225,6 +235,7 @@ test_files:
 - spec/support/capture_warning.rb
 - spec/support/dummy_server.rb
 - spec/support/dummy_server/servlet.rb
+- spec/support/fakeio.rb
 - spec/support/http_handling_shared.rb
 - spec/support/proxy_server.rb
 - spec/support/servers/config.rb