gh 0.7.1 → 0.7.3

data/lib/gh/faraday_adapter.rb

@@ -29,10 +29,13 @@ module GH
         @mutex.synchronize { requests, @requests = @requests, {} }
         http = Net::HTTP::Persistent.new 'GH'
         requests.each do |url, envs|
-          requests  = envs.map { |env| env[:adapter].create_request(env) }
-          responses = http.pipeline(url, requests)
-          envs.zip(responses) do |e,r|
-            e[:adapter].save_response(e, r.code.to_i, r.body) { |h| r.each_header { |k,v| h[k] = v } }
+          _requests = envs.map { |env| env[:adapter].create_request(env) }
+          responses = http.pipeline(url, _requests)
+          envs.zip(responses) do |env, http_response|
+            env[:adapter].save_response(env, http_response.code.to_i, http_response.body) do |headers|
+              http_response.each_header { |key, value| headers[key] = value }
+            end
+            env[:response].finish(env)
           end
         end
       end

data/lib/gh/merge_commit.rb

@@ -57,9 +57,18 @@ module GH
         # until then, please look the other way
         while hash['mergeable'].nil?
           url = hash['_links']['html']['href'] + '/mergeable'
-          case frontend.http(:get, url).body
-          when "true"  then hash['mergeable'] = true
-          when "false" then hash['mergeable'] = false
+          payload = frontend.http(:get, url).body
+
+          case payload
+          when "true", /clean/
+            hash['mergeable'] = true
+          when "", "null", /checking/
+            hash['mergeable'] = nil
+            sleep 0.1
+          when /unknown/, /dirty/
+            hash['mergeable'] = false
+          else
+            fail "Unkown payload from #{url}: #{payload}"
           end
         end
       end

data/lib/gh/pagination.rb

@@ -23,6 +23,10 @@ module GH
         next_page[value - @page.size]
       end
 
+      def to_ary
+        to_a # replace with better implementation (use in_parallel)
+      end
+
       private
 
       def next_page

data/lib/gh/version.rb

@@ -1,4 +1,4 @@
 module GH
   # Public: Library version.
-  VERSION = "0.7.1"
+  VERSION = "0.7.3"
 end

data/spec/merge_commit_spec.rb

@@ -8,7 +8,7 @@ describe GH::MergeCommit do
 
   before do
     stub_request(:get, "https://github.com/travis-repos/test-project-1/pull/1/mergeable").
-      to_return(:status => 200, :body => "true", :headers => {})
+      to_return(:status => 200, :body => '{"state":"clean"}', :headers => {})
   end
 
   it 'adds merge commits' do

data/spec/parallel_spec.rb

@@ -68,7 +68,26 @@ describe GH::Parallel do
     end
   end
 
+  it 'works with pagination' do
+    WebMock.allow_net_connect!
+    GH::DefaultStack.replace GH::MockBackend, GH::Remote
+    repos = GH.in_parallel { GH['users/rkh/repos'] }
+    counter = repos.to_a.map { 1 }.reduce(:+)
+    counter.should be > 120
+  end
+
   it 'returns the block value' do
     GH.in_parallel { 42 }.should be == 42
   end
+
+  it 'works two times in a row' do
+    WebMock.allow_net_connect!
+    GH::DefaultStack.replace GH::MockBackend, GH::Remote
+
+    a = GH.in_parallel { GH['users/rkh'] }
+    b = GH.in_parallel { GH['users/svenfuchs'] }
+
+    a['name'].should be == "Konstantin Haase"
+    b['name'].should be == "Sven Fuchs"
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: gh
 version: !ruby/object:Gem::Version
-  version: 0.7.1
+  version: 0.7.3
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-06-10 00:00:00.000000000 Z
+date: 2012-06-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &70105565578080 !ruby/object:Gem::Requirement
+  requirement: &70211459176640 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70105565578080
+  version_requirements: *70211459176640
 - !ruby/object:Gem::Dependency
   name: webmock
-  requirement: &70105565576840 !ruby/object:Gem::Requirement
+  requirement: &70211459176060 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,10 +32,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70105565576840
+  version_requirements: *70211459176060
 - !ruby/object:Gem::Dependency
   name: faraday
-  requirement: &70105565576200 !ruby/object:Gem::Requirement
+  requirement: &70211459175400 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -43,10 +43,10 @@ dependencies:
         version: '0.8'
   type: :runtime
   prerelease: false
-  version_requirements: *70105565576200
+  version_requirements: *70211459175400
 - !ruby/object:Gem::Dependency
   name: backports
-  requirement: &70105565586200 !ruby/object:Gem::Requirement
+  requirement: &70211459174700 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -54,10 +54,10 @@ dependencies:
         version: '2.3'
   type: :runtime
   prerelease: false
-  version_requirements: *70105565586200
+  version_requirements: *70211459174700
 - !ruby/object:Gem::Dependency
   name: multi_json
-  requirement: &70105565583400 !ruby/object:Gem::Requirement
+  requirement: &70211459174140 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -65,10 +65,10 @@ dependencies:
         version: '1.0'
   type: :runtime
   prerelease: false
-  version_requirements: *70105565583400
+  version_requirements: *70211459174140
 - !ruby/object:Gem::Dependency
   name: addressable
-  requirement: &70105565597220 !ruby/object:Gem::Requirement
+  requirement: &70211459173620 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -76,10 +76,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70105565597220
+  version_requirements: *70211459173620
 - !ruby/object:Gem::Dependency
   name: net-http-persistent
-  requirement: &70105565595200 !ruby/object:Gem::Requirement
+  requirement: &70211459172920 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -87,10 +87,10 @@ dependencies:
         version: '2.7'
   type: :runtime
   prerelease: false
-  version_requirements: *70105565595200
+  version_requirements: *70211459172920
 - !ruby/object:Gem::Dependency
   name: net-http-pipeline
-  requirement: &70105565593540 !ruby/object:Gem::Requirement
+  requirement: &70211459172100 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -98,7 +98,7 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70105565593540
+  version_requirements: *70211459172100
 description: multi-layer client for the github api v3
 email:
 - konstantin.mailinglists@googlemail.com