artifactory 2.5.2 → 2.6.0

data/lib/artifactory/errors.rb

@@ -24,8 +24,8 @@ module Artifactory
       attr_reader :code
 
       def initialize(hash = {})
-        @code = hash['status'].to_i
-        @http = hash['message'].to_s
+        @code = hash["status"].to_i
+        @http = hash["message"].to_s
 
         super "The Artifactory server responded with an HTTP Error " \
               "#{@code}: `#{@http}'"

data/lib/artifactory/resources/artifact.rb

@@ -14,7 +14,8 @@
 # limitations under the License.
 #
 
-require 'tempfile'
+require "tempfile"
+require "time"
 
 module Artifactory
   class Resource::Artifact < Resource::Base
@@ -46,8 +47,8 @@ module Artifactory
         params = Util.slice(options, :name, :repos)
         format_repos!(params)
 
-        client.get('/api/search/artifact', params)['results'].map do |artifact|
-          from_url(artifact['uri'], client: client)
+        client.get("/api/search/artifact", params)["results"].map do |artifact|
+          from_url(artifact["uri"], client: client)
         end
       end
 
@@ -97,13 +98,13 @@ module Artifactory
           :group      => :g,
           :name       => :a,
           :version    => :v,
-          :classifier => :c,
+          :classifier => :c
         )
         params = Util.slice(options, :g, :a, :v, :c, :repos)
         format_repos!(params)
 
-        client.get('/api/search/gavc', params)['results'].map do |artifact|
-          from_url(artifact['uri'], client: client)
+        client.get("/api/search/gavc", params)["results"].map do |artifact|
+          from_url(artifact["uri"], client: client)
         end
       end
 
@@ -141,8 +142,8 @@ module Artifactory
         params = options.dup
         format_repos!(params)
 
-        client.get('/api/search/prop', params)['results'].map do |artifact|
-          from_url(artifact['uri'], client: client)
+        client.get("/api/search/prop", params)["results"].map do |artifact|
+          from_url(artifact["uri"], client: client)
         end
       end
 
@@ -180,8 +181,8 @@ module Artifactory
         params = Util.slice(options, :md5, :sha1, :repos)
         format_repos!(params)
 
-        client.get('/api/search/checksum', params)['results'].map do |artifact|
-          from_url(artifact['uri'], client: client)
+        client.get("/api/search/checksum", params)["results"].map do |artifact|
+          from_url(artifact["uri"], client: client)
         end
       end
 
@@ -221,8 +222,8 @@ module Artifactory
         params = Util.slice(options, :notUsedSince, :createdBefore, :repos)
         format_repos!(params)
 
-        client.get('/api/search/usage', params)['results'].map do |artifact|
-          from_url(artifact['uri'], client: client)
+        client.get("/api/search/usage", params)["results"].map do |artifact|
+          from_url(artifact["uri"], client: client)
         end
       end
 
@@ -262,8 +263,8 @@ module Artifactory
         params = Util.slice(options, :from, :to, :repos)
         format_repos!(params)
 
-        client.get('/api/search/creation', params)['results'].map do |artifact|
-          from_url(artifact['uri'], client: client)
+        client.get("/api/search/creation", params)["results"].map do |artifact|
+          from_url(artifact["uri"], client: client)
         end
       end
 
@@ -292,12 +293,12 @@ module Artifactory
         options = Util.rename_keys(options,
           :group   => :g,
           :name    => :a,
-          :version => :v,
+          :version => :v
         )
         params = Util.slice(options, :g, :a, :v, :repos)
         format_repos!(params)
 
-        client.get('/api/search/versions', params)['results']
+        client.get("/api/search/versions", params)["results"]
       rescue Error::HTTPError => e
         raise unless e.code == 404
         []
@@ -343,7 +344,7 @@ module Artifactory
         options = Util.rename_keys(options,
           :group   => :g,
           :name    => :a,
-          :version => :v,
+          :version => :v
         )
         params = Util.slice(options, :g, :a, :v, :repos, :remote)
         format_repos!(params)
@@ -352,7 +353,7 @@ module Artifactory
         # literal "1"...
         params[:remote] = 1 if options[:remote]
 
