chef 18.8.11 → 18.8.54

data/spec/unit/http/api_versions_spec.rb

@@ -1,100 +0,0 @@
-#
-# Copyright:: Copyright (c) Chef Software Inc.
-# License:: Apache License, Version 2.0
-#
-# Licensed under the Apache License, Version 2.0 (the "License");
-# you may not use this file except in compliance with the License.
-# You may obtain a copy of the License at
-#
-#     http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing, software
-# distributed under the License is distributed on an "AS IS" BASIS,
-# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-# See the License for the specific language governing permissions and
-# limitations under the License.
-#
-
-require "spec_helper"
-
-describe Chef::HTTP::APIVersions do
-  class TestVersionClient < Chef::HTTP
-    use Chef::HTTP::APIVersions
-  end
-
-  before do
-    Chef::ServerAPIVersions.instance.reset!
-  end
-
-  let(:method) { "GET" }
-  let(:url) { "http://localhost:60123" }
-  let(:headers) { {} }
-  let(:data) { false }
-
-  let(:request) {}
-  let(:return_value) { "200" }
-
-  # Test Variables
-  let(:response_body) { "Thanks for checking in." }
-  let(:response_headers) do
-    {
-      "x-ops-server-api-version" => { "min_version" => 0, "max_version" => 2 }.to_json,
-    }
-  end
-
-  let(:response) do
-    m = double("HttpResponse", body: response_body)
-    allow(m).to receive(:key?).with("x-ops-server-api-version").and_return(true)
-    allow(m).to receive(:code).and_return(return_value)
-    allow(m).to receive(:[]) do |key|
-      response_headers[key]
-    end
-
-    m
-  end
-
-  let(:version_class) do
-    Class.new do
-      extend Chef::Mixin::VersionedAPIFactory
-
-      version_class_v0 = Class.new do
-        extend Chef::Mixin::VersionedAPI
-        minimum_api_version 0
-      end
-      add_versioned_api_class version_class_v0
-
-      version_class_v2 = Class.new do
-        extend Chef::Mixin::VersionedAPI
-        minimum_api_version 2
-      end
-      add_versioned_api_class version_class_v2
-    end
-  end
-
-  let(:client) do
-    TestVersionClient.new(url, { version_class: version_class })
-  end
-
-  let(:middleware) do
-    client.middlewares[0]
-  end
-
-  def run_api_version_handler
-    middleware.handle_request(method, url, headers, data)
-    middleware.handle_response(response, request, return_value)
-  end
-
-  it "correctly stores server api versions" do
-    run_api_version_handler
-    expect(Chef::ServerAPIVersions.instance.min_server_version).to eq(0)
-  end
-
-  context "with an unacceptable api version" do
-    let(:return_value) { "406" }
-    it "resets the list of supported versions" do
-      Chef::ServerAPIVersions.instance.set_versions({ "min_version" => 1, "max_version" => 3 })
-      run_api_version_handler
-      expect(Chef::ServerAPIVersions.instance.min_server_version).to eq(0)
-    end
-  end
-end

data/spec/unit/http/authenticator_spec.rb

