httpi-ntlm 0.9.6

data/spec/httpi/adapter/httpclient_spec.rb

@@ -0,0 +1,164 @@
+require "spec_helper"
+require "httpi/adapter/httpclient"
+require "httpi/request"
+
+require "httpclient"
+
+describe HTTPI::Adapter::HTTPClient do
+  let(:adapter) { HTTPI::Adapter::HTTPClient.new }
+  let(:httpclient) { HTTPClient.any_instance }
+  let(:ssl_config) { HTTPClient::SSLConfig.any_instance }
+
+  describe "#get" do
+    it "returns a valid HTTPI::Response" do
+      httpclient.expects(:get).with(basic_request.url, nil, basic_request.headers).returns(http_message)
+      adapter.get(basic_request).should match_response(:body => Fixture.xml)
+    end
+  end
+
+  describe "#post" do
+    it "returns a valid HTTPI::Response" do
+      request = HTTPI::Request.new :url => "http://example.com", :body => Fixture.xml
+      httpclient.expects(:post).with(request.url, request.body, request.headers).returns(http_message)
+
+      adapter.post(request).should match_response(:body => Fixture.xml)
+    end
+  end
+
+  describe "#head" do
+    it "returns a valid HTTPI::Response" do
+      httpclient.expects(:head).with(basic_request.url, nil, basic_request.headers).returns(http_message)
+      adapter.head(basic_request).should match_response(:body => Fixture.xml)
+    end
+  end
+
+  describe "#put" do
+    it "returns a valid HTTPI::Response" do
+      request = HTTPI::Request.new :url => "http://example.com", :body => Fixture.xml
+      httpclient.expects(:put).with(request.url, request.body, request.headers).returns(http_message)
+
+      adapter.put(request).should match_response(:body => Fixture.xml)
+    end
+  end
+
+  describe "#delete" do
+    it "returns a valid HTTPI::Response" do
+      httpclient.expects(:delete).with(basic_request.url, basic_request.headers).returns(http_message(""))
+      adapter.delete(basic_request).should match_response(:body => "")
+    end
+  end
+
+  describe "settings:" do
+    before { httpclient.stubs(:get).returns(http_message) }
+
+    describe "proxy" do
+      it "have should specs"
+    end
+
+    describe "connect_timeout" do
+      it "is not set unless specified" do
+        httpclient.expects(:connect_timeout=).never
+        adapter.get(basic_request)
+      end
+
+      it "is set if specified" do
+        request = basic_request { |request| request.open_timeout = 30 }
+
+        httpclient.expects(:connect_timeout=).with(30)
+        adapter.get(request)
+      end
+    end
+
+    describe "receive_timeout" do
+      it "is not set unless specified" do
+        httpclient.expects(:receive_timeout=).never
+        adapter.get(basic_request)
+      end
+
+      it "is set if specified" do
+        request = basic_request { |request| request.read_timeout = 30 }
+
+        httpclient.expects(:receive_timeout=).with(30)
+        adapter.get(request)
+      end
+    end
+
+    describe "set_auth" do
+      it "is set for HTTP basic auth" do
+        request = basic_request { |request| request.auth.basic "username", "password" }
+
+        httpclient.expects(:set_auth).with(request.url, *request.auth.credentials)
+        adapter.get(request)
+      end
+
+      it "is set for HTTP digest auth" do
+        request = basic_request { |request| request.auth.digest "username", "password" }
+
+        httpclient.expects(:set_auth).with(request.url, *request.auth.credentials)
+        adapter.get(request)
+      end
+    end
+
+    context "(for SSL client auth)" do
+      let(:ssl_auth_request) do
+        basic_request do |request|
+          request.auth.ssl.cert_key_file = "spec/fixtures/client_key.pem"
+          request.auth.ssl.cert_file = "spec/fixtures/client_cert.pem"
+        end
+      end
+
+      it "client_cert, client_key and verify_mode should be set" do
+        ssl_config.expects(:client_cert=).with(ssl_auth_request.auth.ssl.cert)
+        ssl_config.expects(:client_key=).with(ssl_auth_request.auth.ssl.cert_key)
+        ssl_config.expects(:verify_mode=).with(ssl_auth_request.auth.ssl.openssl_verify_mode)
+
+        adapter.get(ssl_auth_request)
+      end
+
+      it "sets the client_ca if specified" do
+        ssl_auth_request.auth.ssl.ca_cert_file = "spec/fixtures/client_cert.pem"
+        ssl_config.expects(:client_ca=).with(ssl_auth_request.auth.ssl.ca_cert)
+
+        adapter.get(ssl_auth_request)
+      end
+    end
+
+    context "(for SSL client auth with a verify mode of :none with no certs provided)" do
+      let(:ssl_auth_request) do
+        basic_request do |request|
+          request.auth.ssl.verify_mode = :none
+        end
+      end
+
+      it "verify_mode should be set" do
+        ssl_config.expects(:verify_mode=).with(ssl_auth_request.auth.ssl.openssl_verify_mode)
+
+        adapter.get(ssl_auth_request)
+      end
+
+      it "does not set client_cert and client_key "do
+        ssl_config.expects(:client_cert=).never
+        ssl_config.expects(:client_key=).never
+
+        adapter.get(ssl_auth_request)
+      end
+
+      it "does not raise an exception" do
+        expect { adapter.get(ssl_auth_request) }.to_not raise_error
+      end
+    end
+  end
+
+  def http_message(body = Fixture.xml)
+    message = HTTP::Message.new_response body
+    message.header.set "Accept-encoding", "utf-8"
+    message
+  end
+
+  def basic_request
+    request = HTTPI::Request.new "http://example.com"
+    yield request if block_given?
+    request
+  end
+
+end