-        client.get('/api/search/latestVersion', params)
+        client.get("/api/search/latestVersion", params)
       rescue Error::HTTPError => e
         raise unless e.code == 404
         nil
@@ -371,14 +372,14 @@ module Artifactory
       end
     end
 
-    attribute :uri, ->{ raise 'API path missing!' }
+    attribute :uri, -> { raise "API path missing!" }
     attribute :checksums
     attribute :created
-    attribute :download_uri, ->{ raise 'Download URI missing!' }
+    attribute :download_uri, -> { raise "Download URI missing!" }
     attribute :key
     attribute :last_modified
     attribute :last_updated
-    attribute :local_path, ->{ raise 'Local destination missing!' }
+    attribute :local_path, -> { raise "Local destination missing!" }
     attribute :mime_type
     attribute :repo
     attribute :size
@@ -389,7 +390,7 @@ module Artifactory
     # @return [String]
     #
     def sha1
-      checksums && checksums['sha1']
+      checksums && checksums["sha1"]
     end
 
     #
@@ -398,7 +399,7 @@ module Artifactory
     # @return [String]
     #
     def md5
-      checksums && checksums['md5']
+      checksums && checksums["md5"]
     end
 
     #
@@ -438,7 +439,7 @@ module Artifactory
     #   the list of properties
     #
     def properties
-      @properties ||= client.get(File.join('/api/storage', relative_path), properties: nil)['properties']
+      @properties ||= client.get(File.join("/api/storage", relative_path), properties: nil)["properties"]
     end
 
     #
@@ -453,7 +454,7 @@ module Artifactory
     # @return [Hash<String, Array<Hash>>]
     #
     def compliance
-      @compliance ||= client.get(File.join('/api/compliance', relative_path))
+      @compliance ||= client.get(File.join("/api/compliance", relative_path))
     end
 
     #
@@ -489,7 +490,7 @@ module Artifactory
       # Construct the full path for the file
       destination = File.join(target, filename)
 
-      File.open(destination, 'wb') do |file|
+      File.open(destination, "wb") do |file|
         file.write(client.get(download_uri))
       end
 
@@ -535,8 +536,8 @@ module Artifactory
       endpoint = File.join("#{url_safe(repo)}#{matrix}", remote_path)
 
       # Include checksums in headers if given.
-      headers['X-Checksum-Md5'] = md5   if md5
-      headers['X-Checksum-Sha1'] = sha1 if sha1
+      headers["X-Checksum-Md5"] = md5   if md5
+      headers["X-Checksum-Sha1"] = sha1 if sha1
 
       response = client.put(endpoint, file, headers)
 
@@ -593,8 +594,8 @@ module Artifactory
     #
     def upload_with_checksum(repo, remote_path, checksum, properties = {})
       upload(repo, remote_path, properties,
-        'X-Checksum-Deploy' => true,
-        'X-Checksum-Sha1'   => checksum,
+        "X-Checksum-Deploy" => true,
+        "X-Checksum-Sha1"   => checksum
       )
     end
 
@@ -612,7 +613,7 @@ module Artifactory
     #
     def upload_from_archive(repo, remote_path, properties = {})
       upload(repo, remote_path, properties,
-        'X-Explode-Archive' => true,
+        "X-Explode-Archive" => true
       )
     end
 
@@ -628,7 +629,7 @@ module Artifactory
     # @return [String]
     #
     def relative_path
-      @relative_path ||= uri.split('/api/storage', 2).last
+      @relative_path ||= uri.split("/api/storage", 2).last
     end
 
     #
@@ -664,7 +665,7 @@ module Artifactory
         param[:dry]             = 1 if options[:dry_run]
       end
 
-      endpoint = File.join('/api', action.to_s, relative_path) + "?#{to_query_string_parameters(params)}"
+      endpoint = File.join("/api", action.to_s, relative_path) + "?#{to_query_string_parameters(params)}"
 
       client.post(endpoint, {})
     end

data/lib/artifactory/resources/backup.rb

@@ -14,7 +14,7 @@
 # limitations under the License.
 #
 
-require 'rexml/document'
+require "rexml/document"
 
 module Artifactory
   class Resource::Backup < Resource::Base
