artifactory 2.5.2 → 2.6.0

data/spec/unit/resources/backup_spec.rb

@@ -1,4 +1,4 @@
-require 'spec_helper'
+require "spec_helper"
 
 module Artifactory
   describe Resource::Backup do
@@ -9,7 +9,7 @@ module Artifactory
       allow(client).to receive(:get).and_return(response) if defined?(response)
     end
 
-    describe '.all' do
+    describe ".all" do
       doc = <<-XML
         <config>
           <backups>
@@ -27,14 +27,14 @@ module Artifactory
         allow(Resource::System).to receive(:configuration).and_return(xml)
       end
 
-      it 'returns the backup settings' do
+      it "returns the backup settings" do
         expect(described_class.all).to be_a(Array)
         expect(described_class.all.first).to be_a(described_class)
-        expect(described_class.all.first.key).to eq('backup-daily')
+        expect(described_class.all.first.key).to eq("backup-daily")
       end
     end
 
-    describe '.find' do
+    describe ".find" do
       doc = <<-XML
         <config>
           <backups>
@@ -52,9 +52,9 @@ module Artifactory
         allow(Resource::System).to receive(:configuration).and_return(xml)
       end
 
-      it 'returns the found backup setting' do
-        expect(described_class.find('backup-weekly')).to be_a(described_class)
-        expect(described_class.find('backup-weekly').key).to eq('backup-weekly')
+      it "returns the found backup setting" do
+        expect(described_class.find("backup-weekly")).to be_a(described_class)
+        expect(described_class.find("backup-weekly").key).to eq("backup-weekly")
       end
     end
   end

data/spec/unit/resources/base_spec.rb

@@ -1,82 +1,82 @@
-require 'spec_helper'
+require "spec_helper"
 
 module Artifactory
   describe Resource::Base do
     let(:client) { double }
-    let(:endpoint_host) { 'http://33.33.33.11' }
+    let(:endpoint_host) { "http://33.33.33.11" }
     let(:endpoint) { "#{endpoint_host}/" }
 
     before do
       allow(Artifactory).to receive(:client).and_return(client)
     end
 
-    describe '.attribute' do
+    describe ".attribute" do
       before { described_class.attribute(:bacon) }
 
-      it 'defines an accessor method' do
+      it "defines an accessor method" do
         expect(subject).to respond_to(:bacon)
       end
 
-      it 'defines a setter method' do
+      it "defines a setter method" do
         expect(subject).to respond_to(:bacon=)
       end
 
-      it 'defines a boolean method' do
+      it "defines a boolean method" do
         expect(subject).to respond_to(:bacon?)
       end
     end
 
-    describe '.extract_client!' do
-      context 'when the :client key is present' do
+    describe ".extract_client!" do
+      context "when the :client key is present" do
         let(:client) { double }
         let(:options) { { client: client } }
 
-        it 'extracts the client' do
+        it "extracts the client" do
           result = described_class.extract_client!(options)
           expect(result).to be(client)
         end
 
-        it 'removes the key from the hash' do
+        it "removes the key from the hash" do
           described_class.extract_client!(options)
           expect(options).to_not have_key(:client)
         end
       end
 
-      context 'when the :client key is not present' do
+      context "when the :client key is not present" do
         let(:client) { double }
         before { allow(Artifactory).to receive(:client).and_return(client) }
 
-        it 'uses Artifactory.client' do
+        it "uses Artifactory.client" do
           expect(described_class.extract_client!({})).to be(client)
         end
       end
     end
 
-    describe '.format_repos!' do
-      context 'when the :repos key is present' do
-        it 'joins an array' do
-          options = { repos: ['bacon', 'bits'] }
+    describe ".format_repos!" do
+      context "when the :repos key is present" do
+        it "joins an array" do
+          options = { repos: %w{bacon bits} }
           described_class.format_repos!(options)
-          expect(options[:repos]).to eq('bacon,bits')
+          expect(options[:repos]).to eq("bacon,bits")
         end
 