data/spec/httpi/adapter/net_http_spec.rb

@@ -0,0 +1,142 @@
+require "spec_helper"
+require "httpi/adapter/net_http"
+require "httpi/request"
+
+describe HTTPI::Adapter::NetHTTP do
+  let(:net_http) { Net::HTTP.any_instance }
+  let(:basic_response) { { :body => Fixture.xml, :headers => { "Accept-encoding" => "utf-8" } } }
+
+  def adapter(request)
+    @adapter ||= HTTPI::Adapter::NetHTTP.new request
+  end
+
+  describe "#get" do
+    it "should return a valid HTTPI::Response" do
+      stub_request(:get, basic_request.url.to_s).to_return(basic_response)
+      adapter(basic_request).get(basic_request).should match_response(:body => Fixture.xml)
+    end
+  end
+
+  describe "#post" do
+    it "should return a valid HTTPI::Response" do
+      request = HTTPI::Request.new :url => "http://example.com", :body => Fixture.xml
+      stub_request(:post, request.url.to_s).with(:body => request.body).to_return(basic_response)
+
+      adapter(request).post(request).should match_response(:body => Fixture.xml)
+    end
+  end
+
+  describe "#head" do
+    it "should return a valid HTTPI::Response" do
+      stub_request(:head, basic_request.url.to_s).to_return(basic_response)
+      adapter(basic_request).head(basic_request).should match_response(:body => Fixture.xml)
+    end
+  end
+
+  describe "#put" do
+    it "should return a valid HTTPI::Response" do
+      request = HTTPI::Request.new(
+        :url     => "http://example.com",
+        :headers => { "Accept-encoding" => "utf-8" },
+        :body    => Fixture.xml
+      )
+      stub_request(:put, request.url.to_s).with(:body => request.body).to_return(basic_response)
+
+      adapter(request).put(request).should match_response(:body => Fixture.xml)
+    end
+  end
+
+  describe "#delete" do
+    it "should return a valid HTTPI::Response" do
+      stub_request(:delete, basic_request.url.to_s).to_return(basic_response)
+      adapter(basic_request).delete(basic_request).should match_response(:body => Fixture.xml)
+    end
+  end
+
+  describe "settings:" do
+    before { stub_request(:get, basic_request.url.to_s) }
+
+    describe "use_ssl" do
+      it "should be set to false for non-SSL requests" do
+        net_http.expects(:use_ssl=).with(false)
+        adapter(basic_request).get(basic_request)
+      end
+
+      it "should be set to true for SSL requests" do
+        request = basic_request { |request| request.ssl = true }
+
+        net_http.expects(:use_ssl=).with(true)
+        adapter(request).get(request)
+      end
+    end
+
+    describe "open_timeout" do
+      it "should not be set if not specified" do
+        net_http.expects(:open_timeout=).never
+        adapter(basic_request).get(basic_request)
+      end
+
+      it "should be set if specified" do
+        request = basic_request { |request| request.open_timeout = 30 }
+
+        net_http.expects(:open_timeout=).with(30)
+        adapter(request).get(request)
+      end
+    end
+
+    describe "read_timeout" do
+      it "should not be set if not specified" do
+        net_http.expects(:read_timeout=).never
+        adapter(basic_request).get(basic_request)
+      end
+
+      it "should be set if specified" do
+        request = basic_request { |request| request.read_timeout = 30 }
+
+        net_http.expects(:read_timeout=).with(30)
+        adapter(request).get(request)
+      end
+    end
+
+    describe "basic_auth" do
+      it "should be set for HTTP basic auth" do
+        request = basic_request { |request| request.auth.basic "username", "password" }
+
+        stub_request(:get, "http://username:password@example.com")
+        Net::HTTP::Get.any_instance.expects(:basic_auth).with(*request.auth.credentials)
+        adapter(request).get(request)
+      end
+    end
+
+    context "(for SSL client auth)" do
+      let(:ssl_auth_request) do
+        basic_request do |request|
+          request.auth.ssl.cert_key_file = "spec/fixtures/client_key.pem"
+          request.auth.ssl.cert_file = "spec/fixtures/client_cert.pem"
+        end
+      end
+
+      it "key, cert and verify_mode should be set" do
+        net_http.expects(:cert=).with(ssl_auth_request.auth.ssl.cert)
+        net_http.expects(:key=).with(ssl_auth_request.auth.ssl.cert_key)
+        net_http.expects(:verify_mode=).with(ssl_auth_request.auth.ssl.openssl_verify_mode)
+
+        adapter(ssl_auth_request).get(ssl_auth_request)
+      end
+
+      it "should set the client_ca if specified" do
+        ssl_auth_request.auth.ssl.ca_cert_file = "spec/fixtures/client_cert.pem"
+        net_http.expects(:ca_file=).with(ssl_auth_request.auth.ssl.ca_cert_file)
+
+        adapter(ssl_auth_request).get(ssl_auth_request)
+      end
+    end
+  end
+
+  def basic_request
+    request = HTTPI::Request.new "http://example.com"
+    yield request if block_given?
+    request
+  end
+
+end