@@ -1,247 +0,0 @@
-#
-# Author:: Tyler Cloke (<tyler@chef.io>)
-# Copyright:: Copyright (c) Chef Software Inc.
-# License:: Apache License, Version 2.0
-#
-# Licensed under the Apache License, Version 2.0 (the "License");
-# you may not use this file except in compliance with the License.
-# You may obtain a copy of the License at
-#
-#     http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing, software
-# distributed under the License is distributed on an "AS IS" BASIS,
-# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-# See the License for the specific language governing permissions and
-# limitations under the License.
-#
-
-require "spec_helper"
-require "chef/http/authenticator"
-require "chef/mixin/powershell_exec"
-
-require_relative "../../../lib/chef/win32/registry"
-
-describe Chef::HTTP::Authenticator, :windows_only do
-  let(:class_instance) { Chef::HTTP::Authenticator.new(client_name: "test") }
-  let(:method) { "GET" }
-  let(:url) { URI("https://chef.example.com/organizations/test") }
-  let(:headers) { {} }
-  let(:data) { "" }
-  let(:node_name) { "test" }
-  let(:passwrd) { "some_insecure_password" }
-
-  before(:each) do
-    Chef::Config[:node_name] = node_name
-    cert_name = "chef-#{node_name}"
-    d = Time.now
-    end_date = Time.new + (3600 * 24 * 90)
-    end_date = end_date.utc.iso8601
-
-    my_client = Chef::Client.new
-    class_instance.get_cert_password
-    pfx = my_client.generate_pfx_package(cert_name, end_date)
-    my_client.import_pfx_to_store(pfx)
-  end
-
-  after(:each) do
-    require "chef/mixin/powershell_exec"
-    extend Chef::Mixin::PowershellExec
-    cert_name = "chef-#{node_name}"
-    delete_certificate(cert_name)
-  end
-
-  context "when retrieving a certificate from the certificate store it" do
-    it "properly creates the password hive in the registry when it doesn't exist" do
-      delete_registry_hive
-      class_instance.get_cert_password
-      win32registry = Chef::Win32::Registry.new
-      expected_path = "HKEY_LOCAL_MACHINE\\Software\\Progress\\Authentication"
-      path_created = win32registry.key_exists?(expected_path)
-      expect(path_created).to be(true)
-    end
-
-    it "retrieves a certificate password from the registry when the hive does not already exist" do
-      delete_registry_hive
-      password = class_instance.get_cert_password
-      expect { class_instance.get_cert_password }.not_to raise_error
-      expect(password).not_to be(nil)
-    end
-
-    it "should return a password of at least 14 characters in length" do
-      password = class_instance.get_cert_password
-      expect(password.length).to eql(14)
-    end
-
-    it "will retrieve a password from a partial registry hive and upgrades it while using the old decryptor" do
-      delete_registry_hive
-      load_partial_registry_hive
-      password = class_instance.get_cert_password
-      expect(password).to eql(passwrd)
-    end
-
-    it "verifies that the new password is now using a vector" do
-      win32registry = Chef::Win32::Registry.new
-      path = "HKEY_LOCAL_MACHINE\\Software\\Progress\\Authentication"
-      password_blob = win32registry.get_values(path)
-      if password_blob.nil? || password_blob.empty?
-        raise Chef::Exceptions::Win32RegKeyMissing
-      end
-
-      raw_data = password_blob.map { |x| x[:data] }
-      vector = raw_data[2]
-      expect(vector).not_to be(nil)
-    end
-
-    it "correctly retrieves a valid certificate in pem format from the LocalMachine certstore" do
-      require "openssl"
-      certificate = class_instance.retrieve_certificate_key(node_name)
-      cert_object = OpenSSL::PKey::RSA.new(certificate)
-      expect(cert_object.to_s).to match(/BEGIN RSA PRIVATE KEY/)
-    end
-  end
-
-  def load_partial_registry_hive
-    extend Chef::Mixin::PowershellExec
-    password = "some_insecure_password"
-    powershell_code = <<~CODE
-      $encrypted_string = ConvertTo-SecureString "#{password}" -AsPlainText -Force
-      $secure_string = ConvertFrom-SecureString $encrypted_string
-      return $secure_string
-    CODE
-    encrypted_pass = powershell_exec!(powershell_code).result
-    Chef::Config[:auth_key_registry_type] == "user" ? store = "HKEY_CURRENT_USER" : store = "HKEY_LOCAL_MACHINE"
-    hive_path = "#{store}\\Software\\Progress\\Authentication"
-    win32registry = Chef::Win32::Registry.new
-    unless win32registry.key_exists?(hive_path)
-      win32registry.create_key(hive_path, true)
-    end
-    values = { name: "PfxPass", type: :string, data: encrypted_pass }
-    win32registry.set_value(hive_path, values)
-  end
-
-  def delete_registry_hive
-    win32registry = Chef::Win32::Registry.new
-    hive_path = "HKEY_LOCAL_MACHINE\\Software\\Progress"
-    if win32registry.key_exists?(hive_path)
-      win32registry.delete_key(hive_path, true)
-    end
-  end
-
-  def delete_certificate(cert_name)
-    powershell_code = <<~CODE
-      Get-ChildItem -path cert:\\LocalMachine\\My -Recurse -Force  | Where-Object { $_.Subject -Match "#{cert_name}" } | Remove-item
-    CODE
-    powershell_exec!(powershell_code)
-  end
-end
-
-describe Chef::HTTP::Authenticator do
-  let(:class_instance) { Chef::HTTP::Authenticator.new(client_name: "test") }
-  let(:method) { "GET" }
-  let(:url) { URI("https://chef.example.com/organizations/test") }
-  let(:headers) { {} }
-  let(:data) { "" }
-
-  before do
-    ::Chef::Config[:node_name] = "foo"
-  end
-
-  context "when handle_request is called" do
-    shared_examples_for "merging the server API version into the headers" do
-      before do
-        allow(class_instance).to receive(:authentication_headers).and_return({})
-      end
-
-      it "merges the default version of X-Ops-Server-API-Version into the headers" do
-        # headers returned
-        expect(class_instance.handle_request(method, url, headers, data)[2])
-          .to include({ "X-Ops-Server-API-Version" => Chef::HTTP::Authenticator::DEFAULT_SERVER_API_VERSION })
-      end
-
-      context "when version_class is provided" do
-        class V0Class; extend Chef::Mixin::VersionedAPI; minimum_api_version 0; end
-        class V2Class; extend Chef::Mixin::VersionedAPI; minimum_api_version 2; end
-
-        class AuthFactoryClass
-          extend Chef::Mixin::VersionedAPIFactory
-          add_versioned_api_class V0Class
-          add_versioned_api_class V2Class
-        end
-
-        let(:class_instance) { Chef::HTTP::Authenticator.new({ version_class: AuthFactoryClass }) }
-
-        it "uses it to select the correct http version" do
-          Chef::ServerAPIVersions.instance.reset!
-          expect(AuthFactoryClass).to receive(:best_request_version).and_call_original
-          expect(class_instance.handle_request(method, url, headers, data)[2])
-            .to include({ "X-Ops-Server-API-Version" => "2" })
-        end
-      end
-
-      context "when api_version is set to something other than the default" do
-        let(:class_instance) { Chef::HTTP::Authenticator.new({ api_version: "-10" }) }
-
-        it "merges the requested version of X-Ops-Server-API-Version into the headers" do
-          expect(class_instance.handle_request(method, url, headers, data)[2])
-            .to include({ "X-Ops-Server-API-Version" => "-10" })
-        end
-      end
-    end
-
-    context "when !sign_requests?" do
-      before do
-        allow(class_instance).to receive(:sign_requests?).and_return(false)
-      end
-
-      it_behaves_like "merging the server API version into the headers"
-
-      it "authentication_headers is not called" do
-        expect(class_instance).to_not receive(:authentication_headers)
-        class_instance.handle_request(method, url, headers, data)
-      end
-
-    end
-
-    context "when sign_requests?" do
-      before do
-        allow(class_instance).to receive(:sign_requests?).and_return(true)
-      end
-
-      it_behaves_like "merging the server API version into the headers"
-
-      it "calls authentication_headers with the proper input" do
-        expect(class_instance).to receive(:authentication_headers).with(
-          method, url, data,
-          { "X-Ops-Server-API-Version" => Chef::HTTP::Authenticator::DEFAULT_SERVER_API_VERSION }
-        ).and_return({})
-        class_instance.handle_request(method, url, headers, data)
-      end
-    end
-
-    context "when ssh_agent_signing" do
-      let(:public_key) { <<~EOH }
-        -----BEGIN PUBLIC KEY-----
-        MIIBIjANBgkqhkiG9w0BAQEFAAOCAQ8AMIIBCgKCAQEA49TA0y81ps0zxkOpmf5V
-        4/c4IeR5yVyQFpX3JpxO4TquwnRh8VSUhrw8kkTLmB3cS39Db+3HadvhoqCEbqPE
-        6915kXSuk/cWIcNozujLK7tkuPEyYVsyTioQAddSdfe+8EhQVf3oHxaKmUd6waXr
-        WqYCnhxgOjxocenREYNhZ/OETIeiPbOku47vB4nJK/0GhKBytL2XnsRgfKgDxf42
-        BqAi1jglIdeq8lAWZNF9TbNBU21AO1iuT7Pm6LyQujhggPznR5FJhXKRUARXBJZa
-        wxpGV4dGtdcahwXNE4601aXPra+xPcRd2puCNoEDBzgVuTSsLYeKBDMSfs173W1Q
-        YwIDAQAB
-        -----END PUBLIC KEY-----
-      EOH
-
-      let(:class_instance) { Chef::HTTP::Authenticator.new(client_name: "test", raw_key: public_key, ssh_agent_signing: true) }
-
-      it "sets use_ssh_agent if needed" do
-        expect(Mixlib::Authentication::SignedHeaderAuth).to receive(:signing_object).and_wrap_original { |m, *args|
-          m.call(*args).tap do |signing_obj|
-            expect(signing_obj).to receive(:sign).with(instance_of(OpenSSL::PKey::RSA), use_ssh_agent: true).and_return({})
-          end
-        }
-        class_instance.handle_request(method, url, headers, data)
-      end
-    end
-  end
-end