-        it 'accepts a single repository' do
-          options = { repos: 'bacon' }
+        it "accepts a single repository" do
+          options = { repos: "bacon" }
           described_class.format_repos!(options)
-          expect(options[:repos]).to eq('bacon')
+          expect(options[:repos]).to eq("bacon")
         end
       end
 
-      context 'when the :repos key is not present' do
-        it 'does not modify the hash' do
+      context "when the :repos key is not present" do
+        it "does not modify the hash" do
           options = {}
           described_class.format_repos!(options)
           expect(options).to eq(options)
         end
       end
 
-      context 'when the :repos key is empty' do
-        it 'does not modify the hash' do
+      context "when the :repos key is empty" do
+        it "does not modify the hash" do
           options = { repos: [] }
           described_class.format_repos!(options)
           expect(options).to eq(options)
@@ -84,14 +84,14 @@ module Artifactory
       end
     end
 
-    describe '.from_url' do
-      let(:relative_path) { '/api/storage/omnibus-unstable-local/com/getchef/harmony/0.1.0+20151111083608.git.15.8736e1e/el/5/harmony-0.1.0+20151111083608.git.15.8736e1e-1.el5.x86_64.rpm' }
+    describe ".from_url" do
+      let(:relative_path) { "/api/storage/omnibus-unstable-local/com/getchef/harmony/0.1.0+20151111083608.git.15.8736e1e/el/5/harmony-0.1.0+20151111083608.git.15.8736e1e-1.el5.x86_64.rpm" }
 
-      context 'when endpoint path part is not empty' do
+      context "when endpoint path part is not empty" do
         let(:endpoint) { "#{endpoint_host}/artifactory" }
         let(:full_url) { "#{endpoint}#{relative_path}" }
 
-        it 'uses the path minus the path part of the endpoint' do
+        it "uses the path minus the path part of the endpoint" do
           expect(client).to receive(:endpoint).and_return(endpoint)
           expect(described_class).to receive(:from_hash)
           expect(client).to receive(:get).with(relative_path)
@@ -99,11 +99,11 @@ module Artifactory
         end
       end
 
-      context 'when endpoint has empty path part' do
+      context "when endpoint has empty path part" do
         let(:endpoint) { "#{endpoint_host}/" }
         let(:full_url) { "#{endpoint}#{relative_path}" }
 
-        it 'only uses the path from absolute URLs' do
+        it "only uses the path from absolute URLs" do
           expect(client).to receive(:endpoint).and_return(endpoint)
           expect(described_class).to receive(:from_hash)
           expect(client).to receive(:get).with(relative_path)
@@ -112,26 +112,26 @@ module Artifactory
       end
     end
 
-    describe '.url_safe' do
-      let(:string) { double(to_s: 'string') }
+    describe ".url_safe" do
+      let(:string) { double(to_s: "string") }
 
-      it 'delegates to URI.escape' do
+      it "delegates to URI.escape" do
         expect(URI).to receive(:escape).once
         described_class.url_safe(string)
       end
 
-      it 'converts the value to a string' do
+      it "converts the value to a string" do
         expect(string).to receive(:to_s).once
         described_class.url_safe(string)
       end
     end
 
-    describe '#client' do
-      it 'defines a :client method' do
+    describe "#client" do
+      it "defines a :client method" do
         expect(subject).to respond_to(:client)
       end
 
-      it 'defaults to the Artifactory.client' do
+      it "defaults to the Artifactory.client" do
         client = double
         allow(Artifactory).to receive(:client).and_return(client)
         allow(client).to receive(:endpoint).and_return(endpoint)
@@ -140,40 +140,40 @@ module Artifactory
       end
     end
 
-    describe '#extract_client!' do
-      it 'delegates to the class method' do
+    describe "#extract_client!" do
+      it "delegates to the class method" do
         expect(described_class).to receive(:extract_client!).once
         subject.extract_client!({})
       end
     end
 
-    describe '#format_repos!' do
-      it 'delegates to the class method' do
+    describe "#format_repos!" do
+      it "delegates to the class method" do
         expect(described_class).to receive(:format_repos!).once
         subject.format_repos!({})
       end
     end
 