data/spec/httpi/adapter_spec.rb

@@ -0,0 +1,55 @@
+require "spec_helper"
+require "httpi/adapter"
+
+describe HTTPI::Adapter do
+  let(:adapter) { HTTPI::Adapter }
+
+  describe ".use" do
+    around do |example|
+      adapter.use = nil
+      example.run
+      adapter.use = nil
+    end
+
+    it "sets the adapter to use" do
+      adapter.use.should_not == :net_http
+
+      adapter.use = :net_http
+      adapter.use.should == :net_http
+    end
+
+    it "defaults to use the HTTPClient adapter" do
+      adapter.use.should == :httpclient
+    end
+
+    it "loads the adapter's client library" do
+      adapter.expects(:require).with("httpclient")
+      adapter.use = :httpclient
+    end
+
+    it "raises an ArgumentError in case of an invalid adapter" do
+      expect { adapter.use = :unknown }.to raise_error(ArgumentError)
+    end
+  end
+
+  describe ".load" do
+    context "called with a valid adapter" do
+      it "returns the adapter's name and class" do
+        adapter.load(:curb).should == [:curb, HTTPI::Adapter::Curb]
+      end
+    end
+
+    context "called with nil" do
+      it "returns the default adapter's name and class" do
+        adapter.load(nil).should == [:httpclient, HTTPI::Adapter::HTTPClient]
+      end
+    end
+
+    context "called with an invalid adapter" do
+      it "raises an ArgumentError" do
+        expect { adapter.use = :unknown }.to raise_error(ArgumentError)
+      end
+    end
+  end
+
+end