data/spec/unit/http/basic_client_spec.rb

@@ -1,102 +0,0 @@
-#
-# Author:: Cameron Cope (<ccope@brightcove.com>)
-# License:: Apache License, Version 2.0
-#
-# Licensed under the Apache License, Version 2.0 (the "License");
-# you may not use this file except in compliance with the License.
-# You may obtain a copy of the License at
-#
-#     http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing, software
-# distributed under the License is distributed on an "AS IS" BASIS,
-# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-# See the License for the specific language governing permissions and
-# limitations under the License.
-#
-
-require "spec_helper"
-require "chef/http/basic_client"
-
-describe "HTTP Connection" do
-
-  let(:uri) { URI("https://example.com:4443") }
-  subject(:basic_client) { Chef::HTTP::BasicClient.new(uri) }
-
-  describe ".new" do
-    it "creates an instance" do
-      subject
-    end
-  end
-
-  describe "#initialize" do
-    it "calls .start when doing keepalives" do
-      basic_client = Chef::HTTP::BasicClient.new(uri, keepalives: true)
-      expect(basic_client).to receive(:configure_ssl)
-      net_http_mock = instance_double(Net::HTTP, proxy_address: nil, "proxy_port=" => nil, "read_timeout=" => nil, "open_timeout=" => nil)
-      expect(net_http_mock).to receive(:start).and_return(net_http_mock)
-      expect(Net::HTTP).to receive(:new).and_return(net_http_mock)
-      expect(basic_client.http_client).to eql(net_http_mock)
-    end
-
-    it "does not call .start when not doing keepalives" do
-      basic_client = Chef::HTTP::BasicClient.new(uri)
-      expect(basic_client).to receive(:configure_ssl)
-      net_http_mock = instance_double(Net::HTTP, proxy_address: nil, "proxy_port=" => nil, "read_timeout=" => nil, "open_timeout=" => nil)
-      expect(net_http_mock).not_to receive(:start)
-      expect(Net::HTTP).to receive(:new).and_return(net_http_mock)
-      expect(basic_client.http_client).to eql(net_http_mock)
-    end
-
-    it "allows setting net-http accessor options" do
-      basic_client = Chef::HTTP::BasicClient.new(uri, nethttp_opts: {
-        "continue_timeout" => 5,
-        "max_retries" => 5,
-        "read_timeout" => 5,
-        "write_timeout" => 5,
-        "ssl_timeout" => 5,
-      })
-      expect(basic_client.http_client.continue_timeout).to eql(5)
-      expect(basic_client.http_client.max_retries).to eql(5)
-      expect(basic_client.http_client.read_timeout).to eql(5)
-      expect(basic_client.http_client.write_timeout).to eql(5)
-      expect(basic_client.http_client.ssl_timeout).to eql(5)
-    end
-
-    it "allows setting net-http accssor options as symbols" do
-      basic_client = Chef::HTTP::BasicClient.new(uri, nethttp_opts: {
-        continue_timeout: 5,
-        max_retries: 5,
-        read_timeout: 5,
-        write_timeout: 5,
-        ssl_timeout: 5,
-      })
-      expect(basic_client.http_client.continue_timeout).to eql(5)
-      expect(basic_client.http_client.max_retries).to eql(5)
-      expect(basic_client.http_client.read_timeout).to eql(5)
-      expect(basic_client.http_client.write_timeout).to eql(5)
-      expect(basic_client.http_client.ssl_timeout).to eql(5)
-    end
-  end
-
-  describe "#build_http_client" do
-    it "should build an http client" do
-      subject.build_http_client
-    end
-
-    it "should set an open timeout" do
-      expect(subject.build_http_client.open_timeout).not_to be_nil
-    end
-  end
-
-  describe "#proxy_uri" do
-    subject(:proxy_uri) { basic_client.proxy_uri }
-
-    it "uses ChefConfig's proxy_uri method" do
-      expect(ChefConfig::Config).to receive(:proxy_uri).at_least(:once).with(
-        uri.scheme, uri.host, uri.port
-      )
-      proxy_uri
-    end
-  end
-end

