dpl-releases 1.10.2.travis.2979.6 → 1.10.2.travis.2987.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f95fdcc296b216b418617a0c302b1ef6650e3c64020202e73bd4c0906e484489
-  data.tar.gz: 6ac4a45fdf59e04c6bfb59996ba3affb15b6be553b57577f6d76aec1a7f35e45
+  metadata.gz: 745b63e7e5778453396bd778b8a34cbb110514bf833ceac6739ea30e448678d2
+  data.tar.gz: e9889f276ac693997b57d8955ca26f2cbfd48b270a677324ab5d6f279ca98e3d
 SHA512:
-  metadata.gz: 0a6befafd3936fb49cbcbcf52d3b71f28488d2a503fe6664f7e25faf4e1e5b839cd9e4e65cb6842749c797007f427aca5dd3b0be43567f1c761bf33c09aa4050
-  data.tar.gz: 4e3c78ea4ab1124a73096bb2b9975754e6942a3c299f3d70102639a874a1e5bcd62208ab2afe65b647a78bb49833c3987d35ab25323f10522dce25ce4049722c
+  metadata.gz: 7fdf3931bd43fb97b99abc0de981cf764a53dad153c043d82adc8a89b59bd7ea846257cfa5c811182a96eee035c7cb61ab5019fca20053b149a017faff8daf2b
+  data.tar.gz: e3aae922ad30007712b682c7ef0356d0c433329fa027880fa486e49641d791d48e6eeb947b78a53bb538de68bcd13efc2de1151f71c2492bb746e6d034c83698

data/lib/dpl/provider/releases.rb

@@ -36,9 +36,9 @@ module DPL
           }
         }
         if options[:user] and options[:password]
-          @api ||= Octokit::Client.new(:login => options[:user], :password => options[:password], :connection_options => connection_options)
+          @api ||= Octokit::Client.new(:login => options[:user], :password => options[:password], :auto_paginate => true, :connection_options => connection_options)
         else
-          @api ||= Octokit::Client.new(:access_token => option(:api_key), :connection_options => connection_options)
+          @api ||= Octokit::Client.new(:access_token => option(:api_key), :auto_paginate => true, :connection_options => connection_options)
         end
       end
 
@@ -116,7 +116,7 @@ module DPL
           next unless File.file?(file)
           existing_url = nil
           filename = Pathname.new(file).basename.to_s
-          api.release(release_url).rels[:assets].get.data.each do |existing_file|
+          api.release_assets(release_url).each do |existing_file|
             if existing_file.name == filename
               existing_url = existing_file.url
             end

data/spec/provider/releases_spec.rb

@@ -30,7 +30,7 @@ describe DPL::Provider::Releases do
   describe "#api" do
     example "With API key" do
       api = double(:api)
-      expect(::Octokit::Client).to receive(:new).with(:access_token => '0123445789qwertyuiop0123445789qwertyuiop', :connection_options=>{:request=>{:timeout=>180, :open_timeout=>180}}).and_return(api)
+      expect(::Octokit::Client).to receive(:new).with(:access_token => '0123445789qwertyuiop0123445789qwertyuiop', :auto_paginate => true, :connection_options=>{:request=>{:timeout=>180, :open_timeout=>180}}).and_return(api)
       expect(provider.api).to eq(api)
     end
 
@@ -39,7 +39,7 @@ describe DPL::Provider::Releases do
       provider.options.update(:user => 'foo')
       provider.options.update(:password => 'bar')
 
-      expect(::Octokit::Client).to receive(:new).with(:login => 'foo', :password  => 'bar', :connection_options=>{:request=>{:timeout=>180, :open_timeout=>180}}).and_return(api)
+      expect(::Octokit::Client).to receive(:new).with(:login => 'foo', :password  => 'bar', :auto_paginate => true, :connection_options=>{:request=>{:timeout=>180, :open_timeout=>180}}).and_return(api)
       expect(provider.api).to eq(api)
     end
   end
