webmock 3.18.1 → 3.19.0

data/spec/acceptance/net_http/net_http_shared.rb

@@ -1,190 +0,0 @@
-require 'set'
-
-shared_examples_for "Net::HTTP" do
-  describe "when making real requests", net_connect: true do
-    let(:port){ WebMockServer.instance.port }
-
-    before(:each) do
-      @http = Net::HTTP.new("localhost", port)
-    end
-
-    it "should return a Net::ReadAdapter from response.body when a real request is made with a block and #read_body", net_connect: true do
-      response = Net::HTTP.new("localhost", port).request_get('/') { |r| r.read_body { } }
-      expect(response.body).to be_a(Net::ReadAdapter)
-    end
-
-    it "should handle requests with block passed to read_body", net_connect: true do
-      body = "".dup
-      req = Net::HTTP::Get.new("/")
-      Net::HTTP.start("localhost", port) do |http|
-        http.request(req) do |res|
-          res.read_body do |str|
-            body << str
-          end
-        end
-      end
-      expect(body).to match(/hello world/)
-    end
-
-    it "should connect only once when connected on start", net_connect: true do
-      @http = Net::HTTP.new('localhost', port)
-      socket_before_request = socket_after_request = nil
-      @http.start {|conn|
-        socket_before_request = conn.instance_variable_get(:@socket)
-        conn.request(Net::HTTP::Get.new("/"))
-        socket_after_request = conn.instance_variable_get(:@socket)
-      }
-
-      if !defined?(WebMock::NetHTTPUtility) || WebMock::Config.instance.net_http_connect_on_start
-        expect(socket_before_request).to be_a(Net::BufferedIO)
-        expect(socket_after_request).to be_a(Net::BufferedIO)
-        expect(socket_after_request).to be(socket_before_request)
-      else
-        expect(socket_before_request).to be_a(StubSocket)
-        expect(socket_after_request).to be_a(Net::BufferedIO)
-      end
-    end
-
-    it "should allow sending multiple requests when persisted", net_connect: true do
-      @http = Net::HTTP.new('example.org')
-      @http.start
-      expect(@http.get("/")).to be_a(Net::HTTPSuccess)
-      expect(@http.get("/")).to be_a(Net::HTTPSuccess)
-      expect(@http.get("/")).to be_a(Net::HTTPSuccess)
-      @http.finish
-    end
-
-    it "should not leak file descriptors", net_connect: true do
-      sockets = Set.new
-
-      @http = Net::HTTP.new('example.org')
-      @http.start
-      sockets << @http.instance_variable_get(:@socket)
-      @http.get("/")
-      sockets << @http.instance_variable_get(:@socket)
-      @http.get("/")
-      sockets << @http.instance_variable_get(:@socket)
-      @http.get("/")
-      sockets << @http.instance_variable_get(:@socket)
-      @http.finish
-
-      if !defined?(WebMock::NetHTTPUtility) || WebMock.net_http_connect_on_start?(Addressable::URI.parse("http://example.com/"))
-        expect(sockets.length).to eq(1)
-        expect(sockets.to_a[0]).to be_a(Net::BufferedIO)
-      else
-        expect(sockets.length).to eq(2)
-        expect(sockets.to_a[0]).to be_a(StubSocket)
-        expect(sockets.to_a[1]).to be_a(Net::BufferedIO)
-      end
-
-      expect(sockets.all?(&:closed?)).to be(true)
-    end
-
-    it "should pass the read_timeout value on", net_connect: true do
-      @http = Net::HTTP.new('localhost', port)
-      read_timeout = @http.read_timeout + 1
-      @http.read_timeout = read_timeout
-      @http.start {|conn|
-        conn.request(Net::HTTP::Get.new("/"))
-        socket = conn.instance_variable_get(:@socket)
-        expect(socket.read_timeout).to eq(read_timeout)
-      }
-    end
-
-    describe "without start" do
-      it "should close connection after a real request" do
-        @http.get('/') { }
-        expect(@http).not_to be_started
-      end
-
-      it "should execute block exactly once" do
-        times = 0
-        @http.get('/') { times += 1 }
-        expect(times).to eq(1)
-      end
-
-      it "should have socket open during a real request" do
-        socket_id = nil
-        @http.get('/') {
-          socket_id = @http.instance_variable_get(:@socket).object_id
-        }
-        expect(socket_id).not_to be_nil
-      end
-
-      it "should be started during a real request" do
-        started = nil
-        @http.get('/') {
-          started = @http.started?
-        }
-        expect(started).to eq(true)
-        expect(@http.started?).to eq(false)
-      end
-    end
-
-    describe "with start" do
-      it "should close connection after a real request" do
-        @http.start {|conn| conn.get('/') { } }
-        expect(@http).not_to be_started
-      end
-
-      it "should execute block exactly once" do
-        times = 0
-        @http.start {|conn| conn.get('/') { times += 1 }}
-        expect(times).to eq(1)
-      end
-
-      it "should have socket open during a real request" do
-        socket_id = nil
-        @http.start {|conn| conn.get('/') {
-            socket_id = conn.instance_variable_get(:@socket).object_id
-          }
-        }
-        expect(socket_id).not_to be_nil
-      end
-
-      it "should be started during a real request" do
-        started = nil
-        @http.start {|conn| conn.get('/') {
-            started = conn.started?
-          }
-        }
-        expect(started).to eq(true)
-        expect(@http.started?).to eq(false)
-      end
-    end
-
-    describe "with start without request block" do
-      it "should close connection after a real request" do
-        @http.start {|conn| conn.get('/') }
-        expect(@http).not_to be_started
-      end
-
-      it "should have socket open during a real request" do
-        socket_id = nil
-        @http.start {|conn|
-          socket_id = conn.instance_variable_get(:@socket).object_id
-        }
-        expect(socket_id).not_to be_nil
-      end
-
-      it "should be started during a real request" do
-        started = nil
-        @http.start {|conn|
-          started = conn.started?
-        }
-        expect(started).to eq(true)
-        expect(@http.started?).to eq(false)
-      end
-    end
-
-    describe "with start without a block and finish" do
-      it "should gracefully start and close connection" do
-        @http.start
-        @http.get("/")
-        expect(@http).to be_started
-        @http.finish
-        expect(@http).not_to be_started
-      end
-    end
-  end
-end

