mbleigh-twitter-auth 0.1.3 → 0.1.5

data/README.markdown

@@ -6,11 +6,9 @@ TwitterAuth aims to provide a complete authentication and API access solution fo
 Installation
 ============
 
-**NOTE:** The GemPlugin version of TwitterAuth is currently broken...Rails isn't picking up namespaced models in the Engines. I'm working on a fix, but please use the plugin version until then.
-
 You can include TwitterAuth as a gem in your project like so:
 
-    config.gem 'mbleigh-twitter-auth', :source => 'http://gems.github.com'
+    config.gem 'twitter-auth', :lib => 'twitter_auth'
 
 Or you can install it as a traditional Rails plugin:
 

data/Rakefile

@@ -24,6 +24,7 @@ s.files =  FileList["[A-Z]*", "{bin,generators,lib,spec,config,app,rails}/**/*"]
     s.authors = ["Michael Bleigh"]
     s.add_dependency('oauth', '>= 0.3.1')
     s.add_dependency('ezcrypto', '>= 0.7.2')
+    s.rubyforge_project = 'twitter-auth'
   end
 rescue LoadError
   puts "Jeweler not available. Install it with: sudo gem install technicalpickles-jeweler -s http://gems.github.com"

data/VERSION.yml

@@ -1,4 +1,4 @@
 --- 
 :minor: 1
-:patch: 3
+:patch: 5
 :major: 0

data/lib/twitter_auth/dispatcher/basic.rb

@@ -3,6 +3,8 @@ require 'net/http'
 module TwitterAuth
   module Dispatcher
     class Basic
+      include TwitterAuth::Dispatcher::Shared
+
       attr_accessor :user
 
       def initialize(user)
@@ -20,9 +22,7 @@ module TwitterAuth
           http.request(req)
         }
         
-        JSON.parse(response.body)
-      rescue JSON::ParserError
-        response.body
+        handle_response(response)      
       end
 
       def get(path, *arguments)

data/lib/twitter_auth/dispatcher/oauth.rb

@@ -3,6 +3,8 @@ require 'oauth'
 module TwitterAuth
   module Dispatcher
     class Oauth < OAuth::AccessToken
+      include TwitterAuth::Dispatcher::Shared
+
       attr_accessor :user
 
       def initialize(user)
@@ -14,9 +16,8 @@ module TwitterAuth
       def request(http_method, path, *arguments)
         path << '.json' unless path.match(/\.(:?xml|json)\z/i)
         response = super
-        JSON.parse(response.body)
-      rescue JSON::ParserError
-        response.body
+
+        handle_response(response)
       end
     end
   end

data/lib/twitter_auth/dispatcher/shared.rb

@@ -0,0 +1,28 @@
+module TwitterAuth
+  module Dispatcher
+    module Shared
+      def handle_response(response)
+        case response
+        when Net::HTTPOK 
+          begin
+            JSON.parse(response.body)
+          rescue JSON::ParserError
+            response.body
+          end
+        else
+          message = begin
+            JSON.parse(response.body)['error']
+          rescue JSON::ParserError
+            if match = response.body.match(/<error>(.*)<\/error>/)
+              match[1]
+            else
+              'An error occurred processing your Twitter request.'
+            end
+          end
+
+          raise TwitterAuth::Dispatcher::Error, message
+        end
+      end
+    end
+  end
+end

data/lib/twitter_auth.rb

@@ -67,3 +67,7 @@ end
 require 'twitter_auth/controller_extensions'
 require 'twitter_auth/cryptify'
 require 'twitter_auth/dispatcher/oauth'
+require 'twitter_auth/dispatcher/basic'
+require 'twitter_auth/dispatcher/shared'
+
+class TwitterAuth::Dispatcher::Error < StandardError; end

data/spec/twitter_auth/dispatcher/basic_spec.rb

