httpotato 1.0.2

data/spec/httpotato/response_spec.rb

@@ -0,0 +1,188 @@
+require File.expand_path(File.join(File.dirname(__FILE__), '..', 'spec_helper'))
+
+describe HTTPotato::Response do
+  before do
+    @last_modified = Date.new(2010, 1, 15).to_s
+    @content_length = '1024'
+    @response_object = {'foo' => 'bar'}
+    @response_object = Net::HTTPOK.new('1.1', 200, 'OK')
+    @response_object.stub(:body => "{foo:'bar'}")
+    @response_object['last-modified'] = @last_modified
+    @response_object['content-length'] = @content_length
+    @parsed_response = {"foo" => "bar"}
+    @response = HTTPotato::Response.new(@response_object, @parsed_response)
+  end
+
+  describe ".underscore" do
+    it "works with one capitalized word" do
+      HTTPotato::Response.underscore("Accepted").should == "accepted"
+    end
+
+    it "works with titlecase" do
+      HTTPotato::Response.underscore("BadGateway").should == "bad_gateway"
+    end
+
+    it "works with all caps" do
+      HTTPotato::Response.underscore("OK").should == "ok"
+    end
+  end
+
+  describe "initialization" do
+    it "should set the Net::HTTP Response" do
+      @response.response.should == @response_object
+    end
+
+    it "should set body" do
+      @response.body.should == @response_object.body
+    end
+
+    it "should set code" do
+      @response.code.should.to_s == @response_object.code
+    end
+
+    it "should set code as a Fixnum" do
+      @response.code.should be_an_instance_of(Fixnum)
+    end
+  end
+
+  it "returns response headers" do
+    response = HTTPotato::Response.new(@response_object, @parsed_response)
+    response.headers.should == {'last-modified' => [@last_modified], 'content-length' => [@content_length]}
+  end
+
+  it "should send missing methods to delegate" do
+    response = HTTPotato::Response.new(@response_object, {'foo' => 'bar'})
+    response['foo'].should == 'bar'
+  end
+
+  it "should be able to iterate if it is array" do
+    response = HTTPotato::Response.new(@response_object, [{'foo' => 'bar'}, {'foo' => 'baz'}])
+    response.size.should == 2
+    expect {
+      response.each { |item| }
+    }.to_not raise_error
+  end
+
+  it "allows headers to be accessed by mixed-case names in hash notation" do
+    response = HTTPotato::Response.new(@response_object, @parsed_response)
+    response.headers['Content-LENGTH'].should == @content_length
+  end
+
+  it "returns a comma-delimited value when multiple values exist" do
+    @response_object.add_field 'set-cookie', 'csrf_id=12345; path=/'
+    @response_object.add_field 'set-cookie', '_github_ses=A123CdE; path=/'
+    response = HTTPotato::Response.new(@response_object, @parsed_response)
+    response.headers['set-cookie'].should == "csrf_id=12345; path=/, _github_ses=A123CdE; path=/"
+  end
+
+  # Backwards-compatibility - previously, #headers returned a Hash
+  it "responds to hash methods" do
+    response = HTTPotato::Response.new(@response_object, @parsed_response)
+    hash_methods = {}.methods - response.headers.methods
+    hash_methods.each do |method_name|
+      response.headers.respond_to?(method_name).should be_true
+    end
+  end
+
+  xit "should allow hashes to be accessed with dot notation" do
+    response = HTTPotato::Response.new({'foo' => 'bar'}, "{foo:'bar'}", 200, 'OK')
+    response.foo.should == 'bar'
+  end
+
+  xit "should allow nested hashes to be accessed with dot notation" do
+    response = HTTPotato::Response.new({'foo' => {'bar' => 'baz'}}, "{foo: {bar:'baz'}}", 200, 'OK')
+    response.foo.should == {'bar' => 'baz'}
+    response.foo.bar.should == 'baz'
+  end
+
+  describe "semantic methods for response codes" do
+    def response_mock(klass)
+      r = klass.new('', '', '')
+      r.stub(:body)
+      r
+    end
+
+    context "major codes" do
+      it "is information" do
+        net_response = response_mock(Net::HTTPInformation)
+        response = HTTPotato::Response.new(net_response, '')
+        response.information?.should be_true
+      end
+
+      it "is success" do
+        net_response = response_mock(Net::HTTPSuccess)
+        response = HTTPotato::Response.new(net_response, '')
+        response.success?.should be_true
+      end
+
+      it "is redirection" do
+        net_response = response_mock(Net::HTTPRedirection)
+        response = HTTPotato::Response.new(net_response, '')
+        response.redirection?.should be_true
+      end
+
+      it "is client error" do
+        net_response = response_mock(Net::HTTPClientError)
+        response = HTTPotato::Response.new(net_response, '')
+        response.client_error?.should be_true
+      end
+
+      it "is server error" do
+        net_response = response_mock(Net::HTTPServerError)
+        response = HTTPotato::Response.new(net_response, '')
+        response.server_error?.should be_true
+      end
+    end
+
+    context "for specific codes" do
+      SPECIFIC_CODES = {
+        :accepted?                        => Net::HTTPAccepted,
+        :bad_gateway?                     => Net::HTTPBadGateway,
+        :bad_request?                     => Net::HTTPBadRequest,
+        :conflict?                        => Net::HTTPConflict,
+        :continue?                        => Net::HTTPContinue,
+        :created?                         => Net::HTTPCreated,
+        :expectation_failed?              => Net::HTTPExpectationFailed,
+        :forbidden?                       => Net::HTTPForbidden,
+        :found?                           => Net::HTTPFound,
+        :gateway_time_out?                => Net::HTTPGatewayTimeOut,
+        :gone?                            => Net::HTTPGone,
+        :internal_server_error?           => Net::HTTPInternalServerError,
+        :length_required?                 => Net::HTTPLengthRequired,
+        :method_not_allowed?              => Net::HTTPMethodNotAllowed,
+        :moved_permanently?               => Net::HTTPMovedPermanently,
+        :multiple_choice?                 => Net::HTTPMultipleChoice,
+        :no_content?                      => Net::HTTPNoContent,
+        :non_authoritative_information?   => Net::HTTPNonAuthoritativeInformation,
+        :not_acceptable?                  => Net::HTTPNotAcceptable,
+        :not_found?                       => Net::HTTPNotFound,
+        :not_implemented?                 => Net::HTTPNotImplemented,
+        :not_modified?                    => Net::HTTPNotModified,
+        :ok?                              => Net::HTTPOK,
+        :partial_content?                 => Net::HTTPPartialContent,
+        :payment_required?                => Net::HTTPPaymentRequired,
+        :precondition_failed?             => Net::HTTPPreconditionFailed,
+        :proxy_authentication_required?   => Net::HTTPProxyAuthenticationRequired,
+        :request_entity_too_large?        => Net::HTTPRequestEntityTooLarge,
+        :request_time_out?                => Net::HTTPRequestTimeOut,
+        :request_uri_too_long?            => Net::HTTPRequestURITooLong,
+        :requested_range_not_satisfiable? => Net::HTTPRequestedRangeNotSatisfiable,
+        :reset_content?                   => Net::HTTPResetContent,
+        :see_other?                       => Net::HTTPSeeOther,
+        :service_unavailable?             => Net::HTTPServiceUnavailable,
+        :switch_protocol?                 => Net::HTTPSwitchProtocol,
+        :temporary_redirect?              => Net::HTTPTemporaryRedirect,
+        :unauthorized?                    => Net::HTTPUnauthorized,
+        :unsupported_media_type?          => Net::HTTPUnsupportedMediaType,
+        :use_proxy?                       => Net::HTTPUseProxy,
+        :version_not_supported?           => Net::HTTPVersionNotSupported
+      }.each do |method, klass|
+        it "responds to #{method}" do
+          net_response = response_mock(klass)
+          response = HTTPotato::Response.new(net_response, '')
+          response.__send__(method).should be_true
+        end
+      end
+    end
+  end
+end