data/spec/unit/http/http_request_spec.rb

@@ -1,97 +0,0 @@
-#
-# Author:: Klaas Jan Wierenga (<k.j.wierenga@gmail.com>)
-# Copyright:: Copyright (c) Chef Software Inc.
-# License:: Apache License, Version 2.0
-#
-# Licensed under the Apache License, Version 2.0 (the "License");
-# you may not use this file except in compliance with the License.
-# You may obtain a copy of the License at
-#
-#     http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing, software
-# distributed under the License is distributed on an "AS IS" BASIS,
-# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-# See the License for the specific language governing permissions and
-# limitations under the License.
-#
-
-require "spec_helper"
-
-describe Chef::HTTP::HTTPRequest do
-
-  context "with HTTP url scheme" do
-
-    it "should not include port 80 in Host header" do
-      request = Chef::HTTP::HTTPRequest.new(:GET, URI("http://dummy.com"), "")
-
-      expect(request.headers["Host"]).to eql("dummy.com")
-    end
-
-    it "should not include explicit port 80 in Host header" do
-      request = Chef::HTTP::HTTPRequest.new(:GET, URI("http://dummy.com:80"), "")
-
-      expect(request.headers["Host"]).to eql("dummy.com")
-    end
-
-    it "should include explicit port 8000 in Host header" do
-      request = Chef::HTTP::HTTPRequest.new(:GET, URI("http://dummy.com:8000"), "")
-
-      expect(request.headers["Host"]).to eql("dummy.com:8000")
-    end
-
-    it "should include explicit 443 port in Host header" do
-      request = Chef::HTTP::HTTPRequest.new(:GET, URI("http://dummy.com:443"), "")
-
-      expect(request.headers["Host"]).to eql("dummy.com:443")
-    end
-
-    it "should pass on explicit Host header unchanged" do
-      request = Chef::HTTP::HTTPRequest.new(:GET, URI("http://dummy.com:8000"), "", { "Host" => "yourhost.com:8888" })
-
-      expect(request.headers["Host"]).to eql("yourhost.com:8888")
-    end
-
-    it "should not mutate the URI when it contains parameters" do
-      # buggy constructor code mutated strings owned by the URI parameter
-      uri = URI("http://dummy.com/foo?bar=baz")
-      request = Chef::HTTP::HTTPRequest.new(:GET, uri, "")
-      expect(uri).to eql(URI("http://dummy.com/foo?bar=baz"))
-    end
-  end
-
-  context "with HTTPS url scheme" do
-
-    it "should not include port 443 in Host header" do
-      request = Chef::HTTP::HTTPRequest.new(:GET, URI("https://dummy.com"), "")
-
-      expect(request.headers["Host"]).to eql("dummy.com")
-    end
-
-    it "should include explicit port 80 in Host header" do
-      request = Chef::HTTP::HTTPRequest.new(:GET, URI("https://dummy.com:80"), "")
-
-      expect(request.headers["Host"]).to eql("dummy.com:80")
-    end
-
-    it "should include explicit port 8000 in Host header" do
-      request = Chef::HTTP::HTTPRequest.new(:GET, URI("https://dummy.com:8000"), "")
-
-      expect(request.headers["Host"]).to eql("dummy.com:8000")
-    end
-
-    it "should not include explicit port 443 in Host header" do
-      request = Chef::HTTP::HTTPRequest.new(:GET, URI("https://dummy.com:443"), "")
-
-      expect(request.headers["Host"]).to eql("dummy.com")
-    end
-
-  end
-
-  it "should pass on explicit Host header unchanged" do
-    request = Chef::HTTP::HTTPRequest.new(:GET, URI("http://dummy.com:8000"), "", { "Host" => "myhost.com:80" })
-
-    expect(request.headers["Host"]).to eql("myhost.com:80")
-  end
-
-end