@@ -33,7 +33,7 @@ module Artifactory
       #
       def all(options = {})
         config = Resource::System.configuration(options)
-        list_from_config('config/backups/backup', config, options)
+        list_from_config("config/backups/backup", config, options)
       end
 
       #
@@ -81,7 +81,7 @@ module Artifactory
       #
       def list_from_config(xpath, config, options = {})
         REXML::XPath.match(config, xpath).map do |r|
-          hash = Util.xml_to_hash(r, 'excludedRepositories', false)
+          hash = Util.xml_to_hash(r, "excludedRepositories", false)
           from_hash(hash, options)
         end
       end
@@ -102,12 +102,12 @@ module Artifactory
       def find_from_config(xpath, config, options = {})
         name_node = REXML::XPath.match(config, xpath)
         return nil if name_node.empty?
-        properties = Util.xml_to_hash(name_node[0].parent, 'excludedRepositories', false)
+        properties = Util.xml_to_hash(name_node[0].parent, "excludedRepositories", false)
         from_hash(properties, options)
       end
     end
 
-    attribute :key, ->{ raise 'name missing!' }
+    attribute :key, -> { raise "name missing!" }
     attribute :enabled, true
     attribute :dir
     attribute :cron_exp

data/lib/artifactory/resources/base.rb

@@ -14,9 +14,9 @@
 # limitations under the License.
 #
 
-require 'cgi'
-require 'json'
-require 'uri'
+require "cgi"
+require "json"
+require "uri"
 
 module Artifactory
   class Resource::Base
@@ -227,7 +227,7 @@ module Artifactory
       #
       def format_repos!(options)
         return options if options[:repos].nil? || options[:repos].empty?
-        options[:repos] = Array(options[:repos]).compact.join(',')
+        options[:repos] = Array(options[:repos]).compact.join(",")
         options
       end
 
@@ -245,7 +245,7 @@ module Artifactory
       end
     end
 
-    attribute :client, ->{ Artifactory.client }
+    attribute :client, -> { Artifactory.client }
 
     #
     # Create a new instance
@@ -360,7 +360,7 @@ module Artifactory
       if properties.empty?
         nil
       else
-        properties.join('&')
+        properties.join("&")
       end
     end
 
@@ -383,7 +383,7 @@ module Artifactory
     private
 
     def short_classname
-      @short_classname ||= self.class.name.split('::').last
+      @short_classname ||= self.class.name.split("::").last
     end
   end
 end

data/lib/artifactory/resources/build.rb

@@ -14,13 +14,13 @@
 # limitations under the License.
 #
 
-require 'time'
+require "time"
 
 module Artifactory
   class Resource::Build < Resource::Base
-    BUILD_SCHEMA_VERSION = '1.0.1'.freeze
+    BUILD_SCHEMA_VERSION = "1.0.1".freeze
     # valid build types as dictated by the Artifactory API
-    BUILD_TYPES = %w( ANT IVY MAVEN GENERIC GRADLE )
+    BUILD_TYPES = %w{ ANT IVY MAVEN GENERIC GRADLE }
 
     class << self
       #
@@ -39,9 +39,9 @@ module Artifactory
       #
       def all(name, options = {})
         client = extract_client!(options)
-        client.get("/api/build/#{url_safe(name)}")['buildsNumbers'].map do |build_number|
+        client.get("/api/build/#{url_safe(name)}")["buildsNumbers"].map do |build_number|
           # Remove the leading / from the `uri` value. Converts `/484` to `484`.
-          number = build_number['uri'].slice(1..-1)
+          number = build_number["uri"].slice(1..-1)
           find(name, number, client: client)
         end.compact.flatten
       rescue Error::HTTPError => e
@@ -75,7 +75,7 @@ module Artifactory
       def find(name, number, options = {})
         client = extract_client!(options)
         response = client.get("/api/build/#{url_safe(name)}/#{url_safe(number)}")
-        from_hash(response['buildInfo'], client: client)
+        from_hash(response["buildInfo"], client: client)
       rescue Error::HTTPError => e
         raise unless e.code == 404
         nil
@@ -95,9 +95,9 @@ module Artifactory
     # Based on https://github.com/JFrogDev/build-info/blob/master/README.md#build-info-json-format
     attribute :properties, {}
     attribute :version, BUILD_SCHEMA_VERSION
