octokit 0.6.0 → 0.6.1

data/.travis.yml

@@ -0,0 +1,6 @@
+rvm:
+  - 1.8.7
+  - 1.9.2
+  - ree
+  - rbx
+  - jruby

data/lib/faraday/raise_error.rb

@@ -3,38 +3,30 @@ require 'faraday'
 # @api private
 module Faraday
   class Response::RaiseError < Response::Middleware
-    def self.register_on_complete(env)
-      env[:response].on_complete do |response|
-        case response[:status].to_i
-        when 400
-          raise Octokit::BadRequest, error_message(response)
-        when 401
-          raise Octokit::Unauthorized, error_message(response)
-        when 403
-          raise Octokit::Forbidden, error_message(response)
-        when 404
-          raise Octokit::NotFound, error_message(response)
-        when 406
-          raise Octokit::NotAcceptable, error_message(response)
-        when 500
-          raise Octokit::InternalServerError, error_message(response)
-        when 501
-          raise Octokit::NotImplemented, error_message(response)
-        when 502
-          raise Octokit::BadGateway, error_message(response)
-        when 503
-          raise Octokit::ServiceUnavailable, error_message(response)
-        end
+    def on_complete(response)
+      case response[:status].to_i
+      when 400
+        raise Octokit::BadRequest, error_message(response)
+      when 401
+        raise Octokit::Unauthorized, error_message(response)
+      when 403
+        raise Octokit::Forbidden, error_message(response)
+      when 404
+        raise Octokit::NotFound, error_message(response)
+      when 406
+        raise Octokit::NotAcceptable, error_message(response)
+      when 500
+        raise Octokit::InternalServerError, error_message(response)
+      when 501
+        raise Octokit::NotImplemented, error_message(response)
+      when 502
+        raise Octokit::BadGateway, error_message(response)
+      when 503
+        raise Octokit::ServiceUnavailable, error_message(response)
       end
     end
 
-    def initialize(app)
-      super
-    end
-
-    private
-
-    def self.error_message(response)
+    def error_message(response)
       "#{response[:method].to_s.upcase} #{response[:url].to_s}: #{response[:status]}#{(': ' + response[:body]) if response[:body]}"
     end
   end

data/lib/octokit/client/connection.rb

@@ -17,16 +17,16 @@ module Octokit
         options.merge!(:params => { :access_token => oauth_token }) if oauthed? && !authenticated?
 
         Faraday::Connection.new(options) do |connection|
-          connection.adapter(adapter)
-          connection.basic_auth authentication[:login], authentication[:password] if authenticate and authenticated?
           connection.use Faraday::Response::RaiseError
           unless raw
+            connection.use Faraday::Response::Mashify
             case format.to_s.downcase
             when 'json' then connection.use Faraday::Response::ParseJson
             when 'xml' then connection.use Faraday::Response::ParseXml
             end
-            connection.use Faraday::Response::Mashify
           end
+          connection.basic_auth authentication[:login], authentication[:password] if authenticate and authenticated?
+          connection.adapter(adapter)
         end
       end
     end

data/lib/octokit/version.rb

@@ -1,3 +1,3 @@
 module Octokit
-  VERSION = "0.6.0".freeze unless defined?(Octokit::VERSION)
+  VERSION = "0.6.1".freeze unless defined?(Octokit::VERSION)
 end

data/octokit.gemspec

@@ -11,8 +11,9 @@ Gem::Specification.new do |s|
   s.add_development_dependency('ZenTest', '~> 4.5')
   s.add_runtime_dependency('addressable', '~> 2.2.4')
   s.add_runtime_dependency('hashie', '~> 1.0.0')
-  s.add_runtime_dependency('faraday', '~> 0.5.4')
-  s.add_runtime_dependency('faraday_middleware', '~> 0.3.2')
+  s.add_runtime_dependency('faraday', '~> 0.6.0')
+  s.add_runtime_dependency('faraday_middleware', '~> 0.6.0')
+  s.add_runtime_dependency('jruby-openssl', '~> 0.7.3') if RUBY_PLATFORM == 'java'
   s.add_runtime_dependency('multi_json', '~> 0.0.5')
   s.add_runtime_dependency('multi_xml', '~> 0.2.0')
   s.name = 'octokit'