data/spec/unit/http/json_input_spec.rb

@@ -1,128 +0,0 @@
-#--
-# Author:: Daniel DeLeo (<dan@chef.io>)
-# Copyright:: Copyright (c) Chef Software Inc.
-# License:: Apache License, Version 2.0
-#
-# Licensed under the Apache License, Version 2.0 (the "License");
-# you may not use this file except in compliance with the License.
-# You may obtain a copy of the License at
-#
-#     http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing, software
-# distributed under the License is distributed on an "AS IS" BASIS,
-# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-# See the License for the specific language governing permissions and
-# limitations under the License.
-#
-
-require "spec_helper"
-require "chef/http/json_input"
-
-describe Chef::HTTP::JSONInput do
-
-  let(:json_encoder) { described_class.new }
-
-  let(:url) { URI.parse("http://example.com") }
-  let(:headers) { {} }
-
-  def handle_request
-    json_encoder.handle_request(http_method, url, headers, data)
-  end
-
-  it "passes the response unmodified" do
-    http_response = double("Net::HTTPSuccess")
-    request = double("Chef::HTTP::HTTPRequest")
-    return_value = "response body"
-
-    result = json_encoder.handle_response(http_response, request, return_value)
-    expect(result).to eq([http_response, request, return_value])
-  end
-
-  it "doesn't handle streaming responses" do
-    http_response = double("Net::HTTPSuccess")
-    expect(json_encoder.stream_response_handler(http_response)).to be nil
-  end
-
-  it "does nothing for stream completion" do
-    http_response = double("Net::HTTPSuccess")
-    request = double("Chef::HTTP::HTTPRequest")
-    return_value = "response body"
-
-    result = json_encoder.handle_response(http_response, request, return_value)
-    expect(result).to eq([http_response, request, return_value])
-  end
-
-  context "when handling a request with no body" do
-
-    let(:http_method) { :get }
-    let(:data) { nil }
-
-    it "passes the request unmodified" do
-      expect(handle_request).to eq([http_method, url, headers, data])
-    end
-  end
-
-  context "when the request should be serialized" do
-
-    let(:http_method) { :put }
-    let(:data) { { foo: "bar" } }
-    let(:expected_data) { %q[{"foo":"bar"}] }
-
-    context "and the request has a ruby object as the body and no explicit content-type" do
-
-      it "serializes the body to json" do
-        # Headers Hash get mutated, so we start by asserting it's empty:
-        expect(headers).to be_empty
-
-        expect(handle_request).to eq([http_method, url, headers, expected_data])
-
-        # Now the headers Hash should have json content type:
-        expect(headers).to have_key("Content-Type")
-        expect(headers["Content-Type"]).to eq("application/json")
-      end
-    end
-
-    context "ant the request has an explicit content type of json" do
-
-      it "serializes the body to json when content-type is all lowercase" do
-        headers["content-type"] = "application/json"
-
-        expect(handle_request).to eq([http_method, url, headers, expected_data])
-
-        # Content-Type header should be normalized:
-        expect(headers.size).to eq(1)
-        expect(headers).to have_key("Content-Type")
-        expect(headers["Content-Type"]).to eq("application/json")
-      end
-
-    end
-
-  end
-
-  context "when handling a request with an explicit content type other than json" do
-
-    let(:http_method) { :put }
-    let(:data) { "some arbitrary bytes" }
-
-    it "does not serialize the body to json when content type is given as lowercase" do
-      headers["content-type"] = "application/x-binary"
-
-      expect(handle_request).to eq([http_method, url, headers, data])
-
-      # not normalized
-      expect(headers).to eq({ "content-type" => "application/x-binary" })
-    end
-
-    it "does not serialize the body to json when content type is given in capitalized form" do
-      headers["Content-Type"] = "application/x-binary"
-
-      expect(handle_request).to eq([http_method, url, headers, data])
-
-      # not normalized
-      expect(headers).to eq({ "Content-Type" => "application/x-binary" })
-    end
-
-  end
-
-end