-    describe '#url_safe' do
-      it 'delegates to the class method' do
+    describe "#url_safe" do
+      it "delegates to the class method" do
         expect(described_class).to receive(:url_safe).once
-        subject.url_safe('string')
+        subject.url_safe("string")
       end
     end
 
-    describe '#to_s' do
-      it 'returns the name of the class' do
-        expect(subject.to_s).to eq('#<Base>')
+    describe "#to_s" do
+      it "returns the name of the class" do
+        expect(subject.to_s).to eq("#<Base>")
       end
     end
 
-    describe '#inspect' do
-      it 'includes all the attributes' do
+    describe "#inspect" do
+      it "includes all the attributes" do
         allow(subject).to receive(:attributes) do
-          { foo: 'bar' }
+          { foo: "bar" }
         end
 
-        expect(subject.inspect).to eq(%q|#<Base foo: "bar">|)
+        expect(subject.inspect).to eq(%q{#<Base foo: "bar">})
       end
     end
   end

data/spec/unit/resources/build_component_spec.rb

@@ -1,4 +1,4 @@
-require 'spec_helper'
+require "spec_helper"
 
 module Artifactory
   describe Resource::BuildComponent do
@@ -7,111 +7,111 @@ module Artifactory
     before(:each) do
       allow(Artifactory).to receive(:client).and_return(client)
       allow(client).to receive(:get).and_return(response) if defined?(response)
-      subject.name = 'wicket'
+      subject.name = "wicket"
     end
 
-    describe '.all' do
+    describe ".all" do
       let(:response) do
         {
-          'uri'    => "#{Artifactory.endpoint}/api/build",
-          'builds' => [
+          "uri"    => "#{Artifactory.endpoint}/api/build",
+          "builds" => [
             {
-              'uri'         => "/wicket",
-              'lastStarted' => '2015-06-19T20:13:20.222Z',
+              "uri"         => "/wicket",
+              "lastStarted" => "2015-06-19T20:13:20.222Z",
             },
             {
-              'uri'         => "/jackrabbit",
-              'lastStarted' => '2015-06-20T20:13:20.333Z',
+              "uri"         => "/jackrabbit",
+              "lastStarted" => "2015-06-20T20:13:20.333Z",
             },
-          ]
+          ],
         }
       end
 
       before do
-        allow(described_class).to receive(:from_hash).with(hash_including('uri' => '/wicket'), client: client).and_return('wicket')
-        allow(described_class).to receive(:from_hash).with(hash_including('uri' => '/jackrabbit'), client: client).and_return('jackrabbit')
+        allow(described_class).to receive(:from_hash).with(hash_including("uri" => "/wicket"), client: client).and_return("wicket")
+        allow(described_class).to receive(:from_hash).with(hash_including("uri" => "/jackrabbit"), client: client).and_return("jackrabbit")
       end
 
-      it 'GETS /api/build' do
-        expect(client).to receive(:get).with('/api/build').once
+      it "GETS /api/build" do
+        expect(client).to receive(:get).with("/api/build").once
         described_class.all
       end
 
-      context 'when there are components with builds' do
-        it 'returns the components' do
-          expect(described_class.all).to eq(%w( wicket jackrabbit ))
+      context "when there are components with builds" do
+        it "returns the components" do
+          expect(described_class.all).to eq(%w{ wicket jackrabbit })
         end
       end
 
-      context 'when the system has no components with builds' do
-        it 'returns an empty array' do
-          allow(client).to receive(:get).and_raise(Error::HTTPError.new('status' => 404))
+      context "when the system has no components with builds" do
+        it "returns an empty array" do
+          allow(client).to receive(:get).and_raise(Error::HTTPError.new("status" => 404))
           expect(described_class.all).to be_empty
         end
       end
     end
 
-    describe '.find' do
+    describe ".find" do
       before do
         allow(described_class).to receive(:all).and_return([
-          described_class.new(name: 'wicket'),
-          described_class.new(name: 'jackrabbit')
+          described_class.new(name: "wicket"),
+          described_class.new(name: "jackrabbit"),
         ])
       end
 
-      it 'filters the full build component list by name' do
-        expect(described_class.find('wicket')).to_not be_nil
+      it "filters the full build component list by name" do
+        expect(described_class.find("wicket")).to_not be_nil
       end
 
-      context 'when the build component does not exist' do
-        it 'returns nil' do
-          expect(described_class.find('fricket')).to be_nil
+      context "when the build component does not exist" do
+        it "returns nil" do
+          expect(described_class.find("fricket")).to be_nil
         end
       end
     end
 
-    describe '.from_hash' do
+    describe ".from_hash" do
       let(:time) { Time.now.utc.round }
       let(:hash) do
         {
-          'uri'         => '/wicket',
-          'lastStarted' => time.iso8601(3),
+          "uri"         => "/wicket",
+          "lastStarted" => time.iso8601(3),
         }
       end
 
-      it 'creates a new instance' do
+      it "creates a new instance" do
         instance = described_class.from_hash(hash)
-        expect(instance.uri).to eq('/wicket')
-        expect(instance.name).to eq('wicket')
+        expect(instance.uri).to eq("/wicket")
+        expect(instance.name).to eq("wicket")
         expect(instance.last_started).to eq(time)
       end
     end
 
-    describe '#builds' do
+    describe "#builds" do
 
-      it 'returns a build collection' do
+      it "returns a build collection" do
         expect(subject.builds).to be_a(Collection::Build)
       end
     end
 
-    describe '#delete' do
+    describe "#delete" do
 
-      it 'sends DELETE to the client' do
+      it "sends DELETE to the client" do
         expect(client).to receive(:delete)
         subject.delete
       end
 
-      it 'adds the correct parameters to the request' do
-        expect(client).to receive(:delete).with('/api/build/wicket?buildNumbers=51,52,55&artifacts=1&deleteAll=1', {})
-        subject.delete(build_numbers: %w( 51 52 55 ), artifacts: true, delete_all: true)
+      it "adds the correct parameters to the request" do
+        expect(client).to receive(:delete).with("/api/build/wicket?buildNumbers=51,52,55&artifacts=1&deleteAll=1", {})
+        subject.delete(build_numbers: %w{ 51 52 55 }, artifacts: true, delete_all: true)
       end
     end
 
-    describe '#rename' do
+    describe "#rename" do
 
-      it 'sends POST to the client with the correct param' do
-        expect(client).to receive(:post).with('/api/build/rename/wicket?to=fricket', {})
-        subject.rename('fricket')
+      it "sends POST to the client with the correct param" do
+        expect(client).to receive(:post).with("/api/build/rename/wicket?to=fricket", {})
+        subject.rename("fricket")
       end
     end
   end

data/spec/unit/resources/build_spec.rb

@@ -1,4 +1,4 @@
-require 'spec_helper'
+require "spec_helper"
 
 module Artifactory
   describe Resource::Build do
@@ -9,68 +9,68 @@ module Artifactory
       allow(Artifactory).to receive(:client).and_return(client)
       allow(client).to receive(:get).and_return(response) if defined?(response)
 
-      subject.name   = 'wicket'
-      subject.number = '51'
+      subject.name   = "wicket"
+      subject.number = "51"
     end
 
-    describe '.all' do
+    describe ".all" do
       let(:response) do
         {
-          'uri'    => "#{Artifactory.endpoint}/api/build/wicket",
-          'buildsNumbers' => [
+          "uri" => "#{Artifactory.endpoint}/api/build/wicket",
+          "buildsNumbers" => [
             {
-              'uri'     => "/51",
-              'started' => '2014-01-01 12:00:00',
+              "uri"     => "/51",
+              "started" => "2014-01-01 12:00:00",
             },
             {
-              'uri'     => "/52",
-              'started' => '2015-01-05 12:00:00',
+              "uri"     => "/52",
+              "started" => "2015-01-05 12:00:00",
             },
-          ]
+          ],
         }
       end
 
       before do
-        allow(described_class).to receive(:find).with('wicket', '51', client: client).and_return('51')
-        allow(described_class).to receive(:find).with('wicket', '52', client: client).and_return('52')
+        allow(described_class).to receive(:find).with("wicket", "51", client: client).and_return("51")
+        allow(described_class).to receive(:find).with("wicket", "52", client: client).and_return("52")
       end
 
       it 'GETS /api/build/#{name}' do
-        expect(client).to receive(:get).with('/api/build/wicket').once
-        described_class.all('wicket')
+        expect(client).to receive(:get).with("/api/build/wicket").once
+        described_class.all("wicket")
       end
 
-      context 'when there are builds' do
-        it 'returns the builds' do
-          expect(described_class.all('wicket')).to eq(['51', '52'])
+      context "when there are builds" do
+        it "returns the builds" do
+          expect(described_class.all("wicket")).to eq(%w{51 52})
         end
       end
 
-      context 'when the system has no builds' do
-        it 'returns an empty array' do
-          allow(client).to receive(:get).and_raise(Error::HTTPError.new('status' => 404))
-          expect(described_class.all('wicket')).to be_empty
+      context "when the system has no builds" do
+        it "returns an empty array" do
+          allow(client).to receive(:get).and_raise(Error::HTTPError.new("status" => 404))
+          expect(described_class.all("wicket")).to be_empty
         end
       end
     end
 
-    describe '.find' do
-      let(:response) { {'buildInfo' => {}} }
+    describe ".find" do
+      let(:response) { { "buildInfo" => {} } }
 
       it 'GETS /api/repositories/#{name}/#{number}' do
         expect(client).to receive(:get).with("/api/build/wicket/51").once
-        described_class.find('wicket', 51)
+        described_class.find("wicket", 51)
       end
 
-      context 'when the build does not exist' do
-        it 'returns nil' do
-          allow(client).to receive(:get).and_raise(Error::HTTPError.new('status' => 404))
-          expect(described_class.find('wicket', '4445')).to be_nil
+      context "when the build does not exist" do
+        it "returns nil" do
+          allow(client).to receive(:get).and_raise(Error::HTTPError.new("status" => 404))
+          expect(described_class.find("wicket", "4445")).to be_nil
         end
       end
     end
 
-    describe '.from_hash' do
+    describe ".from_hash" do
       let(:hash) do
         {
           "properties" => {
@@ -82,11 +82,11 @@ module Artifactory
           "type" => "MAVEN",
           "buildAgent" => {
             "name" => "Maven",
-            "version" => "3.0.5"
+            "version" => "3.0.5",
           },
           "agent" => {
             "name" => "Jenkins",
-            "version" => "1.565.2"
+            "version" => "1.565.2",
           },
           "started" => time.iso8601(3),
           "durationMillis" => 8926,
@@ -99,12 +99,12 @@ module Artifactory
             "includePublishedArtifacts" => false,
             "autoDiscover" => true,
             "scopesList" => "",
-            "licenseViolationsRecipientsList" => ""
+            "licenseViolationsRecipientsList" => "",
           },
           "buildRetention" => {
             "count" => -1,
             "deleteBuildArtifacts" => true,
-            "buildNumbersNotToBeDiscarded" => []
+            "buildNumbersNotToBeDiscarded" => [],
           },
           "modules" => modules,
           "governance" => {
@@ -112,16 +112,16 @@ module Artifactory
               "runChecks" => false,
               "includePublishedArtifacts" => false,
               "autoCreateMissingComponentRequests" => false,
-              "autoDiscardStaleComponentRequests" => false
-            }
-          }
+              "autoDiscardStaleComponentRequests" => false,
+            },
+          },
         }
       end
       let(:modules) do
         [
           {
             "properties" => {
-              "project.build.sourceEncoding" => "UTF-8"
+              "project.build.sourceEncoding" => "UTF-8",
             },
             "id" => "org.jfrog.test:multi:2.19-SNAPSHOT",
             "artifacts" => [
@@ -129,107 +129,107 @@ module Artifactory
                 "type" => "pom",
                 "sha1" => "045b66ebbf8504002b626f592d087612aca36582",
                 "md5" => "c25542a353dab1089cd186465dc47a68",
-                "name" => "multi-2.19-SNAPSHOT.pom"
-              }
-            ]
+                "name" => "multi-2.19-SNAPSHOT.pom",
+              },
+            ],
           },
           {
             "properties" => {
-              "project.build.sourceEncoding" => "UTF-8"
+              "project.build.sourceEncoding" => "UTF-8",
             },
             "id" => "org.jfrog.test:multi1:2.19-SNAPSHOT",
-            "artifacts"=> [
+            "artifacts" => [
               {
                 "type" => "jar",
                 "sha1" => "f4c5c9cb3091011ec2a895b3dedd7f10d847361c",
                 "md5" => "d1fd850a3582efba41092c624e0b46b8",
-                "name" => "multi1-2.19-SNAPSHOT.jar"
+                "name" => "multi1-2.19-SNAPSHOT.jar",
               },
               {
                 "type" => "pom",
                 "sha1" => "2ddbf9824676f548d637726d3bcbb494ba823090",
                 "md5" => "a64aa7f305f63a85e63a0155ff0fb404",
-                "name" => "multi1-2.19-SNAPSHOT.pom"
+                "name" => "multi1-2.19-SNAPSHOT.pom",
               },
               {
                 "type" => "jar",
                 "sha1" => "6fdd143a44cea3a2636660c5c266c95c27e50abc",
                 "md5" => "12a1e438f4bef8c4b740fe848a1704a4",
                 "id" => "org.slf4j:slf4j-simple:1.4.3",
-                "scopes" => [ "compile" ]
+                "scopes" => [ "compile" ],
               },
               {
                 "type" => "jar",
                 "sha1" => "496e91f7df8a0417e00cecdba840cdf0e5f2472c",
                 "md5" => "76a412a37c9d18659d2dacccdb1c24ff",
                 "id" => "org.jenkins-ci.lib:dry-run-lib:0.1",
-                "scopes" => [ "compile" ]
-              }
-            ]
+                "scopes" => [ "compile" ],
+              },
+            ],
           },
           {
             "properties" => {
               "daversion" => "2.19-SNAPSHOT",
-              "project.build.sourceEncoding"=>"UTF-8"
+              "project.build.sourceEncoding" => "UTF-8",
             },
             "id" => "org.jfrog.test:multi2:2.19-SNAPSHOT",
             "artifacts" => [
               {
                 "type" => "txt",
-                "name" => "multi2-2.19-SNAPSHOT.txt"
+                "name" => "multi2-2.19-SNAPSHOT.txt",
               },
               {
                 "type" => "java-source-jar",
                 "sha1" => "49108b0c7db5fdb4efe3c29a5a9f54e806aecb62",
                 "md5" => "0e2c5473cf2a9b694afb4a2e8da34b53",
-                "name" => "multi2-2.19-SNAPSHOT-sources.jar"},
+                "name" => "multi2-2.19-SNAPSHOT-sources.jar" },
              {
                 "type" => "jar",
                 "sha1" => "476e89d290ae36dabb38ff22f75f264ae019d542",
                 "md5" => "fa9b3df58ac040fffcff9310f261be80",
-                "name" => "multi2-2.19-SNAPSHOT.jar"
+                "name" => "multi2-2.19-SNAPSHOT.jar",
              },
              {
                 "type" => "pom",
                 "sha1" => "b719b90364e5ae38cda358072f61f821bdae5d5d",
                 "md5" => "8d5060005235d75907baca4490cf60bf",
-                "name" => "multi2-2.19-SNAPSHOT.pom"
-              }
+                "name" => "multi2-2.19-SNAPSHOT.pom",
+              },
             ],
