dpl-releases 1.10.6.travis.3118.5 → 1.10.6.travis.3119.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 40eb05f8bef540e7f45e878759b4c52b170c4d531961ad300065d4cf0b2e028d
-  data.tar.gz: 96ee48d95da25687046e74f6e3cbdc5ed0838e05cf2b0f7dbf1706a082bba1cf
+  metadata.gz: 1421a875eed37cc223d409b277d2561cbab484de1d4207c873f090e1f77b57cd
+  data.tar.gz: a48df6ae8d4cd0c60b64cc0552af8ea83068565362f1aca10326710ad3f2f651
 SHA512:
-  metadata.gz: 130da6adb0c0c9381497313ef5f5fde7a6eca715ffd3bc776d1f8e20c3dd9b42e148e09f84dfdc2b35479bc7918eb00d33bae2797c5c2aa1f1bd8cf941f6f3a1
-  data.tar.gz: b574dbd0aefc310734658e5d2b165b8c16493438199523abe86fb83042a309b42c2c114348e7cc00940783eb39e11e9f1a009b50b47f6f526125a391da39db37
+  metadata.gz: 4a7ba603fe13b13d756b9161e6404f3cc890f004b9e80a97863636ab58568f9d8bee519ab8c3cd41aa8e94e37b73355c5badaa8f1089f1057599380f2a12b199
+  data.tar.gz: e92619a0441f32e4b0e68c8e93677f8ee57068a244b187a8d52e0818158aea603d0a4e1a539b395053fe611627086cc579b5ab8cc015093382ab122013131b54

data/lib/dpl/provider/releases.rb

@@ -11,6 +11,10 @@ module DPL
         prerelease
       )
 
+      def self.new(context, options)
+        super(context, options.merge!({needs_git_http_user_agent: false}))
+      end
+
       def travis_tag
         # Check if $TRAVIS_TAG is unset or set but empty
         if context.env.fetch('TRAVIS_TAG','') == ''
@@ -113,7 +117,14 @@ module DPL
         end
 
         files.each do |file|
-          next unless File.file?(file)
+          unless File.exist?(file)
+            log "#{file} does not exist."
+            next
+          end
+          unless File.file?(file)
+            log "#{file} is not a regular file. Skipping."
+            next
+          end
           existing_url = nil
           filename = Pathname.new(file).basename.to_s
           api.release_assets(release_url).each do |existing_file|
@@ -132,6 +143,14 @@ module DPL
           end
         end
 
+        if ! options.key?(:tag_name) && options[:draft]
+          options[:tag_name] = get_tag.tap {|tag| log "Setting tag_name to #{tag}"}
+        end
+
+        unless options.key?(:target_commitish)
+          options[:target_commitish] = sha.tap {|commitish| log "Setting target_commitish to #{commitish}"}
+        end
+
         api.update_release(release_url, {:draft => false}.merge(options))
       end
 

data/spec/provider/releases_spec.rb

@@ -160,6 +160,8 @@ describe DPL::Provider::Releases do
 
       allow(provider).to receive(:releases).and_return([""])
       allow(provider).to receive(:get_tag).and_return("v0.0.0")
+      expect(File).to receive(:exist?).with("test/foo.bar").and_return(true)
+      expect(File).to receive(:exist?).with("bar.txt").and_return(true)
       expect(File).to receive(:file?).with("test/foo.bar").and_return(true)
       expect(File).to receive(:file?).with("bar.txt").and_return(true)
 
@@ -185,6 +187,8 @@ describe DPL::Provider::Releases do
 
       allow(provider).to receive(:releases).and_return([""])
       allow(provider).to receive(:get_tag).and_return("v0.0.0")
+      expect(File).to receive(:exist?).with("test/foo.bar").and_return(true)
+      expect(File).to receive(:exist?).with("bar.txt").and_return(true)
       expect(File).to receive(:file?).with("test/foo.bar").and_return(true)
       expect(File).to receive(:file?).with("bar.txt").and_return(true)
 
@@ -197,7 +201,7 @@ describe DPL::Provider::Releases do
       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.")
+      allow(provider).to receive(:log)
       expect(provider.api).to receive(:update_release).with(anything, hash_including(:draft => false))
 
       provider.push_app
@@ -211,6 +215,7 @@ describe DPL::Provider::Releases do
 
       allow(provider).to receive(:releases).and_return([""])
       allow(provider).to receive(:get_tag).and_return("v0.0.0")
+      expect(File).to receive(:exist?).with("exists.txt").and_return(true)
       expect(File).to receive(:file?).with("exists.txt").and_return(true)
 
       provider.releases.map do |release|
@@ -234,6 +239,8 @@ describe DPL::Provider::Releases do
       provider.options.update(:file => ["test/foo.bar", "bar.txt"])
 
       allow(provider).to receive(:releases).and_return([""])
+      expect(File).to receive(:exist?).with("test/foo.bar").and_return(true)
+      expect(File).to receive(:exist?).with("bar.txt").and_return(true)
       expect(File).to receive(:file?).with("test/foo.bar").and_return(true)
       expect(File).to receive(:file?).with("bar.txt").and_return(true)
 
@@ -263,6 +270,7 @@ describe DPL::Provider::Releases do
       provider.options.update(:release_number => "1234")
 
       allow(provider).to receive(:slug).and_return("foo/bar")
+      expect(File).to receive(:exist?).with("bar.txt").and_return(true)
       expect(File).to receive(:file?).with("bar.txt").and_return(true)
 
       allow(provider.api).to receive(:release_assets).and_return([])
@@ -280,6 +288,7 @@ describe DPL::Provider::Releases do
       provider.options.update(:release_number => "1234")
       provider.options.update(:draft => true)
       allow(provider).to receive(:slug).and_return("foo/bar")
+      expect(File).to receive(:exist?).with("bar.txt").and_return(true)
       expect(File).to receive(:file?).with("bar.txt").and_return(true)
 
       allow(provider.api).to receive(:release_assets).and_return([])
@@ -297,6 +306,7 @@ describe DPL::Provider::Releases do
       provider.options.update(:release_number => "1234")
       provider.options.update(:prerelease => 'true')
       allow(provider).to receive(:slug).and_return("foo/bar")
+      expect(File).to receive(:exist?).with("bar.txt").and_return(true)
       expect(File).to receive(:file?).with("bar.txt").and_return(true)
 
       allow(provider.api).to receive(:release_assets).and_return([])
@@ -315,6 +325,7 @@ describe DPL::Provider::Releases do
       provider.options.update(:prerelease => 'true')
       provider.options.update(:name => 'true')
       allow(provider).to receive(:slug).and_return("foo/bar")
+      expect(File).to receive(:exist?).with("bar.txt").and_return(true)
       expect(File).to receive(:file?).with("bar.txt").and_return(true)
 
       allow(provider.api).to receive(:release_assets).and_return([])

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: dpl-releases
 version: !ruby/object:Gem::Version
-  version: 1.10.6.travis.3118.5
+  version: 1.10.6.travis.3119.5
 platform: ruby
 authors:
 - Konstantin Haase
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-11-16 00:00:00.000000000 Z
+date: 2018-11-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: dpl
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.10.6.travis.3118.5
+        version: 1.10.6.travis.3119.5
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.10.6.travis.3118.5
+        version: 1.10.6.travis.3119.5
 - !ruby/object:Gem::Dependency
   name: octokit
   requirement: !ruby/object:Gem::Requirement