-    attribute :name, ->{ raise 'Build component missing!' }
-    attribute :number, ->{ raise 'Build number missing!' }
-    attribute :type, 'GENERIC'
+    attribute :name, -> { raise "Build component missing!" }
+    attribute :number, -> { raise "Build number missing!" }
+    attribute :type, "GENERIC"
     attribute :build_agent, {}
     attribute :agent, {}
     attribute :started, Time.now.utc.iso8601(3)
@@ -126,7 +126,7 @@ module Artifactory
     #   the list of properties
     #
     def diff(previous_build_number)
-      endpoint = api_path + '?' "diff=#{url_safe(previous_build_number)}"
+      endpoint = api_path + "?" "diff=#{url_safe(previous_build_number)}"
       client.get(endpoint, {})
     end
 
@@ -173,8 +173,8 @@ module Artifactory
     #
     def promote(target_repo, options = {})
       request_body = {}.tap do |body|
-        body[:status]       = options[:status] || 'promoted'
-        body[:comment]      = options[:comment] || ''
+        body[:status]       = options[:status] || "promoted"
+        body[:comment]      = options[:comment] || ""
         body[:ciUser]       = options[:user] || Artifactory.username
         body[:dryRun]       = options[:dry_run] || false
         body[:targetRepo]   = target_repo
@@ -188,7 +188,7 @@ module Artifactory
 
       endpoint = "/api/build/promote/#{url_safe(name)}/#{url_safe(number)}"
       client.post(endpoint, JSON.fast_generate(request_body),
-        'Content-Type' => 'application/json'
+        "Content-Type" => "application/json"
       )
     end
 
@@ -204,8 +204,8 @@ module Artifactory
       file.write(to_json)
       file.rewind
 
-      client.put('/api/build', file,
-        'Content-Type' => 'application/json'
+      client.put("/api/build", file,
+        "Content-Type" => "application/json"
       )
       true
     ensure

data/lib/artifactory/resources/build_component.rb

@@ -14,7 +14,7 @@
 # limitations under the License.
 #
 
-require 'time'
+require "time"
 
 module Artifactory
   class Resource::BuildComponent < Resource::Base
@@ -33,7 +33,7 @@ module Artifactory
       #
       def all(options = {})
         client = extract_client!(options)
-        client.get('/api/build')['builds'].map do |component|
+        client.get("/api/build")["builds"].map do |component|
           from_hash(component, client: client)
         end.compact.flatten
       rescue Error::HTTPError => e
@@ -82,7 +82,7 @@ module Artifactory
     end
 
     attribute :uri
-    attribute :name, ->{ raise 'Name missing!' }
+    attribute :name, -> { raise "Name missing!" }
     attribute :last_started
 
     #
@@ -117,7 +117,7 @@ module Artifactory
     #
     def delete(options = {})
       params = {}.tap do |param|
-        param[:buildNumbers] = options[:build_numbers].join(',') if options[:build_numbers]
+        param[:buildNumbers] = options[:build_numbers].join(",") if options[:build_numbers]
         param[:artifacts]    = 1 if options[:artifacts]
         param[:deleteAll]    = 1 if options[:delete_all]
       end

data/lib/artifactory/resources/group.rb

@@ -31,8 +31,8 @@ module Artifactory
       #
       def all(options = {})
         client = extract_client!(options)
-        client.get('/api/security/groups').map do |hash|
-          from_url(hash['uri'], client: client)
+        client.get("/api/security/groups").map do |hash|
+          from_url(hash["uri"], client: client)
         end
       end
 
@@ -67,7 +67,7 @@ module Artifactory
 
     attribute :auto_join
     attribute :description
-    attribute :name, ->{ raise 'Name missing!' }
+    attribute :name, -> { raise "Name missing!" }
     attribute :realm
     attribute :realm_attributes
 
@@ -118,7 +118,7 @@ module Artifactory
     #
     def headers
       @headers ||= {
-        'Content-Type' => 'application/vnd.org.jfrog.artifactory.security.Group+json'
+        "Content-Type" => "application/vnd.org.jfrog.artifactory.security.Group+json",
       }
     end
   end