Fix specs with recent Async-http versions.

Recent versions of async-http (and its sub-libraries like request-http) are strict about content-length header:

- Get requests should not have any content-length request header provided
- POST requests with the body and all responses with stringio body have the content-length header appended automagically

Author: misdoro

lib/webmock/http_lib_adapters/async_http_client_adapter.rb

@@ -211,7 +211,7 @@ def webmock_responses
           def build_response(webmock_response)
             headers = (webmock_response.headers || {}).each_with_object([]) do |(k, value), o|
               Array(value).each do |v|
-                o.push [k, v]
+                o.push [k, v] unless k.downcase == 'content-length' # async-http appends the exact content-length automatically
               end
             end
 

spec/acceptance/shared/stubbing_requests.rb

@@ -366,10 +366,10 @@
 
     describe "based on headers" do
       it "should match requests if headers are the same" do
-        stub_request(:get, "www.example.com").with(headers: SAMPLE_HEADERS )
+        stub_request(:get, "www.example.com").with(headers: { "Accept" => "application/json" } )
         expect(http_request(
           :get, "http://www.example.com/",
-        headers: SAMPLE_HEADERS).status).to eq("200")
+        headers: { "Accept" => "application/json" }).status).to eq("200")
       end
 
       it "should match requests if headers are the same and declared as array" do
@@ -413,12 +413,12 @@
       end
 
       it "should not match requests if headers are different" do
-        stub_request(:get, "www.example.com").with(headers: SAMPLE_HEADERS)
+        stub_request(:get, "www.example.com").with(headers: { "Accept" => "application/json" })
 
         expect {
           http_request(
             :get, "http://www.example.com/",
-          headers: { 'Content-Length' => '9999'})
+          headers: { "Accept" => "text/html" })
         }.to raise_error(WebMock::NetConnectNotAllowedError, %r(Real HTTP connections are disabled. Unregistered request: GET http://www.example.com/ with headers))
       end
 

spec/acceptance/webmock_shared.rb

@@ -9,7 +9,7 @@
 require 'acceptance/shared/complex_cross_concern_behaviors'
 
 unless defined? SAMPLE_HEADERS
-  SAMPLE_HEADERS = { "Content-Length" => "8888", "Accept" => "application/json" }
+  SAMPLE_HEADERS = { "Accept" => "application/json" }
   ESCAPED_PARAMS = "x=ab%20c&z=%27Stop%21%27%20said%20Fred%20m"
   NOT_ESCAPED_PARAMS = "z='Stop!' said Fred m&x=ab c"
 end

webmock.gemspec

@@ -46,7 +46,6 @@ Gem::Specification.new do |s|
   s.add_development_dependency 'rdoc',            '>  3.5.0'
   s.add_development_dependency 'webrick'
   s.add_development_dependency 'rspec-retry'
-  s.add_development_dependency 'protocol-http',   '<= 0.50.0'
 
   s.files        = Dir["CHANGELOG.md", "LICENSE", "README.md", "lib/**/*"]
   s.require_path = "lib"