transport 1.0.0

data/spec/lib/transport/http/formatter_spec.rb

@@ -0,0 +1,28 @@
+require File.expand_path(File.join(File.dirname(__FILE__), "..", "..", "..", "spec_helper"))
+require File.expand_path(File.join(File.dirname(__FILE__), "..", "..", "..", "..", "lib", "transport", "http", "formatter"))
+require 'logger'
+
+describe Transport::HTTP::Formatter do
+
+  before :each do
+    @logger = mock Logger, :info => nil
+
+    @formatter = described_class.new @logger
+  end
+
+  describe "log_transport" do
+
+    before :each do
+      @uri = mock URI, :host => "host", :port => 1234
+      @request = mock Net::HTTP::Get, :class => Net::HTTP::Get, :path => "/path", :each_capitalized_name => nil, :body => "line one\nline two"
+      @response = mock Net::HTTPResponse, :code => "200", :body => "body"
+    end
+
+    it "should write the formatted request to the logger" do
+      @logger.should_receive(:info).with("transport to host 1234\nrequest: Net::HTTP::Get /path\n  body:\n    line one\n    line two\nresponse: 200\n  body")
+      @formatter.log_transport @uri, @request, @response
+    end
+
+  end
+
+end

data/spec/lib/transport/http/request_builder/parameter_serializer_spec.rb

@@ -0,0 +1,18 @@
+require File.expand_path(File.join(File.dirname(__FILE__), "..", "..", "..", "..", "spec_helper"))
+require File.expand_path(File.join(File.dirname(__FILE__), "..", "..", "..", "..", "..", "lib", "transport", "http", "request_builder", "parameter_serializer"))
+
+describe Transport::HTTP::RequestBuilder::ParameterSerializer do
+
+  it "should return nil on an empty parameter hash" do
+    serializer = described_class.new
+    serializer.perform
+    serializer.result.should be_nil
+  end
+
+  it "should return a correctly encoded query string" do
+    serializer = described_class.new :foo => "bar", :test => [ "value1", "value2" ]
+    serializer.perform
+    serializer.result.should == "foo=bar&test=value1&test=value2"
+  end
+
+end

data/spec/lib/transport/http/request_builder_spec.rb