-            "dependencies"=> [
+            "dependencies" => [
               {
                 "type" => "jar",
                 "sha1" => "19d4e90b43059058f6e056f794f0ea4030d60b86",
                 "md5" => "dcd95bcb84b09897b2b66d4684c040da",
                 "id" => "xpp3:xpp3_min:1.1.4c",
-                "scopes" => [ "provided" ]
+                "scopes" => [ "provided" ],
               },
               {
                 "type" => "jar",
                 "sha1" => "e2d866af5518e81282838301b49a1bd2452619d3",
                 "md5" => "e9e4b59c69305ba3698dd61c5dfc4fc8",
                 "id" => "org.jvnet.hudson.plugins:perforce:1.3.7",
-                "scopes" => [ "compile" ]
+                "scopes" => [ "compile" ],
               },
               {
                 "type" => "jar",
                 "sha1" => "6fdd143a44cea3a2636660c5c266c95c27e50abc",
                 "md5" => "12a1e438f4bef8c4b740fe848a1704a4",
                 "id" => "org.slf4j:slf4j-simple:1.4.3",
-                "scopes" => [ "compile" ]
+                "scopes" => [ "compile" ],
               },
               {
                 "type" => "jar",
                 "sha1" => "496e91f7df8a0417e00cecdba840cdf0e5f2472c",
                 "md5" => "76a412a37c9d18659d2dacccdb1c24ff",
                 "id" => "org.jenkins-ci.lib:dry-run-lib:0.1",
-                "scopes" => [ "compile" ]
-              }
-            ]
+                "scopes" => [ "compile" ],
+              },
+            ],
           },
           {
             "properties" => {
-              "project.build.sourceEncoding" => "UTF-8"
+              "project.build.sourceEncoding" => "UTF-8",
             },
             "id" => "org.jfrog.test:multi3:2.19-SNAPSHOT",
             "artifacts" => [
@@ -237,43 +237,43 @@ module Artifactory
                 "type" => "java-source-jar",
                 "sha1" => "3cd104785167ac37ef999431f308ffef10810348",
                 "md5" => "c683276f8dda97078ae8eb5e26bb3ee5",
-                "name" => "multi3-2.19-SNAPSHOT-sources.jar"
+                "name" => "multi3-2.19-SNAPSHOT-sources.jar",
               },
               {
                 "type" => "war",
                 "sha1" => "34aeebeb805b23922d9d05507404533518cf81e4",
                 "md5" => "55af06a2175cfb23cc6dc3931475b57c",
-                "name" => "multi3-2.19-SNAPSHOT.war"
+                "name" => "multi3-2.19-SNAPSHOT.war",
               },
               {
                 "type" => "jar",
                 "sha1" => "496e91f7df8a0417e00cecdba840cdf0e5f2472c",
                 "md5" => "76a412a37c9d18659d2dacccdb1c24ff",
                 "id" => "org.jenkins-ci.lib:dry-run-lib:0.1",
-                "scopes" => [ "compile" ]
+                "scopes" => [ "compile" ],
               },
               {
                 "type" => "jar",
                 "sha1" => "7e9978fdb754bce5fcd5161133e7734ecb683036",
                 "md5" => "7df83e09e41d742cc5fb20d16b80729c",
                 "id" => "hsqldb:hsqldb:1.8.0.10",
-                "scopes" => [ "runtime" ]
-              }
-            ]
-          }
+                "scopes" => [ "runtime" ],
+              },
+            ],
+          },
         ]
       end
 