data/spec/httpi/auth/config_spec.rb

@@ -0,0 +1,117 @@
+require "spec_helper"
+require "httpi/auth/config"
+
+describe HTTPI::Auth::Config do
+  let(:auth) { HTTPI::Auth::Config.new }
+
+  describe "#basic" do
+    it "lets you specify the basic auth credentials" do
+      auth.basic "username", "password"
+      auth.basic.should == ["username", "password"]
+    end
+
+    it "also accepts an Array of credentials" do
+      auth.basic ["username", "password"]
+      auth.basic.should == ["username", "password"]
+    end
+
+    it "sets the authentication type to :basic" do
+      auth.basic "username", "password"
+      auth.type.should == :basic
+    end
+  end
+
+  describe "#basic?" do
+    it "defaults to return false" do
+      auth.should_not be_basic
+    end
+
+    it "returns true for HTTP basic auth" do
+      auth.basic "username", "password"
+      auth.should be_basic
+    end
+  end
+
+  describe "#digest" do
+    it "lets you specify the digest auth credentials" do
+      auth.digest "username", "password"
+      auth.digest.should == ["username", "password"]
+    end
+
+    it "also accepts an Array of credentials" do
+      auth.digest ["username", "password"]
+      auth.digest.should == ["username", "password"]
+    end
+
+    it "sets the authentication type to :digest" do
+      auth.digest "username", "password"
+      auth.type.should == :digest
+    end
+  end
+
+  describe "#digest?" do
+    it "defaults to return false" do
+      auth.should_not be_digest
+    end
+
+    it "returns true for HTTP digest auth" do
+      auth.digest "username", "password"
+      auth.should be_digest
+    end
+  end
+
+  describe "#http?" do
+    it "defaults to return false" do
+      auth.should_not be_http
+    end
+
+    it "returns true for HTTP basic auth" do
+      auth.basic "username", "password"
+      auth.should be_http
+    end
+
+    it "returns true for HTTP digest auth" do
+      auth.digest "username", "password"
+      auth.should be_http
+    end
+  end
+
+  describe "#ssl" do
+    it "returns the HTTPI::Auth::SSL object" do
+      auth.ssl.should be_a(HTTPI::Auth::SSL)
+    end
+  end
+
+  describe "#ssl?" do
+    it "defaults to return false" do
+      auth.should_not be_ssl
+    end
+
+    it "returns true for SSL client auth" do
+      auth.ssl.cert_key_file = "spec/fixtures/client_key.pem"
+      auth.ssl.cert_file = "spec/fixtures/client_cert.pem"
+
+      auth.should be_ssl
+    end
+  end
+
+  describe "#type" do
+    it "returns the authentication type" do
+      auth.basic "username", "password"
+      auth.type.should == :basic
+    end
+  end
+
+  describe "#credentials" do
+    it "returns the credentials for HTTP basic auth" do
+      auth.basic "username", "basic"
+      auth.credentials.should == ["username", "basic"]
+    end
+
+    it "returns the credentials for HTTP digest auth" do
+      auth.digest "username", "digest"
+      auth.credentials.should == ["username", "digest"]
+    end
+  end
+
+end