@@ -0,0 +1,134 @@
+require File.expand_path(File.join(File.dirname(__FILE__), "..", "..", "..", "spec_helper"))
+require File.expand_path(File.join(File.dirname(__FILE__), "..", "..", "..", "..", "lib", "transport", "http", "request_builder"))
+
+describe Transport::HTTP::RequestBuilder do
+
+  before :each do
+    @url = "http://host:1234/test"
+    @options = {
+      :headers => { "Test-Header" => "test" },
+      :parameters => { "test_parameter" => "test" }
+    }
+    @parameter_serializer = mock described_class::ParameterSerializer, :perform => nil, :result => "test_parameter=test"
+    described_class::ParameterSerializer.stub(:new).and_return(@parameter_serializer)
+  end
+
+  def mock_request(http_method)
+    @request_class = Net::HTTP.const_get(:"#{http_method.to_s.capitalize}")
+    @request = mock @request_class, :path => "/test", :basic_auth => nil, :body= => nil
+    @request_class.stub(:new).and_return(@request)
+  end
+
+  shared_examples_for "any http request builder perform" do
+
+    it "should initialize the parameter serializer correctly" do
+      described_class::ParameterSerializer.should_receive(:new).with(@options[:parameters]).and_return(@parameter_serializer)
+      @request_builder.perform
+    end
+
+    it "should set basic authentication for the request" do
+      @request.should_receive(:basic_auth).with("test_username", "test_password")
+
+      @request_builder.options.merge! :auth_type => :basic, :username => "test_username", :password => "test_password"
+      @request_builder.perform
+    end
+
+    it "should raise a #{NotImplementedError} if auth_type is not supported" do
+      @request_builder.options.merge! :auth_type => :invalid
+      lambda do
+        @request_builder.perform
+      end.should raise_error(NotImplementedError)
+    end
+
+  end
+
+  shared_examples_for "any http request builder request" do
+
+    before :each do
+      @request_builder.perform
+    end
+
+    it "should return the request" do
+      @request_builder.request.should == @request
+    end
+
+  end
+
+  described_class::HTTP_METHODS_WITH_PARAMETERS.each do |http_method|
+
+    context "#{http_method} request" do
+
+      before :each do
+        mock_request http_method
+        @request_builder = described_class.new http_method, @url, @options
+      end
+
+      describe "perform" do
+
+        it_should_behave_like "any http request builder perform"
+
+        it "should initialize the request correctly" do
+          @request_class.should_receive(:new).with("/test?test_parameter=test", @options[:headers]).and_return(@request)
+          @request_builder.perform
+        end
+
+        it "should add no query string if parameter serializer returns nil" do
+          @request_class.should_receive(:new).with("/test", @options[:headers]).and_return(@request)
+
+          @parameter_serializer.stub(:result).and_return(nil)
+          @request_builder.perform
+        end
+
+        it "should not set the request body" do
+          @request.should_not_receive(:body=)
+          @request_builder.perform
+        end
+
+      end
+
+      describe "request" do
+
+        it_should_behave_like "any http request builder request"
+
+      end
+
+    end
+
+  end
+
+  described_class::HTTP_METHODS_WITH_BODY.each do |http_method|
+
+    context "#{http_method} request" do
+
+      before :each do
+        mock_request http_method
+        @request_builder = described_class.new http_method, @url, @options
+      end
+
+      describe "perform" do
+
+        it_should_behave_like "any http request builder perform"
+
+        it "should initialize the request correctly" do
+          @request_class.should_receive(:new).with("/test", @options[:headers]).and_return(@request)
+          @request_builder.perform
+        end
+
+        it "should set the request body" do
+          @request.should_receive(:body=).with("test_parameter=test")
+          @request_builder.perform
+        end
+
+      end
+
+      describe "request" do
+
+        it_should_behave_like "any http request builder request"
+
+      end
+
+    end
+
+  end
+
+end

data/spec/lib/transport/http_spec.rb

@@ -0,0 +1,61 @@
+require File.expand_path(File.join(File.dirname(__FILE__), "..", "..", "spec_helper"))
+require File.expand_path(File.join(File.dirname(__FILE__), "..", "..", "..", "lib", "transport", "http"))
+require 'logger'
+
+describe Transport::HTTP do
+
+  before :each do
+    @uri = mock URI, :host => "host", :port => 1234
+    @request = mock Net::HTTPRequest, :path => "/path"
+    @logger = mock ::Logger, :info => nil
+    @options = { :expected_status_code => 200, :logger => @logger }
+
+    @response = mock Net::HTTPResponse, :code => "200", :body => "test\ntest"
+    Net::HTTP.stub(:start).and_return(@response)
+
+    @formatter = mock described_class::Formatter, :log_transport => nil
+    described_class::Formatter.stub(:new).and_return(@formatter)
+
+    @transport = described_class.new @uri, @request, @options
+  end
+
+  describe "perform" do
+
+    it "should perform the request" do
+      Net::HTTP.should_receive(:start).with("host", 1234).and_return(@response)
+      @transport.perform
+    end
+
+    it "should initialize the formatter" do
+      described_class::Formatter.should_receive(:new).and_return(@formatter)
+      @transport.perform
+    end
+
+    it "should log the transport" do
+      @formatter.should_receive(:log_transport).with(@uri, @request, @response)
+      @transport.perform
+    end
+
+    it "should raise UnexpectedStatusCodeError if responded status code is wrong" do
+      @transport.options.merge! :expected_status_code => 201
+      lambda do
+        @transport.perform
+      end.should raise_error(Transport::UnexpectedStatusCodeError)
+    end
+
+  end
+
+  describe "response" do
+
+    before :each do
+      @transport.perform
+    end
+
+    it "should return the response body" do
+      @transport.perform
+      @transport.response.should == "test\ntest"
+    end
+
+  end
+
+end

data/spec/lib/transport/json/request_builder_spec.rb