@@ -169,10 +169,7 @@ describe DPL::Provider::Releases do
         allow(release.rels[:self]).to receive(:href)
       end
 
-      allow(provider.api).to receive(:release)
-      allow(provider.api.release).to receive(:rels).and_return({:assets => nil})
-      allow(provider.api.release.rels[:assets]).to receive(:get).and_return({:data => [""]})
-      allow(provider.api.release.rels[:assets].get).to receive(:data).and_return([])
+      allow(provider.api).to receive(:release_assets).and_return([])
 
       expect(provider.api).to receive(:upload_asset).with(anything, "test/foo.bar", {:name=>"foo.bar", :content_type=>"application/octet-stream"})
       expect(provider.api).to receive(:upload_asset).with(anything, "bar.txt", {:name=>"bar.txt", :content_type=>"text/plain"})
@@ -197,10 +194,7 @@ describe DPL::Provider::Releases do
         allow(release.rels[:self]).to receive(:href)
       end
 
-      allow(provider.api).to receive(:release)
-      allow(provider.api.release).to receive(:rels).and_return({:assets => nil})
-      allow(provider.api.release.rels[:assets]).to receive(:get).and_return({:data => [""]})
-      allow(provider.api.release.rels[:assets].get).to receive(:data).and_return([double(:name => "foo.bar", :url => 'foo-bar-url'), double(:name => "foo.foo", :url => 'foo-foo-url')])
+      allow(provider.api).to receive(:release_assets).and_return([double(:name => "foo.bar", :url => 'foo-bar-url'), double(:name => "foo.foo", :url => 'foo-foo-url')])
 
       expect(provider.api).to receive(:upload_asset).with(anything, "bar.txt", {:name=>"bar.txt", :content_type=>"text/plain"})
       expect(provider).to receive(:log).with("foo.bar already exists, skipping.")
@@ -225,10 +219,7 @@ describe DPL::Provider::Releases do
         allow(release.rels[:self]).to receive(:href)
       end
 
-      allow(provider.api).to receive(:release)
-      allow(provider.api.release).to receive(:rels).and_return({:assets => nil})
-      allow(provider.api.release.rels[:assets]).to receive(:get).and_return({:data => [""]})
-      allow(provider.api.release.rels[:assets].get).to receive(:data).and_return([double(:name => "exists.txt", :url => "release-url")])
+      allow(provider.api).to receive(:release_assets).and_return([double(:name => "exists.txt", :url => "release-url")])
 
       expect(provider.api).to receive(:delete_release_asset).with("release-url").and_return(true)
       expect(provider.api).to receive(:upload_asset).with(anything, "exists.txt", {:name=>"exists.txt", :content_type=>"text/plain"})
@@ -256,10 +247,7 @@ describe DPL::Provider::Releases do
       allow(provider.api.create_release).to receive(:rels).and_return({:self => nil})
       allow(provider.api.create_release.rels[:slef]).to receive(:href)
 
-      allow(provider.api).to receive(:release)
-      allow(provider.api.release).to receive(:rels).and_return({:assets => nil})
-      allow(provider.api.release.rels[:assets]).to receive(:get).and_return({:data => nil})
-      allow(provider.api.release.rels[:assets].get).to receive(:data).and_return([])
+      allow(provider.api).to receive(:release_assets).and_return([])
 
       expect(provider.api).to receive(:upload_asset).with(anything, "test/foo.bar", {:name=>"foo.bar", :content_type=>"application/octet-stream"})
       expect(provider.api).to receive(:upload_asset).with(anything, "bar.txt", {:name=>"bar.txt", :content_type=>"text/plain"})
@@ -277,10 +265,7 @@ describe DPL::Provider::Releases do
       allow(provider).to receive(:slug).and_return("foo/bar")
       expect(File).to receive(:file?).with("bar.txt").and_return(true)
 