-      it 'creates a new instance' do
+      it "creates a new instance" do
         instance = described_class.from_hash(hash)
         expect(instance.properties).to eq("buildInfo.env.JAVA_HOME" => "/usr/jdk/latest")
-        expect(instance.version).to eq('1.0.1')
-        expect(instance.name).to eq('Maven2-3')
-        expect(instance.number).to eq('9')
-        expect(instance.type).to eq('MAVEN')
+        expect(instance.version).to eq("1.0.1")
+        expect(instance.name).to eq("Maven2-3")
+        expect(instance.number).to eq("9")
+        expect(instance.type).to eq("MAVEN")
         expect(instance.build_agent).to eq(
           "name" => "Maven",
-          "version" => "3.0.5",
+          "version" => "3.0.5"
         )
         expect(instance.agent).to eq(
           "name" => "Jenkins",
@@ -281,10 +281,10 @@ module Artifactory
         )
         expect(instance.started).to eq(time)
         expect(instance.duration_millis).to eq(8926)
-        expect(instance.artifactory_principal).to eq('admin')
-        expect(instance.url).to eq('http://localhost:8080/job/Maven2-3/9/')
-        expect(instance.vcs_revision).to eq('83049487ecc61bef3dce798838e7a9457e174a5a')
-        expect(instance.vcs_url).to eq('https://github.com/aseftel/project-examples')
+        expect(instance.artifactory_principal).to eq("admin")
+        expect(instance.url).to eq("http://localhost:8080/job/Maven2-3/9/")
+        expect(instance.vcs_revision).to eq("83049487ecc61bef3dce798838e7a9457e174a5a")
+        expect(instance.vcs_url).to eq("https://github.com/aseftel/project-examples")
         expect(instance.build_retention).to eq(
           "count" => -1,
           "deleteBuildArtifacts" => true,
@@ -298,47 +298,47 @@ module Artifactory
             "runChecks" => false,
             "includePublishedArtifacts" => false,
             "autoCreateMissingComponentRequests" => false,
-            "autoDiscardStaleComponentRequests" => false
+            "autoDiscardStaleComponentRequests" => false,
           }
         )
       end
     end
 