data/spec/httpotato/ssl_spec.rb

@@ -0,0 +1,54 @@
+require File.expand_path(File.join(File.dirname(__FILE__), '..', 'spec_helper'))
+
+describe HTTPotato::Request do
+  context "SSL certificate verification" do
+    before do
+      FakeWeb.allow_net_connect = true    # enable network connections just for this test
+    end
+
+    after do
+      FakeWeb.allow_net_connect = false   # Restore allow_net_connect value for testing
+    end
+
+    it "should work with when no trusted CA list is specified" do
+      ssl_verify_test(nil, nil, "selfsigned.crt").should == {'success' => true}
+    end
+
+    it "should work with when no trusted CA list is specified, even with a bogus hostname" do
+      ssl_verify_test(nil, nil, "bogushost.crt").should == {'success' => true}
+    end
+
+    it "should work when using ssl_ca_file with a self-signed CA" do
+      ssl_verify_test(:ssl_ca_file, "selfsigned.crt", "selfsigned.crt").should == {'success' => true}
+    end
+
+    it "should work when using ssl_ca_file with a certificate authority" do
+      ssl_verify_test(:ssl_ca_file, "ca.crt", "server.crt").should == {'success' => true}
+    end
+    it "should work when using ssl_ca_path with a certificate authority" do
+      ssl_verify_test(:ssl_ca_path, ".", "server.crt").should == {'success' => true}
+    end
+
+    it "should fail when using ssl_ca_file and the server uses an unrecognized certificate authority" do
+      lambda do
+        ssl_verify_test(:ssl_ca_file, "ca.crt", "selfsigned.crt")
+      end.should raise_error(OpenSSL::SSL::SSLError)
+    end
+    it "should fail when using ssl_ca_path and the server uses an unrecognized certificate authority" do
+      lambda do
+        ssl_verify_test(:ssl_ca_path, ".", "selfsigned.crt")
+      end.should raise_error(OpenSSL::SSL::SSLError)
+    end
+
+    it "should fail when using ssl_ca_file and the server uses a bogus hostname" do
+      lambda do
+        ssl_verify_test(:ssl_ca_file, "ca.crt", "bogushost.crt")
+      end.should raise_error(OpenSSL::SSL::SSLError)
+    end
+    it "should fail when using ssl_ca_path and the server uses a bogus hostname" do
+      lambda do
+        ssl_verify_test(:ssl_ca_path, ".", "bogushost.crt")
+      end.should raise_error(OpenSSL::SSL::SSLError)
+    end
+  end
+end