@@ -0,0 +1,78 @@
+require File.expand_path(File.join(File.dirname(__FILE__), "..", "..", "..", "spec_helper"))
+require File.expand_path(File.join(File.dirname(__FILE__), "..", "..", "..", "..", "lib", "transport", "json", "request_builder"))
+
+describe Transport::JSON::RequestBuilder do
+
+  before :each do
+    @http_method = :get
+    @url = "test_url"
+    @options = { :encode_parameters => true }
+
+    @request = mock Net::HTTP::Get
+
+    @http_request_builder = mock Transport::HTTP::RequestBuilder, :perform => nil, :request => @request
+    Transport::HTTP::RequestBuilder.stub(:new).and_return(@http_request_builder)
+
+    @request_builder = described_class.new @http_method, @url, @options
+  end
+
+  describe "perform" do
+
+    it "should initialize the http request builder correctly" do
+      Transport::HTTP::RequestBuilder.should_receive(:new).with(
+        @http_method,
+        @url,
+        @options.merge(:headers => { "Accept" => "application/json" })
+      ).and_return(@http_request_builder)
+
+      @request_builder.perform
+    end
+
+    it "should set the content type to 'application/json' and convert body to json if a body is given" do
+      Transport::HTTP::RequestBuilder.should_receive(:new).with(
+        @http_method,
+        @url,
+        @options.merge(
+          :headers => { "Accept" => "application/json", "Content-Type" => "application/json" },
+          :body => "{\"test\":\"body\"}"
+        )
+      ).and_return(@http_request_builder)
+
+      @request_builder.options.merge! :body => { "test" => "body" }
+      @request_builder.perform
+    end
+
+    it "should convert the parameters to json if requested" do
+      Transport::HTTP::RequestBuilder.should_receive(:new).with(
+        @http_method,
+        @url,
+        @options.merge(
+          :headers => { "Accept" => "application/json" },
+          :parameters => { "test_parameter" => "\"test\"" }
+        )
+      ).and_return(@http_request_builder)
+
+      @request_builder.options.merge! :parameters => { "test_parameter" => "test" }
+      @request_builder.perform
+    end
+
+    it "should perform a http request build" do
+      @http_request_builder.should_receive(:perform)
+      @request_builder.perform
+    end
+
+  end
+
+  describe "request" do
+
+    before :each do
+      @request_builder.perform
+    end
+
+    it "should return the request" do
+      @request_builder.request.should == @request
+    end
+
+  end
+
+end

data/spec/lib/transport/json/response_parser_spec.rb

@@ -0,0 +1,52 @@
+require File.expand_path(File.join(File.dirname(__FILE__), "..", "..", "..", "spec_helper"))
+require File.expand_path(File.join(File.dirname(__FILE__), "..", "..", "..", "..", "lib", "transport", "json", "response_parser"))
+
+describe Transport::JSON::ResponseParser do
+
+  before :each do
+    @http_response = "{\"test\":\"test value\"}"
+
+    @response_parser = described_class.new @http_response
+  end
+
+  describe "perform" do
+
+    it "should parse the response body" do
+      JSON.should_receive(:parse).with(@http_response).and_return("test" => "test value")
+      @response_parser.perform
+    end
+
+    it "should set result to nil if http response is nil" do
+      response_parser = described_class.new nil
+      response_parser.perform
+      response_parser.result.should be_nil
+    end
+
+    it "should set result to nil if http response is empty" do
+      response_parser = described_class.new " "
+      response_parser.perform
+      response_parser.result.should be_nil
+    end
+
+    it "should raise a #{Transport::JSON::ParserError} if a #{JSON::ParserError} is raised" do
+      JSON.stub(:parse).and_raise(JSON::ParserError)
+      lambda do
+        @response_parser.perform
+      end.should raise_error(Transport::JSON::ParserError)
+    end
+
+  end
+
+  describe "result" do
+
+    before :each do
+      @response_parser.perform
+    end
+
+    it "should return the result" do
+      @response_parser.result.should == { "test" => "test value" }
+    end
+
+  end
+
+end

data/spec/lib/transport/json_spec.rb