-      allow(provider.api).to receive(:release)
-      allow(provider.api.release).to receive(:rels).and_return({:assets => nil})
-      allow(provider.api.release.rels[:assets]).to receive(:get).and_return({:data => nil})
-      allow(provider.api.release.rels[:assets].get).to receive(:data).and_return([])
+      allow(provider.api).to receive(:release_assets).and_return([])
 
       expect(provider.api).to receive(:upload_asset).with("https://api.github.com/repos/foo/bar/releases/1234", "bar.txt", {:name=>"bar.txt", :content_type=>"text/plain"})
       expect(provider.api).to receive(:update_release).with(anything, hash_including(:draft => false))
@@ -297,10 +282,7 @@ describe DPL::Provider::Releases do
       allow(provider).to receive(:slug).and_return("foo/bar")
       expect(File).to receive(:file?).with("bar.txt").and_return(true)
 
-      allow(provider.api).to receive(:release)
-      allow(provider.api.release).to receive(:rels).and_return({:assets => nil})
-      allow(provider.api.release.rels[:assets]).to receive(:get).and_return({:data => nil})
-      allow(provider.api.release.rels[:assets].get).to receive(:data).and_return([])
+      allow(provider.api).to receive(:release_assets).and_return([])
 
       expect(provider.api).to receive(:upload_asset).with("https://api.github.com/repos/foo/bar/releases/1234", "bar.txt", {:name=>"bar.txt", :content_type=>"text/plain"})
       expect(provider.api).to receive(:update_release).with(anything, hash_including(:draft => true))
@@ -317,10 +299,7 @@ describe DPL::Provider::Releases do
       allow(provider).to receive(:slug).and_return("foo/bar")
       expect(File).to receive(:file?).with("bar.txt").and_return(true)
 
-      allow(provider.api).to receive(:release)
-      allow(provider.api.release).to receive(:rels).and_return({:assets => nil})
-      allow(provider.api.release.rels[:assets]).to receive(:get).and_return({:data => nil})
-      allow(provider.api.release.rels[:assets].get).to receive(:data).and_return([])
+      allow(provider.api).to receive(:release_assets).and_return([])
 
       expect(provider.api).to receive(:upload_asset).with("https://api.github.com/repos/foo/bar/releases/1234", "bar.txt", {:name=>"bar.txt", :content_type=>"text/plain"})
       expect(provider.api).to receive(:update_release).with(anything, hash_including(:prerelease => true))
@@ -338,10 +317,7 @@ describe DPL::Provider::Releases do
       allow(provider).to receive(:slug).and_return("foo/bar")
       expect(File).to receive(:file?).with("bar.txt").and_return(true)
 
-      allow(provider.api).to receive(:release)
-      allow(provider.api.release).to receive(:rels).and_return({:assets => nil})
-      allow(provider.api.release.rels[:assets]).to receive(:get).and_return({:data => nil})
-      allow(provider.api.release.rels[:assets].get).to receive(:data).and_return([])
+      allow(provider.api).to receive(:release_assets).and_return([])
 
       expect(provider.api).to receive(:upload_asset).with("https://api.github.com/repos/foo/bar/releases/1234", "bar.txt", {:name=>"bar.txt", :content_type=>"text/plain"})
       expect(provider.api).to receive(:update_release).with(anything, hash_including(:prerelease => true, :name => 'true'))

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: dpl-releases
 version: !ruby/object:Gem::Version
-  version: 1.10.2.travis.2979.6
+  version: 1.10.2.travis.2987.6
 platform: ruby
 authors:
 - Konstantin Haase
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.10.2.travis.2979.6
+        version: 1.10.2.travis.2987.6
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.10.2.travis.2979.6
+        version: 1.10.2.travis.2987.6
 - !ruby/object:Gem::Dependency
   name: octokit
   requirement: !ruby/object:Gem::Requirement