data/spec/httpotato_spec.rb

@@ -0,0 +1,570 @@
+require File.expand_path(File.join(File.dirname(__FILE__), 'spec_helper'))
+
+describe HTTPotato do
+  before(:each) do
+    @klass = Class.new
+    @klass.instance_eval { include HTTPotato }
+  end
+
+  describe "AllowedFormats deprecated" do
+    before do
+      Kernel.stub(:warn)
+    end
+    it "warns with a deprecation message" do
+      Kernel.should_receive(:warn).with("Deprecated: Use HTTPotato::Parser::SupportedFormats")
+      HTTPotato::AllowedFormats
+    end
+
+    it "returns HTTPart::Parser::SupportedFormats" do
+      HTTPotato::AllowedFormats.should == HTTPotato::Parser::SupportedFormats
+    end
+  end
+
+  describe "base uri" do
+    before(:each) do
+      @klass.base_uri('api.foo.com/v1')
+    end
+
+    it "should have reader" do
+      @klass.base_uri.should == 'http://api.foo.com/v1'
+    end
+
+    it 'should have writer' do
+      @klass.base_uri('http://api.foobar.com')
+      @klass.base_uri.should == 'http://api.foobar.com'
+    end
+
+    it 'should not modify the parameter during assignment' do
+      uri = 'http://api.foobar.com'
+      @klass.base_uri(uri)
+      uri.should == 'http://api.foobar.com'
+    end
+  end
+
+  describe "#normalize_base_uri" do
+    it "should add http if not present for non ssl requests" do
+      uri = HTTPotato.normalize_base_uri('api.foobar.com')
+      uri.should == 'http://api.foobar.com'
+    end
+
+    it "should add https if not present for ssl requests" do
+      uri = HTTPotato.normalize_base_uri('api.foo.com/v1:443')
+      uri.should == 'https://api.foo.com/v1:443'
+    end
+
+    it "should not remove https for ssl requests" do
+      uri = HTTPotato.normalize_base_uri('https://api.foo.com/v1:443')
+      uri.should == 'https://api.foo.com/v1:443'
+    end
+
+    it 'should not modify the parameter' do
+      uri = 'http://api.foobar.com'
+      HTTPotato.normalize_base_uri(uri)
+      uri.should == 'http://api.foobar.com'
+    end
+  end
+
+  describe "headers" do
+    def expect_headers(header={})
+      HTTPotato::Request.should_receive(:new) \
+        .with(anything, anything, hash_including({ :headers => header })) \
+        .and_return(mock("mock response", :perform => nil))
+    end
+
+    it "should default to empty hash" do
+      @klass.headers.should == {}
+    end
+
+    it "should be able to be updated" do
+      init_headers = {:foo => 'bar', :baz => 'spax'}
+      @klass.headers init_headers
+      @klass.headers.should == init_headers
+    end
+
+    it "uses the class headers when sending a request" do
+      expect_headers(:foo => 'bar')
+      @klass.headers(:foo => 'bar')
+      @klass.get('')
+    end
+
+    it "overwrites class headers when passing in headers" do
+      expect_headers(:baz => 'spax')
+      @klass.headers(:foo => 'bar')
+      @klass.get('', :headers => {:baz => 'spax'})
+    end
+
+    context "with cookies" do
+      it 'utilizes the class-level cookies' do
+        expect_headers(:foo => 'bar', 'cookie' => 'type=snickerdoodle')
+        @klass.headers(:foo => 'bar')
+        @klass.cookies(:type => 'snickerdoodle')
+        @klass.get('')
+      end
+
+      it 'adds cookies to the headers' do
+        expect_headers(:foo => 'bar', 'cookie' => 'type=snickerdoodle')
+        @klass.headers(:foo => 'bar')
+        @klass.get('', :cookies => {:type => 'snickerdoodle'})
+      end
+
+      it 'adds optional cookies to the optional headers' do
+        expect_headers(:baz => 'spax', 'cookie' => 'type=snickerdoodle')
+        @klass.get('', :cookies => {:type => 'snickerdoodle'}, :headers => {:baz => 'spax'})
+      end
+    end
+  end
+
+  describe "cookies" do
+    def expect_cookie_header(s)
+      HTTPotato::Request.should_receive(:new) \
+        .with(anything, anything, hash_including({ :headers => { "cookie" => s } })) \
+        .and_return(mock("mock response", :perform => nil))
+    end
+
+    it "should not be in the headers by default" do
+      HTTPotato::Request.stub!(:new).and_return(stub(nil, :perform => nil))
+      @klass.get("")
+      @klass.headers.keys.should_not include("cookie")
+    end
+
+    it "should raise an ArgumentError if passed a non-Hash" do
+      lambda do
+        @klass.cookies("nonsense")
+      end.should raise_error(ArgumentError)
+    end
+
+    it "should allow a cookie to be specified with a one-off request" do
+      expect_cookie_header "type=snickerdoodle"
+      @klass.get("", :cookies => { :type => "snickerdoodle" })
+    end
+
+    describe "when a cookie is set at the class level" do
+      before(:each) do
+        @klass.cookies({ :type => "snickerdoodle" })
+      end
+
+      it "should include that cookie in the request" do
+        expect_cookie_header "type=snickerdoodle"
+        @klass.get("")
+      end
+
+      it "should pass the proper cookies when requested multiple times" do
+        2.times do
+          expect_cookie_header "type=snickerdoodle"
+          @klass.get("")
+        end
+      end
+
+      it "should allow the class defaults to be overridden" do
+        expect_cookie_header "type=chocolate_chip"
+
+        @klass.get("", :cookies => { :type => "chocolate_chip" })
+      end
+    end
+
+    describe "in a class with multiple methods that use different cookies" do
+      before(:each) do
+        @klass.instance_eval do
+          def first_method
+            get("first_method", :cookies => { :first_method_cookie => "foo" })
+          end
+
+          def second_method
+            get("second_method", :cookies => { :second_method_cookie => "foo" })
+          end
+        end
+      end
+
+      it "should not allow cookies used in one method to carry over into other methods" do
+        expect_cookie_header "first_method_cookie=foo"
+        @klass.first_method
+
+        expect_cookie_header "second_method_cookie=foo"
+        @klass.second_method
+      end
+    end
+  end
+
+  describe "default params" do
+    it "should default to empty hash" do
+      @klass.default_params.should == {}
+    end
+
+    it "should be able to be updated" do
+      new_defaults = {:foo => 'bar', :baz => 'spax'}
+      @klass.default_params new_defaults
+      @klass.default_params.should == new_defaults
+    end
+  end
+
+  describe "default timeout" do
+    it "should default to nil" do
+      @klass.default_options[:timeout].should == nil
+    end
+
+    it "should support updating" do
+      @klass.default_timeout 10
+      @klass.default_options[:timeout].should == 10
+    end
+  end
+
+  describe "debug_output" do
+    it "stores the given stream as a default_option" do
+      @klass.debug_output $stdout
+      @klass.default_options[:debug_output].should == $stdout
+    end
+
+    it "stores the $stderr stream by default" do
+      @klass.debug_output
+      @klass.default_options[:debug_output].should == $stderr
+    end
+  end
+
+  describe "basic http authentication" do
+    it "should work" do
+      @klass.basic_auth 'foobar', 'secret'
+      @klass.default_options[:basic_auth].should == {:username => 'foobar', :password => 'secret'}
+    end
+  end
+
+  describe "digest http authentication" do
+    it "should work" do
+      @klass.digest_auth 'foobar', 'secret'
+      @klass.default_options[:digest_auth].should == {:username => 'foobar', :password => 'secret'}
+    end
+  end
+
+  describe "parser" do
+    class CustomParser
+      def self.parse(body)
+        return {:sexy => true}
+      end
+    end
+
+    let(:parser) do
+      Proc.new{ |data, format| CustomParser.parse(data) }
+    end
+
+    it "should set parser options" do
+      @klass.parser parser
+      @klass.default_options[:parser].should == parser
+    end
+
+    it "should be able parse response with custom parser" do
+      @klass.parser parser
+      FakeWeb.register_uri(:get, 'http://twitter.com/statuses/public_timeline.xml', :body => 'tweets')
+      custom_parsed_response = @klass.get('http://twitter.com/statuses/public_timeline.xml')
+      custom_parsed_response[:sexy].should == true
+    end
+
+    it "raises UnsupportedFormat when the parser cannot handle the format" do
+      @klass.format :json
+      class MyParser < HTTPotato::Parser
+        SupportedFormats = {}
+      end unless defined?(MyParser)
+      expect do
+        @klass.parser MyParser
+      end.to raise_error(HTTPotato::UnsupportedFormat)
+    end
+
+    it 'does not validate format whe custom parser is a proc' do
+      expect do
+        @klass.format :json
+        @klass.parser lambda {|body, format|}
+      end.to_not raise_error(HTTPotato::UnsupportedFormat)
+    end
+  end
+
+  describe "format" do
+    it "should allow xml" do
+      @klass.format :xml
+      @klass.default_options[:format].should == :xml
+    end
+
+    it "should allow json" do
+      @klass.format :json
+      @klass.default_options[:format].should == :json
+    end
+
+    it "should allow yaml" do
+      @klass.format :yaml
+      @klass.default_options[:format].should == :yaml
+    end
+
+    it "should allow plain" do
+      @klass.format :plain
+      @klass.default_options[:format].should == :plain
+    end
+
+    it 'should not allow funky format' do
+      lambda do
+        @klass.format :foobar
+      end.should raise_error(HTTPotato::UnsupportedFormat)
+    end
+
+    it 'should only print each format once with an exception' do
+      lambda do
+        @klass.format :foobar
+      end.should raise_error(HTTPotato::UnsupportedFormat, "':foobar' Must be one of: html, json, plain, xml, yaml")
+    end
+
+    it 'sets the default parser' do
+      @klass.default_options[:parser].should be_nil
+      @klass.format :json
+      @klass.default_options[:parser].should == HTTPotato::Parser
+    end
+
+    it 'does not reset parser to the default parser' do
+      my_parser = lambda {}
+      @klass.parser my_parser
+      @klass.format :json
+      @klass.parser.should == my_parser
+    end
+  end
+
+  describe "#no_follow" do
+    it "sets no_follow to false by default" do
+      @klass.no_follow
+      @klass.default_options[:no_follow].should be_false
+    end
+
+    it "sets the no_follow option to true" do
+      @klass.no_follow true
+      @klass.default_options[:no_follow].should be_true
+    end
+  end
+
+  describe "#maintain_method_across_redirects" do
+    it "sets maintain_method_across_redirects to true by default" do
+      @klass.maintain_method_across_redirects
+      @klass.default_options[:maintain_method_across_redirects].should be_true
+    end
+
+    it "sets the maintain_method_across_redirects option to false" do
+      @klass.maintain_method_across_redirects false
+      @klass.default_options[:maintain_method_across_redirects].should be_false
+    end
+  end
+
+  describe "with explicit override of automatic redirect handling" do
+    before do
+      @request = HTTPotato::Request.new(Net::HTTP::Get, 'http://api.foo.com/v1', :format => :xml, :no_follow => true)
+      @redirect = stub_response 'first redirect', 302
+      @redirect['location'] = 'http://foo.com/bar'
+      HTTPotato::Request.stub(:new => @request)
+    end
+
+    it "should fail with redirected GET" do
+      lambda do
+        @error = @klass.get('/foo', :no_follow => true)
+      end.should raise_error(HTTPotato::RedirectionTooDeep) {|e| e.response.body.should == 'first redirect'}
+    end
+
+    it "should fail with redirected POST" do
+      lambda do
+        @klass.post('/foo', :no_follow => true)
+      end.should raise_error(HTTPotato::RedirectionTooDeep) {|e| e.response.body.should == 'first redirect'}
+    end
+
+    it "should fail with redirected DELETE" do
+      lambda do
+        @klass.delete('/foo', :no_follow => true)
+      end.should raise_error(HTTPotato::RedirectionTooDeep) {|e| e.response.body.should == 'first redirect'}
+    end
+
+    it "should fail with redirected PUT" do
+      lambda do
+        @klass.put('/foo', :no_follow => true)
+      end.should raise_error(HTTPotato::RedirectionTooDeep) {|e| e.response.body.should == 'first redirect'}
+    end
+
+    it "should fail with redirected HEAD" do
+      lambda do
+        @klass.head('/foo', :no_follow => true)
+      end.should raise_error(HTTPotato::RedirectionTooDeep) {|e| e.response.body.should == 'first redirect'}
+    end
+
+    it "should fail with redirected OPTIONS" do
+      lambda do
+        @klass.options('/foo', :no_follow => true)
+      end.should raise_error(HTTPotato::RedirectionTooDeep) {|e| e.response.body.should == 'first redirect'}
+    end
+  end
+
+  describe "with multiple class definitions" do
+    before(:each) do
+      @klass.instance_eval do
+        base_uri "http://first.com"
+        default_params :one => 1
+      end
+
+      @additional_klass = Class.new
+      @additional_klass.instance_eval do
+        include HTTPotato
+        base_uri "http://second.com"
+        default_params :two => 2
+      end
+    end
+
+    it "should not run over each others options" do
+      @klass.default_options.should == { :base_uri => 'http://first.com', :default_params => { :one => 1 } }
+      @additional_klass.default_options.should == { :base_uri => 'http://second.com', :default_params => { :two => 2 } }
+    end
+  end
+
+  describe "two child classes inheriting from one parent" do
+    before(:each) do
+      @parent = Class.new do
+        include HTTPotato
+        def self.name
+          "Parent"
+        end
+      end
+
+      @child1 = Class.new(@parent)
+      @child2 = Class.new(@parent)
+    end
+
+    it "does not modify each others inherited attributes" do
+      @child1.default_params :joe => "alive"
+      @child2.default_params :joe => "dead"
+
+      @child1.default_options.should == { :default_params => {:joe => "alive"} }
+      @child2.default_options.should == { :default_params => {:joe => "dead"} }
+
+      @parent.default_options.should == { }
+    end
+
+    it "inherits default_options from the superclass" do
+      @parent.basic_auth 'user', 'password'
+      @child1.default_options.should == {:basic_auth => {:username => 'user', :password => 'password'}}
+      @child1.basic_auth 'u', 'p' # modifying child1 has no effect on child2
+      @child2.default_options.should == {:basic_auth => {:username => 'user', :password => 'password'}}
+    end
+
+    it "doesn't modify the parent's default options" do
+      @parent.basic_auth 'user', 'password'
+
+      @child1.basic_auth 'u', 'p'
+      @child1.default_options.should == {:basic_auth => {:username => 'u', :password => 'p'}}
+
+      @child1.basic_auth 'email', 'token'
+      @child1.default_options.should == {:basic_auth => {:username => 'email', :password => 'token'}}
+
+      @parent.default_options.should == {:basic_auth => {:username => 'user', :password => 'password'}}
+    end
+
+    it "inherits default_cookies from the parent class" do
+      @parent.cookies 'type' => 'chocolate_chip'
+      @child1.default_cookies.should == {"type" => "chocolate_chip"}
+      @child1.cookies 'type' => 'snickerdoodle'
+      @child1.default_cookies.should == {"type" => "snickerdoodle"}
+      @child2.default_cookies.should == {"type" => "chocolate_chip"}
+    end
+
+    it "doesn't modify the parent's default cookies" do
+      @parent.cookies 'type' => 'chocolate_chip'
+
+      @child1.cookies 'type' => 'snickerdoodle'
+      @child1.default_cookies.should == {"type" => "snickerdoodle"}
+
+      @parent.default_cookies.should == {"type" => "chocolate_chip"}
+    end
+  end
+
+  describe "grand parent with inherited callback" do
+    before do
+      @grand_parent = Class.new do
+        def self.inherited(subclass)
+          subclass.instance_variable_set(:@grand_parent, true)
+        end
+      end
+      @parent = Class.new(@grand_parent) do
+        include HTTPotato
+      end
+    end
+    it "continues running the #inherited on the parent" do
+      child = Class.new(@parent)
+      child.instance_variable_get(:@grand_parent).should be_true
+    end
+  end
+
+  describe "#get" do
+    it "should be able to get html" do
+      stub_http_response_with('google.html')
+      HTTPotato.get('http://www.google.com').should == file_fixture('google.html')
+    end
+
+    it "should be able parse response type json automatically" do
+      stub_http_response_with('twitter.json')
+      tweets = HTTPotato.get('http://twitter.com/statuses/public_timeline.json')
+      tweets.size.should == 20
+      tweets.first['user'].should == {
+        "name"              => "Pyk",
+        "url"               => nil,
+        "id"                => "7694602",
+        "description"       => nil,
+        "protected"         => false,
+        "screen_name"       => "Pyk",
+        "followers_count"   => 1,
+        "location"          => "Opera Plaza, California",
+        "profile_image_url" => "http://static.twitter.com/images/default_profile_normal.png"
+      }
+    end
+
+    it "should be able parse response type xml automatically" do
+      stub_http_response_with('twitter.xml')
+      tweets = HTTPotato.get('http://twitter.com/statuses/public_timeline.xml')
+      tweets['statuses'].size.should == 20
+      tweets['statuses'].first['user'].should == {
+        "name"              => "Magic 8 Bot",
+        "url"               => nil,
+        "id"                => "17656026",
+        "description"       => "ask me a question",
+        "protected"         => "false",
+        "screen_name"       => "magic8bot",
+        "followers_count"   => "90",
+        "profile_image_url" => "http://s3.amazonaws.com/twitter_production/profile_images/65565851/8ball_large_normal.jpg",
+        "location"          => nil
+      }
+    end
+
+    it "should not get undefined method add_node for nil class for the following xml" do
+      stub_http_response_with('undefined_method_add_node_for_nil.xml')
+      result = HTTPotato.get('http://foobar.com')
+      result.should == {"Entities"=>{"href"=>"https://s3-sandbox.parature.com/api/v1/5578/5633/Account", "results"=>"0", "total"=>"0", "page_size"=>"25", "page"=>"1"}}
+    end
+
+    it "should parse empty response fine" do
+      stub_http_response_with('empty.xml')
+      result = HTTPotato.get('http://foobar.com')
+      result.should be_nil
+    end
+
+    it "should accept http URIs" do
+      stub_http_response_with('google.html')
+      lambda do
+        HTTPotato.get('http://google.com')
+      end.should_not raise_error(HTTPotato::UnsupportedURIScheme)
+    end
+
+    it "should accept https URIs" do
+      stub_http_response_with('google.html')
+      lambda do
+        HTTPotato.get('https://google.com')
+      end.should_not raise_error(HTTPotato::UnsupportedURIScheme)
+    end
+
+    it "should raise an ArgumentError on URIs that are not http or https" do
+      lambda do
+        HTTPotato.get("file:///there_is_no_party_on/my/filesystem")
+      end.should raise_error(HTTPotato::UnsupportedURIScheme)
+    end
+
+    it "should raise an InvalidURIError on URIs that can't be parsed at all" do
+      lambda do
+        HTTPotato.get("It's the one that says 'Bad URI'")
+      end.should raise_error(URI::InvalidURIError)
+    end
+  end
+end