@@ -47,6 +47,21 @@ describe TwitterAuth::Dispatcher::Basic do
       @net.should_receive(:start)
       lambda{@dispatcher.request(:get, '/fake')}.should raise_error(NoMethodError)
     end
+
+    it "should raise a TwitterAuth::Dispatcher::Error if response code isn't 200" do
+      FakeWeb.register_uri('https://twitter.com:443/bad_response.json', :string => {'error' => 'bad response'}.to_json, :status => ['401', 'Unauthorized'])
+      lambda{@dispatcher.request(:get, '/bad_response')}.should raise_error(TwitterAuth::Dispatcher::Error)
+    end
+
+    it 'should set the error message to the JSON message' do
+      FakeWeb.register_uri('https://twitter.com:443/bad_response.json', :string => {'error' => 'bad response'}.to_json, :status => ['401', 'Unauthorized'])
+      lambda{@dispatcher.request(:get, '/bad_response')}.should raise_error(TwitterAuth::Dispatcher::Error, 'bad response')
+    end
+
+    it 'should set the error message to the XML message' do
+      FakeWeb.register_uri('https://twitter.com:443/bad_response.xml', :string => "<hash>\n<request>/bad_response.xml</request>\n<error>bad response</error>\n</hash>", :status => ['401', 'Unauthorized'])
+      lambda{@dispatcher.request(:get, '/bad_response')}.should raise_error(TwitterAuth::Dispatcher::Error, 'bad response')
+    end
   end
 
   %w(get post delete put).each do |method|

data/spec/twitter_auth/dispatcher/oauth_spec.rb

@@ -45,6 +45,21 @@ describe TwitterAuth::Dispatcher::Oauth do
       @dispatcher.request(:get, '/fake').should == @dispatcher.request(:get, '/fake.json')
     end
 
+    it "should raise a TwitterAuth::Dispatcher::Error if response code isn't 200" do
+      FakeWeb.register_uri('https://twitter.com:443/bad_response.json', :string => {'error' => 'bad response'}.to_json, :status => ['401', 'Unauthorized'])
+      lambda{@dispatcher.request(:get, '/bad_response')}.should raise_error(TwitterAuth::Dispatcher::Error)
+    end
+
+    it 'should set the error message to the JSON message' do
+      FakeWeb.register_uri('https://twitter.com:443/bad_response.json', :string => {'error' => 'bad response'}.to_json, :status => ['401', 'Unauthorized'])
+      lambda{@dispatcher.request(:get, '/bad_response')}.should raise_error(TwitterAuth::Dispatcher::Error, 'bad response')
+    end
+
+    it 'should set the error message to the XML message' do
+      FakeWeb.register_uri('https://twitter.com:443/bad_response.xml', :string => "<hash>\n<request>/bad_response.xml</request>\n<error>bad response</error>\n</hash>", :status => ['401', 'Unauthorized'])
+      lambda{@dispatcher.request(:get, '/bad_response')}.should raise_error(TwitterAuth::Dispatcher::Error, 'bad response')
+    end
+
     it 'should work with verb methods' do
       @dispatcher.get('/fake').should == @dispatcher.request(:get, '/fake')
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: mbleigh-twitter-auth
 version: !ruby/object:Gem::Version 
-  version: 0.1.3
+  version: 0.1.5
 platform: ruby
 authors: 
 - Michael Bleigh
@@ -67,6 +67,7 @@ files:
 - lib/twitter_auth/dispatcher
 - lib/twitter_auth/dispatcher/basic.rb
 - lib/twitter_auth/dispatcher/oauth.rb
+- lib/twitter_auth/dispatcher/shared.rb
 - lib/twitter_auth.rb
 - lib/twitteresque
 - spec/controllers
@@ -130,7 +131,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
   version: 
 requirements: []
 
-rubyforge_project: 
+rubyforge_project: twitter-auth
 rubygems_version: 1.2.0
 signing_key: 
 specification_version: 2