data/spec/acceptance/net_http/net_http_spec.rb

@@ -1,395 +0,0 @@
-require 'spec_helper'
-require 'ostruct'
-require 'acceptance/webmock_shared'
-require 'acceptance/net_http/net_http_spec_helper'
-require 'acceptance/net_http/net_http_shared'
-
-include NetHTTPSpecHelper
-
-describe "Net:HTTP" do
-  include_examples "with WebMock", :no_url_auth
-
-  let(:port) { WebMockServer.instance.port }
-
-  describe "marshalling" do
-    class TestMarshalingInWebMockNetHTTP
-      attr_accessor :r
-    end
-    before(:each) do
-      @b = TestMarshalingInWebMockNetHTTP.new
-    end
-    after(:each) do
-      WebMock.enable!
-    end
-    it "should be possible to load object marshalled when webmock was disabled" do
-      WebMock.disable!
-      original_constants = [
-        Net::HTTP::Get,
-        Net::HTTP::Post,
-        Net::HTTP::Put,
-        Net::HTTP::Delete,
-        Net::HTTP::Head,
-        Net::HTTP::Options
-      ]
-      @b.r = original_constants
-      original_serialized = Marshal.dump(@b)
-      Marshal.load(original_serialized)
-      WebMock.enable!
-      Marshal.load(original_serialized)
-    end
-
-    it "should be possible to load object marshalled when webmock was enabled"  do
-      WebMock.enable!
-      new_constants = [
-        Net::HTTP::Get,
-        Net::HTTP::Post,
-        Net::HTTP::Put,
-        Net::HTTP::Delete,
-        Net::HTTP::Head,
-        Net::HTTP::Options
-      ]
-      @b.r = new_constants
-      new_serialized = Marshal.dump(@b)
-      Marshal.load(new_serialized)
-      WebMock.disable!
-      Marshal.load(new_serialized)
-    end
-  end
-
-  describe "constants" do
-    it "should still have const Get defined on replaced Net::HTTP" do
-      expect(Object.const_get("Net").const_get("HTTP").const_defined?("Get")).to be_truthy
-    end
-
-    it "should still have const Get within constants on replaced Net::HTTP" do
-      expect(Object.const_get("Net").const_get("HTTP").constants.map(&:to_s)).to include("Get")
-    end
-
-    it "should still have const Get within constants on replaced Net::HTTP" do
-      expect(Object.const_get("Net").const_get("HTTP").const_get("Get")).not_to be_nil
-    end
-
-    if Module.method(:const_defined?).arity != 1
-      it "should still have const Get defined (and not inherited) on replaced Net::HTTP" do
-        expect(Object.const_get("Net").const_get("HTTP").const_defined?("Get", false)).to be_truthy
-      end
-    end
-
-    if Module.method(:const_get).arity != 1
-      it "should still be able to get non inherited constant Get on replaced Net::HTTP" do
-        expect(Object.const_get("Net").const_get("HTTP").const_get("Get", false)).not_to be_nil
-      end
-    end
-
-    if Module.method(:constants).arity != 0
-      it "should still Get within non inherited constants on replaced Net::HTTP" do
-        expect(Object.const_get("Net").const_get("HTTP").constants(false).map(&:to_s)).to include("Get")
-      end
-    end
-
-    describe "after WebMock is disabled" do
-      after(:each) do
-        WebMock.enable!
-      end
-      it "Net::HTTP should have the same constants" do
-        orig_consts_number = WebMock::HttpLibAdapters::NetHttpAdapter::OriginalNetHTTP.constants.size
-        Net::HTTP.send(:const_set, "TEST_CONST", 10)
-        expect(Net::HTTP.constants.size).to eq(orig_consts_number + 1)
-        WebMock.disable!
-        expect(Net::HTTP.constants.size).to eq(orig_consts_number + 1)
-      end
-    end
-  end
-
-  it "should work with block provided" do
-    stub_http_request(:get, "www.example.com").to_return(body: "abc"*100000)
-    expect(Net::HTTP.start("www.example.com") { |query| query.get("/") }.body).to eq("abc"*100000)
-  end
-
-  it "should handle requests with raw binary data" do
-    body = "\x14\x00\x00\x00\x70\x69\x6e\x67\x00\x00"
-    stub_http_request(:post, "www.example.com").with(body: body).to_return(body: "abc")
-    req = Net::HTTP::Post.new("/")
-    req.body = body
-    req.content_type = "application/octet-stream"
-    expect(Net::HTTP.start("www.example.com") { |http| http.request(req)}.body).to eq("abc")
-  end
-
-  it "raises an ArgumentError if passed headers as symbols" do
-    uri = URI.parse("http://google.com/")
-    http = Net::HTTP.new(uri.host, uri.port)
-    request = Net::HTTP::Get.new(uri.request_uri)
-    request[:InvalidHeaderSinceItsASymbol] = "this will not be valid"
-
-    stub_http_request(:get, "google.com").with(headers: { InvalidHeaderSinceItsASymbol: "this will not be valid" })
-    expect do
-      http.request(request)
-    end.not_to raise_error
-  end
-
-  it "should handle multiple values for the same response header" do
-    stub_http_request(:get, "www.example.com").to_return(headers: { 'Set-Cookie' => ['foo=bar', 'bar=bazz'] })
-    response = Net::HTTP.get_response(URI.parse("http://www.example.com/"))
-    expect(response.get_fields('Set-Cookie')).to eq(['bar=bazz', 'foo=bar'])
-  end
-
-  it "should yield block on response" do
-    stub_http_request(:get, "www.example.com").to_return(body: "abc")
-    response_body = ""
-    http_request(:get, "http://www.example.com/") do |response|
-      response_body = response.body
-    end
-    expect(response_body).to eq("abc")
-  end
-
-  it "should handle Net::HTTP::Post#body" do
-    stub_http_request(:post, "www.example.com").with(body: "my_params").to_return(body: "abc")
-    req = Net::HTTP::Post.new("/")
-    req.body = "my_params"
-    expect(Net::HTTP.start("www.example.com") { |http| http.request(req)}.body).to eq("abc")
-  end
-
-  it "should handle Net::HTTP::Post#body_stream" do
-    stub_http_request(:post, "www.example.com").with(body: "my_params").to_return(body: "abc")
-    req = Net::HTTP::Post.new("/")
-    req.body_stream = StringIO.new("my_params")
-    expect(Net::HTTP.start("www.example.com") { |http| http.request(req)}.body).to eq("abc")
-  end
-
-  it "should behave like Net::HTTP and raise error if both request body and body argument are set" do
-    stub_http_request(:post, "www.example.com").with(body: "my_params").to_return(body: "abc")
-    req = Net::HTTP::Post.new("/")
-    req.body = "my_params"
-    expect {
-      Net::HTTP.start("www.example.com") { |http| http.request(req, "my_params")}
-    }.to raise_error("both of body argument and HTTPRequest#body set")
-  end
-
-  it "should return a Net::ReadAdapter from response.body when a stubbed request is made with a block and #read_body" do
-    WebMock.stub_request(:get, 'http://example.com/').to_return(body: "the body")
-    response = Net::HTTP.new('example.com', 80).request_get('/') { |r| r.read_body { } }
-    expect(response.body).to be_a(Net::ReadAdapter)
-  end
-
-  it "should have request 1 time executed in registry after 1 real request", net_connect: true do
-    WebMock.allow_net_connect!
-    http = Net::HTTP.new('localhost', port)
-    http.get('/') {}
-    expect(WebMock::RequestRegistry.instance.requested_signatures.hash.size).to eq(1)
-    expect(WebMock::RequestRegistry.instance.requested_signatures.hash.values.first).to eq(1)
-  end
-
-  it "should work with Addressable::URI passed to Net::HTTP.get_response" do
-    stub_request(:get, 'http://www.example.com/hello?a=1').to_return(body: "abc")
-    expect(Net::HTTP.get_response(Addressable::URI.parse('http://www.example.com/hello?a=1')).body).to eq("abc")
-  end
-
-  it "should support method calls on stubbed socket" do
-    WebMock.allow_net_connect!
-    stub_request(:get, 'www.google.com')#.with(headers: {"My-Header" => 99})
-    req = Net::HTTP::Get.new('/')
-    Net::HTTP.start('www.google.com') do |http|
-      http.request(req, '')
-      socket = http.instance_variable_get(:@socket)
-      expect(socket).to be_a(StubSocket)
-      expect { socket.io.setsockopt(Socket::IPPROTO_TCP, Socket::TCP_NODELAY, 1) }.to_not raise_error
-    end
-  end
-
-  if Gem::Version.new(RUBY_VERSION) >= Gem::Version.new('2.7.0')
-    it "uses the StubSocket to provide IP address" do
-      Net::HTTP.start("http://example.com") do |http|
-        expect(http.ipaddr).to eq("127.0.0.1")
-      end
-    end
-  end
-
-  it "defines common socket methods" do
-    Net::HTTP.start("http://example.com") do |http|
-      socket = http.instance_variable_get(:@socket)
-      expect(socket.io.ssl_version).to eq("TLSv1.3")
-      expect(socket.io.cipher).to eq(["TLS_AES_128_GCM_SHA256", "TLSv1.3", 128, 128])
-    end
-  end
-
-  describe "connecting on Net::HTTP.start" do
-    before(:each) do
-      @http = Net::HTTP.new('www.google.com', 443)
-      @http.use_ssl = true
-      @http.verify_mode = OpenSSL::SSL::VERIFY_NONE
-    end
-
-    describe "when net http is allowed" do
-      it "should not connect to the server until the request", net_connect: true do
-        WebMock.allow_net_connect!
-        @http.start {|conn|
-          expect(conn.peer_cert).to be_nil
-        }
-      end
-
-      it "should connect to the server on start", net_connect: true do
-        WebMock.allow_net_connect!(net_http_connect_on_start: true)
-        @http.start {|conn|
-          cert = OpenSSL::X509::Certificate.new conn.peer_cert
-          expect(cert).to be_a(OpenSSL::X509::Certificate)
-        }
-      end
-
-    end
-
-    describe "when net http is disabled and allowed only for some hosts" do
-      it "should not connect to the server until the request", net_connect: true do
-        WebMock.disable_net_connect!(allow: "www.google.com")
-        @http.start {|conn|
-          expect(conn.peer_cert).to be_nil
-        }
-      end
-
-      it "should connect to the server on start", net_connect: true do
-        WebMock.disable_net_connect!(allow: "www.google.com", net_http_connect_on_start: true)
-        @http.start {|conn|
-          cert = OpenSSL::X509::Certificate.new conn.peer_cert
-          expect(cert).to be_a(OpenSSL::X509::Certificate)
-        }
-      end
-
-      it "should connect to the server on start when allowlisted", net_connect: true do
-        WebMock.disable_net_connect!(allow: "www.google.com", net_http_connect_on_start: "www.google.com")
-        @http.start {|conn|
-          cert = OpenSSL::X509::Certificate.new conn.peer_cert
-          expect(cert).to be_a(OpenSSL::X509::Certificate)
-        }
-      end
-
-      it "should not connect to the server on start when not allowlisted", net_connect: true do
-        WebMock.disable_net_connect!(allow: "www.google.com", net_http_connect_on_start: "www.yahoo.com")
-        @http.start {|conn|
-          expect(conn.peer_cert).to be_nil
-        }
-      end
-
-      it "should connect to the server if the URI matches an regex", net_connect: true do
-        WebMock.disable_net_connect!(allow: /google.com/)
-        Net::HTTP.get('www.google.com','/')
-      end
-
-      it "should connect to the server if the URI matches any regex the array", net_connect: true do
-        WebMock.disable_net_connect!(allow: [/google.com/, /yahoo.com/])
-        Net::HTTP.get('www.google.com','/')
-      end
-
-    end
-
-  end
-
-  describe "when net_http_connect_on_start is true" do
-    before(:each) do
-      WebMock.allow_net_connect!(net_http_connect_on_start: true)
-    end
-    it_should_behave_like "Net::HTTP"
-  end
-
-  describe "when net_http_connect_on_start is false" do
-    before(:each) do
-      WebMock.allow_net_connect!(net_http_connect_on_start: false)
-    end
-    it_should_behave_like "Net::HTTP"
-  end
-
-  describe "when net_http_connect_on_start is a specific host" do
-    before(:each) do
-      WebMock.allow_net_connect!(net_http_connect_on_start: "localhost")
-    end
-    it_should_behave_like "Net::HTTP"
-  end
-
-  describe 'after_request callback support', net_connect: true do
-    let(:expected_body_regex) { /hello world/ }
-
-    before(:each) do
-      WebMock.allow_net_connect!
-      @callback_invocation_count = 0
-      WebMock.after_request do |_, response|
-        @callback_invocation_count += 1
-        @callback_response = response
-      end
-    end
-
-    after(:each) do
-      WebMock.reset_callbacks
-    end
-
-    def perform_get_with_returning_block
-      http_request(:get, "http://localhost:#{port}/") do |response|
-        return response.body
-      end
-    end
-
-    it "should support the after_request callback on an request with block and read_body" do
-      response_body = ''.dup
-      http_request(:get, "http://localhost:#{port}/") do |response|
-        response.read_body { |fragment| response_body << fragment }
-      end
-      expect(response_body).to match(expected_body_regex)
-
-      expect(@callback_response.body).to eq(response_body)
-    end
-
-    it "should support the after_request callback on a request with a returning block" do
-      response_body = perform_get_with_returning_block
-      expect(response_body).to match(expected_body_regex)
-      expect(@callback_response).to be_instance_of(WebMock::Response)
-      expect(@callback_response.body).to eq(response_body)
-    end
-
-    it "should only invoke the after_request callback once, even for a recursive post request" do
-      Net::HTTP.new('localhost', port).post('/', nil)
-      expect(@callback_invocation_count).to eq(1)
-    end
-  end
-
-  it "should match http headers, even if their values have been set in a request as numbers" do
-    WebMock.disable_net_connect!
-
-    stub_request(:post, "www.example.com").with(headers: {"My-Header" => 99})
-
-    uri = URI.parse('http://www.example.com/')
-    req = Net::HTTP::Post.new(uri.path)
-    req['My-Header'] = 99
-
-    res = Net::HTTP.start(uri.host, uri.port) do |http|
-      http.request(req, '')
-    end
-  end
-
-  describe "hostname handling" do
-    it "should set brackets around the hostname if it is an IPv6 address" do
-      net_http = Net::HTTP.new('b2dc:5bdf:4f0d::3014:e0ca', 80)
-      path = '/example.jpg'
-      expect(WebMock::NetHTTPUtility.get_uri(net_http, path)).to eq('http://[b2dc:5bdf:4f0d::3014:e0ca]:80/example.jpg')
-    end
-
-    it "should not set brackets around the hostname if it is already wrapped by brackets" do
-      net_http = Net::HTTP.new('[b2dc:5bdf:4f0d::3014:e0ca]', 80)
-      path = '/example.jpg'
-      expect(WebMock::NetHTTPUtility.get_uri(net_http, path)).to eq('http://[b2dc:5bdf:4f0d::3014:e0ca]:80/example.jpg')
-    end
-
-    it "should not set brackets around the hostname if it is an IPv4 address" do
-      net_http = Net::HTTP.new('181.152.137.168', 80)
-      path = '/example.jpg'
-      expect(WebMock::NetHTTPUtility.get_uri(net_http, path)).to eq('http://181.152.137.168:80/example.jpg')
-    end
-
-    it "should not set brackets around the hostname if it is a domain" do
-      net_http = Net::HTTP.new('www.example.com', 80)
-      path = '/example.jpg'
-      expect(WebMock::NetHTTPUtility.get_uri(net_http, path)).to eq('http://www.example.com:80/example.jpg')
-    end
-
-    it "does not require a path" do
-      net_http = Net::HTTP.new('www.example.com', 80)
-      expect(WebMock::NetHTTPUtility.get_uri(net_http)).to eq('http://www.example.com:80')
-    end
-  end
-end