-    describe '#diff' do
-      it 'sends GET to the client with the correct param' do
-        expect(client).to receive(:get).with('/api/build/wicket/51?diff=53', {})
-        subject.diff('53')
+    describe "#diff" do
+      it "sends GET to the client with the correct param" do
+        expect(client).to receive(:get).with("/api/build/wicket/51?diff=53", {})
+        subject.diff("53")
       end
     end
 
-    describe '#promote' do
-      let(:to_repo) { 'omnibus-stable-local' }
+    describe "#promote" do
+      let(:to_repo) { "omnibus-stable-local" }
       before do
-        Artifactory.username = 'doge'
+        Artifactory.username = "doge"
       end
 
-      it 'sends POST to the client with promotion JSON' do
-        expect(client).to receive(:post).with('/api/build/promote/wicket/51', /"targetRepo"\:"omnibus-stable-local"/, kind_of(Hash))
+      it "sends POST to the client with promotion JSON" do
+        expect(client).to receive(:post).with("/api/build/promote/wicket/51", /"targetRepo"\:"omnibus-stable-local"/, kind_of(Hash))
         subject.promote(to_repo)
       end
 
-      it 'Uses the configured Artifactory.username as the default value for `ciUser`' do
-        expect(client).to receive(:post).with('/api/build/promote/wicket/51', /"ciUser"\:"doge"/, kind_of(Hash))
+      it "Uses the configured Artifactory.username as the default value for `ciUser`" do
+        expect(client).to receive(:post).with("/api/build/promote/wicket/51", /"ciUser"\:"doge"/, kind_of(Hash))
         subject.promote(to_repo)
       end
     end
 
-    describe '#save' do
+    describe "#save" do
 
-      it 'PUTS the build JSON file to the server' do
-        expect(client).to receive(:put).with('/api/build', kind_of(Tempfile), 'Content-Type' => 'application/json')
+      it "PUTS the build JSON file to the server" do
+        expect(client).to receive(:put).with("/api/build", kind_of(Tempfile), "Content-Type" => "application/json")
         subject.save
       end
 
-      context 'an invalid build type is provided' do
-        before { subject.type = 'PIRATE' }
+      context "an invalid build type is provided" do
+        before { subject.type = "PIRATE" }
 
-        it 'raises an InvalidBuildType error' do
+        it "raises an InvalidBuildType error" do
           expect { subject.save }.to raise_error(Error::InvalidBuildType)
         end
       end