@@ -0,0 +1,58 @@
+require File.expand_path(File.join(File.dirname(__FILE__), "..", "..", "spec_helper"))
+require File.expand_path(File.join(File.dirname(__FILE__), "..", "..", "..", "lib", "transport", "json"))
+
+describe Transport::JSON do
+
+  before :each do
+    @uri = mock URI
+    @request = mock Net::HTTPRequest
+    @options = { }
+
+    @http_response = "test_http_response"
+
+    @http_transport = mock Transport::HTTP, :perform => nil, :response => @http_response
+    Transport::HTTP.stub(:new).and_return(@http_transport)
+
+    @response_parser = mock described_class::ResponseParser, :perform => nil, :result => :test_response
+    described_class::ResponseParser.stub(:new).and_return(@response_parser)
+
+    @transport = described_class.new @uri, @request, @options
+  end
+
+  describe "perform" do
+
+    it "should initialize the http transport" do
+      Transport::HTTP.should_receive(:new).with(@uri, @request, @options).and_return(@http_transport)
+      @transport.perform
+    end
+
+    it "should perform the http transport" do
+      @http_transport.should_receive(:perform)
+      @transport.perform
+    end
+
+    it "should initialize the response parser" do
+      described_class::ResponseParser.should_receive(:new).with(@http_response).and_return(@response_parser)
+      @transport.perform
+    end
+
+    it "should perform the response parse" do
+      @response_parser.should_receive(:perform)
+      @transport.perform
+    end
+
+  end
+
+  describe "response" do
+
+    before :each do
+      @transport.perform
+    end
+
+    it "should return the response parser result" do
+      @transport.response.should == :test_response
+    end
+
+  end
+
+end

data/spec/lib/transport/spec/faker_spec.rb

@@ -0,0 +1,87 @@
+require File.expand_path(File.join(File.dirname(__FILE__), "..", "..", "..", "spec_helper"))
+require File.expand_path(File.join(File.dirname(__FILE__), "..", "..", "..", "..", "lib", "transport", "spec", "faker"))
+
+describe Transport::Spec::Faker do
+
+  before :each do
+    @fake = [
+      {
+        :http_method => "get",
+        :url => "http://host:1234/test.html",
+        :response => {
+          :code => 200,
+          :body => "<html></html>"
+        }
+      }, {
+        :http_method => "get",
+        :url => "http://host:1234/test.json",
+        :response => {
+          :code => 200,
+          :body => { "test" => "body" }
+        }
+      }
+    ]
+    YAML.stub(:load_file).and_return(@fake)
+    @filename = "test_filename"
+  end
+
+  def do_fake
+    described_class.fake! @filename
+  end
+
+  shared_examples_for "any fake" do
+
+    it "should load the yaml file" do
+      YAML.should_receive(:load_file).with(@filename).and_return(@fake)
+      do_fake
+    end
+
+    it "should raise a #{described_class::NoFakeRequestError} if fake request is matching" do
+      do_fake
+      lambda do
+        do_request :get, "invalid"
+      end.should raise_error(described_class::NoFakeRequestError)
+    end
+
+    it "should raise an #{Transport::UnexpectedStatusCodeError} if the wrong status code is returned" do
+      do_fake
+      lambda do
+        do_request :get, "http://host:1234/test.html", :expected_status_code => 201
+      end.should raise_error(Transport::UnexpectedStatusCodeError)
+    end
+
+  end
+
+  describe "fake an http transport" do
+
+    def do_request(*arguments)
+      Transport::HTTP.request *arguments
+    end
+
+    it_should_behave_like "any fake"
+
+    it "should fake a http transport" do
+      do_fake
+      response = do_request :get, "http://host:1234/test.html"
+      response.should == "<html></html>"
+    end
+
+  end
+
+  describe "fake an json transport" do
+
+    def do_request(*arguments)
+      Transport::JSON.request *arguments
+    end
+
+    it_should_behave_like "any fake"
+
+    it "should fake a json transport" do
+      do_fake
+      response = do_request :get, "http://host:1234/test.json"
+      response.should == { "test" => "body" }
+    end
+
+  end
+
+end