data/spec/acceptance/net_http/net_http_spec_helper.rb

@@ -1,64 +0,0 @@
-module NetHTTPSpecHelper
-  def http_request(method, uri, options = {}, &block)
-    begin
-      uri = URI.parse(uri)
-    rescue
-      uri = Addressable::URI.heuristic_parse(uri)
-    end
-    response = nil
-    clazz = Net::HTTP.const_get("#{method.to_s.capitalize}")
-    req = clazz.new("#{uri.path}#{uri.query ? '?' : ''}#{uri.query}", nil)
-    options[:headers].each do |k,v|
-      if v.is_a?(Array)
-        v.each_with_index do |e,i|
-          i == 0 ? (req[k] = e) : req.add_field(k, e)
-        end
-      else
-        req[k] = v
-      end
-    end if options[:headers]
-
-    if options[:basic_auth]
-      req.basic_auth(options[:basic_auth][0], options[:basic_auth][1])
-    end
-
-    http = Net::HTTP.new(uri.host, uri.port)
-    if uri.scheme == "https"
-      http.use_ssl = true
-      #1.9.1 has a bug with ssl_timeout
-      http.ssl_timeout = 20 unless RUBY_PLATFORM =~ /java/
-      http.open_timeout = 60
-      http.read_timeout = 60
-    end
-    http.verify_mode = OpenSSL::SSL::VERIFY_NONE
-    response = http.start {|open_http|
-      open_http.request(req, options[:body], &block)
-    }
-    headers = {}
-    response.each_header {|name, value| headers[name] = value}
-    OpenStruct.new({
-      body: response.body,
-      headers: WebMock::Util::Headers.normalize_headers(headers),
-      status: response.code,
-      message: response.message
-    })
-  end
-
-  def client_timeout_exception_class
-    if defined?(Net::OpenTimeout)
-      Net::OpenTimeout
-    elsif defined?(Net::HTTP::OpenTimeout)
-      Net::HTTP::OpenTimeout
-    else
-      Timeout::Error
-    end
-  end
-
-  def connection_refused_exception_class
-    Errno::ECONNREFUSED
-  end
-
-  def http_library
-    :net_http
-  end
-end

data/spec/acceptance/net_http/real_net_http_spec.rb

@@ -1,20 +0,0 @@
-require 'rubygems'
-require 'rspec'
-require 'net/http'
-require 'net/https'
-require 'stringio'
-require 'acceptance/net_http/net_http_shared'
-require 'support/webmock_server'
-
-describe "Real Net:HTTP without webmock", without_webmock: true do
-  before(:all) do
-    raise "WebMock has no access here!!!" if defined?(WebMock::NetHTTPUtility)
-    WebMockServer.instance.start
-  end
-
-  after(:all) do
-    WebMockServer.instance.stop
-  end
-
-  it_should_behave_like "Net::HTTP"
-end