data/spec/octokit/client/issues_spec.rb

@@ -65,7 +65,6 @@ describe Octokit::Client::Issues do
 
     it "should create an issue" do
       stub_post("issues/open/sferik/rails_admin").
-        with(:body => {:title => "Use OrmAdapter instead of talking directly to ActiveRecord", :body => "Hi,\n\nI just tried to play with this in an app with no ActiveRecord.  I was disappointed.  It seems the only reason the engine relies on AR is to provide History functionality.  I would argue that having the History in a database, and therefore tying the app to AR & SQL, isn't worth it.  How about we change it to just dump to a CSV and remove the AR dep?\n\n$0.02"}).
         to_return(:body => fixture("issue.json"))
       issue = @client.create_issue("sferik/rails_admin", "Use OrmAdapter instead of talking directly to ActiveRecord", "Hi,\n\nI just tried to play with this in an app with no ActiveRecord.  I was disappointed.  It seems the only reason the engine relies on AR is to provide History functionality.  I would argue that having the History in a database, and therefore tying the app to AR & SQL, isn't worth it.  How about we change it to just dump to a CSV and remove the AR dep?\n\n$0.02")
       issue.number.should == 105
@@ -99,7 +98,6 @@ describe Octokit::Client::Issues do
 
     it "should update an issue" do
       stub_post("issues/edit/sferik/rails_admin/105").
-        with(:body => {:title => "Use OrmAdapter instead of talking directly to ActiveRecord", :body => "Hi,\n\nI just tried to play with this in an app with no ActiveRecord.  I was disappointed.  It seems the only reason the engine relies on AR is to provide History functionality.  I would argue that having the History in a database, and therefore tying the app to AR & SQL, isn't worth it.  How about we change it to just dump to a CSV and remove the AR dep?\n\n$0.02"}).
         to_return(:body => fixture("issue.json"))
       issue = @client.update_issue("sferik/rails_admin", 105, "Use OrmAdapter instead of talking directly to ActiveRecord", "Hi,\n\nI just tried to play with this in an app with no ActiveRecord.  I was disappointed.  It seems the only reason the engine relies on AR is to provide History functionality.  I would argue that having the History in a database, and therefore tying the app to AR & SQL, isn't worth it.  How about we change it to just dump to a CSV and remove the AR dep?\n\n$0.02")
       issue.number.should == 105

metadata

@@ -2,7 +2,7 @@
 name: octokit
 version: !ruby/object:Gem::Version 
   prerelease: 
-  version: 0.6.0
+  version: 0.6.1
 platform: ruby
 authors: 
 - Wynn Netherland
@@ -12,8 +12,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-03-20 00:00:00 -07:00
-default_executable: 
+date: 2011-04-06 00:00:00 Z
 dependencies: 
 - !ruby/object:Gem::Dependency 
   name: json
@@ -122,7 +121,7 @@ dependencies:
     requirements: 
     - - ~>
       - !ruby/object:Gem::Version 
-        version: 0.5.4
+        version: 0.6.0
   type: :runtime
   version_requirements: *id010
 - !ruby/object:Gem::Dependency 
@@ -133,7 +132,7 @@ dependencies:
     requirements: 
     - - ~>
       - !ruby/object:Gem::Version 
-        version: 0.3.2
+        version: 0.6.0
   type: :runtime
   version_requirements: *id011
 - !ruby/object:Gem::Dependency 
@@ -172,6 +171,7 @@ files:
 - .gemtest
 - .gitignore
 - .rspec
+- .travis.yml
 - Gemfile
 - LICENSE
 - README.markdown
@@ -248,7 +248,6 @@ files:
 - spec/octokit/client_spec.rb
 - spec/octokit_spec.rb
 - spec/repository_spec.rb
-has_rdoc: true
 homepage: http://wynnnetherland.com/projects/octokit/
 licenses: []
 
@@ -272,7 +271,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
 requirements: []
 
 rubyforge_project: 
-rubygems_version: 1.6.2
+rubygems_version: 1.7.2
 signing_key: 
 specification_version: 3
 summary: Wrapper for the GitHub API