jnunemaker-httparty 0.1.8 → 0.2.0

data/spec/httparty/request_spec.rb

@@ -5,27 +5,9 @@ describe HTTParty::Request do
     @request = HTTParty::Request.new(Net::HTTP::Get, 'http://api.foo.com/v1', :format => :xml)
   end
   
-  describe "#normalize_base_uri" do
-    it "should add http if not present for non ssl requests" do
-      uri = HTTParty::Request.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 = HTTParty::Request.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 = HTTParty::Request.normalize_base_uri('https://api.foo.com/v1:443')
-      uri.should == 'https://api.foo.com/v1:443'
-    end
-  end
-  
-  describe "uri" do
-    it "should be normalized" do
-      request = HTTParty::Request.new(Net::HTTP::Get, '', :base_uri => 'api.foo.com')
-      request.uri.to_s.should == 'http://api.foo.com'
+  describe "#format" do
+    it "should return the correct parsing format" do
+      @request.format.should == :xml
     end
   end
 
@@ -78,7 +60,7 @@ describe HTTParty::Request do
 
     def setup_ok_response
       @ok = Net::HTTPOK.new("1.1", 200, "Content for you")
-      @ok.stub!(:body).and_return({"foo" => "bar"}.to_xml)
+      @ok.stub!(:body).and_return('<hash><foo>bar</foo></hash>')
       @http.should_receive(:request).and_return(@redirect, @ok)
       @request.options[:format] = :xml
     end

data/spec/httparty_spec.rb

@@ -25,7 +25,7 @@ describe HTTParty do
     end
 
     it "should have reader" do
-      Foo.base_uri.should == 'api.foo.com/v1'
+      Foo.base_uri.should == 'http://api.foo.com/v1'
     end
     
     it 'should have writer' do
@@ -34,6 +34,23 @@ describe HTTParty do
     end
   end
   
+  describe "#normalize_base_uri" do
+    it "should add http if not present for non ssl requests" do
+      uri = HTTParty.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 = HTTParty.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 = HTTParty.normalize_base_uri('https://api.foo.com/v1:443')
+      uri.should == 'https://api.foo.com/v1:443'
+    end
+  end
+  
   describe "headers" do
     it "should default to empty hash" do
       Foo.headers.should == {}
@@ -112,8 +129,49 @@ describe HTTParty do
   
   describe "with multiple class definitions" do
     it "should not run over each others options" do
-      HRest.default_options.should == {:base_uri => 'hrest.com', :default_params => {:two => 'three'}}
-      GRest.default_options.should == {:base_uri => 'grest.com', :default_params => {:one => 'two'}}
+      HRest.default_options.should == {:base_uri => 'http://hrest.com', :default_params => {:two => 'three'}}
+      GRest.default_options.should == {:base_uri => 'http://grest.com', :default_params => {:one => 'two'}}
+    end
+  end
+  
+  describe "#get" do
+    it "should be able to get html" do
+      stub_http_response_with('google.html')
+      HTTParty.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 = HTTParty.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 = HTTParty.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
   end
 end

data/spec/spec_helper.rb

@@ -1,9 +1,24 @@
-begin
-  require 'spec'
-rescue LoadError
-  require 'rubygems'
-  gem 'rspec'
-  require 'spec'
+require 'rubygems'
+gem 'rspec'
+require 'spec'
+require File.join(File.dirname(__FILE__), '..', 'lib', 'httparty')
+
+def file_fixture(filename)
+  open(File.join(File.dirname(__FILE__), 'fixtures', "#{filename.to_s}")).read
 end
 
-require File.join(File.dirname(__FILE__), '..', 'lib', 'httparty')
+def stub_http_response_with(filename)
+  format = filename.split('.').last.intern
+  data = file_fixture(filename)
+  http = Net::HTTP.new('localhost', 80)
+  
+  response = Net::HTTPOK.new("1.1", 200, "Content for you")
+  response.stub!(:body).and_return(data)
+  http.stub!(:request).and_return(response)
+  
+  http_request = HTTParty::Request.new(Net::HTTP::Get, '')
+  http_request.stub!(:get_response).and_return(response)
+  http_request.stub!(:format).and_return(format)
+  
+  HTTParty::Request.should_receive(:new).and_return(http_request)
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: jnunemaker-httparty
 version: !ruby/object:Gem::Version 
-  version: 0.1.8
+  version: 0.2.0
 platform: ruby
 authors: 
 - John Nunemaker
@@ -9,17 +9,17 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2008-12-05 00:00:00 -08:00
+date: 2008-12-07 00:00:00 -08:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
-  name: activesupport
+  name: json
   version_requirement: 
   version_requirements: !ruby/object:Gem::Requirement 
     requirements: 
-    - - ">="
+    - - ~>
       - !ruby/object:Gem::Version 
-        version: "2.1"
+        version: "1.1"
     version: 
 - !ruby/object:Gem::Dependency 
   name: echoe
@@ -37,9 +37,12 @@ executables: []
 extensions: []
 
 extra_rdoc_files: 
+- lib/core_extensions.rb
+- lib/httparty/exceptions.rb
 - lib/httparty/request.rb
 - lib/httparty/version.rb
 - lib/httparty.rb
+- lib/module_level_inheritable_attributes.rb
 - README
 files: 
 - examples/aaws.rb
@@ -51,14 +54,21 @@ files:
 - examples/whoismyrep.rb
 - History
 - httparty.gemspec
+- lib/core_extensions.rb
+- lib/httparty/exceptions.rb
 - lib/httparty/request.rb
 - lib/httparty/version.rb
 - lib/httparty.rb
+- lib/module_level_inheritable_attributes.rb
 - Manifest
 - MIT-LICENSE
 - Rakefile
 - README
 - setup.rb
+- spec/fixtures/delicious.xml
+- spec/fixtures/google.html
+- spec/fixtures/twitter.json
+- spec/fixtures/twitter.xml
 - spec/httparty/request_spec.rb
 - spec/httparty_spec.rb
 - spec/spec.opts