chef-dk 1.2.22 → 1.3.40

data/spec/unit/policyfile/null_cookbook_source_spec.rb

@@ -15,9 +15,9 @@
 # limitations under the License.
 #
 
-require 'spec_helper'
+require "spec_helper"
 
-require 'chef-dk/policyfile/null_cookbook_source'
+require "chef-dk/policyfile/null_cookbook_source"
 
 describe ChefDK::Policyfile::NullCookbookSource do
 
@@ -32,4 +32,3 @@ describe ChefDK::Policyfile::NullCookbookSource do
   end
 
 end
-

data/spec/unit/policyfile/read_cookbook_for_compat_mode_upload_spec.rb

@@ -15,9 +15,9 @@
 # limitations under the License.
 #
 
-require 'spec_helper'
-require 'chef-dk/policyfile/read_cookbook_for_compat_mode_upload'
-require 'chef-dk/helpers'
+require "spec_helper"
+require "chef-dk/policyfile/read_cookbook_for_compat_mode_upload"
+require "chef-dk/helpers"
 
 describe ChefDK::Policyfile::ReadCookbookForCompatModeUpload do
 

data/spec/unit/policyfile/reports/install_spec.rb

@@ -15,12 +15,12 @@
 # limitations under the License.
 #
 
-require 'spec_helper'
-require 'chef-dk/policyfile/reports/install'
+require "spec_helper"
+require "chef-dk/policyfile/reports/install"
 
 # Used for verifying doubles
-require 'chef-dk/policyfile_compiler'
-require 'chef-dk/policyfile/cookbook_location_specification'
+require "chef-dk/policyfile_compiler"
+require "chef-dk/policyfile/cookbook_location_specification"
 
 describe ChefDK::Policyfile::Reports::Install do
 
@@ -112,4 +112,3 @@ describe ChefDK::Policyfile::Reports::Install do
 
   end
 end
-

data/spec/unit/policyfile/reports/upload_spec.rb

@@ -15,14 +15,14 @@
 # limitations under the License.
 #
 
-require 'spec_helper'
-require 'chef-dk/policyfile/reports/upload'
+require "spec_helper"
+require "chef-dk/policyfile/reports/upload"
 
 # For the LockedCookbookForUpload class:
-require 'chef-dk/policyfile/uploader'
+require "chef-dk/policyfile/uploader"
 
 # Used for verifying doubles
-require 'chef-dk/policyfile/cookbook_locks'
+require "chef-dk/policyfile/cookbook_locks"
 
 describe ChefDK::Policyfile::Reports::Upload do
 
@@ -92,5 +92,3 @@ E
   end
 
 end
-
-

data/spec/unit/policyfile/solution_dependencies_spec.rb

@@ -15,12 +15,12 @@
 # limitations under the License.
 #
 
-require 'spec_helper'
-require 'chef-dk/policyfile/solution_dependencies'
+require "spec_helper"
+require "chef-dk/policyfile/solution_dependencies"
 
 describe ChefDK::Policyfile::SolutionDependencies do
 
-  let(:dependency_data) { {"Policyfile" => [], "dependencies" => {}} }
+  let(:dependency_data) { { "Policyfile" => [], "dependencies" => {} } }
 
   let(:solution_dependencies) do
     s = described_class.new
@@ -47,8 +47,8 @@ describe ChefDK::Policyfile::SolutionDependencies do
           "nginx (1.2.3)" => [ ["apt", "~> 2.3"], ["yum", "~>3.4"] ],
           "apt (2.5.6)" => [],
           "yum (3.4.1)" => [],
-          "postgresql (5.0.0)" => []
-        }
+          "postgresql (5.0.0)" => [],
+        },
       }
     end
 
@@ -62,7 +62,7 @@ describe ChefDK::Policyfile::SolutionDependencies do
         "nginx (1.2.3)" => [ ["apt", "~> 2.3"], ["yum", "~> 3.4"] ],
         "apt (2.5.6)" => [],
         "yum (3.4.1)" => [],
-        "postgresql (5.0.0)" => []
+        "postgresql (5.0.0)" => [],
       }
       expect(solution_dependencies.cookbook_deps_for_lock).to eq(expected)
     end
@@ -76,7 +76,7 @@ describe ChefDK::Policyfile::SolutionDependencies do
         "nginx (1.2.3)" => [ ["apt", "~> 2.3"], ["yum", "~> 3.4"] ],
         "apt (2.5.6)" => [],
         "yum (3.4.1)" => [],
-        "postgresql (5.0.0)" => []
+        "postgresql (5.0.0)" => [],
       }
     end
 
@@ -102,7 +102,7 @@ describe ChefDK::Policyfile::SolutionDependencies do
     end
 
     it "generates lock info containing both policyfile and cookbook dependencies" do
-      expected = {"Policyfile" => expected_policyfile_deps_for_lock, "dependencies" => expected_deps_for_lock}
+      expected = { "Policyfile" => expected_policyfile_deps_for_lock, "dependencies" => expected_deps_for_lock }
       expect(solution_dependencies.to_lock).to eq(expected)
     end
 
@@ -110,33 +110,33 @@ describe ChefDK::Policyfile::SolutionDependencies do
 
       it "does not raise if a cookbook that's in the dependency set with a different version doesn't conflict" do
         solution_dependencies.update_cookbook_dep("yum", "3.5.0", [ ])
-        expect(solution_dependencies.test_conflict!('yum', '3.5.0')).to be(false)
+        expect(solution_dependencies.test_conflict!("yum", "3.5.0")).to be(false)
       end
 
       it "raises if a cookbook is not in the current solution set" do
         expected_message = "Cookbook foo (1.0.0) not in the working set, cannot test for conflicts"
-        expect { solution_dependencies.test_conflict!('foo', '1.0.0') }.to raise_error(ChefDK::CookbookNotInWorkingSet, expected_message)
+        expect { solution_dependencies.test_conflict!("foo", "1.0.0") }.to raise_error(ChefDK::CookbookNotInWorkingSet, expected_message)
       end
 
       it "raises when a cookbook conflicts with a Policyfile constraint" do
         solution_dependencies.update_cookbook_dep("nginx", "2.0.0", [])
 
         expected_message = "Cookbook nginx (2.0.0) conflicts with other dependencies:\nPolicyfile depends on nginx ~> 1.0"
-        expect { solution_dependencies.test_conflict!('nginx', '2.0.0') }.to raise_error(ChefDK::DependencyConflict, expected_message)
+        expect { solution_dependencies.test_conflict!("nginx", "2.0.0") }.to raise_error(ChefDK::DependencyConflict, expected_message)
       end
 
       it "raises when a cookbook conflicts with another cookbook's dependency constraint" do
         solution_dependencies.update_cookbook_dep("apt", "3.0.0", [])
 
         expected_message = "Cookbook apt (3.0.0) conflicts with other dependencies:\nnginx (1.2.3) depends on apt ~> 2.3"
-        expect { solution_dependencies.test_conflict!('apt', '3.0.0') }.to raise_error(ChefDK::DependencyConflict, expected_message)
+        expect { solution_dependencies.test_conflict!("apt", "3.0.0") }.to raise_error(ChefDK::DependencyConflict, expected_message)
       end
 
       it "raises when a cookbook's dependencies are no longer satisfiable" do
         solution_dependencies.update_cookbook_dep("nginx", "1.2.3", [ [ "apt", "~> 3.0" ] ])
         expected_message = "Cookbook nginx (1.2.3) has dependency constraints that cannot be met by the existing cookbook set:\n" +
           "Dependency on apt ~> 3.0 conflicts with existing version apt (2.5.6)"
-        expect { solution_dependencies.test_conflict!('nginx', '1.2.3') }.to raise_error(ChefDK::DependencyConflict, expected_message)
+        expect { solution_dependencies.test_conflict!("nginx", "1.2.3") }.to raise_error(ChefDK::DependencyConflict, expected_message)
       end
 
     end

data/spec/unit/policyfile/source_uri_spec.rb

@@ -15,18 +15,18 @@
 # limitations under the License.
 #
 
-require 'spec_helper'
+require "spec_helper"
 
-require 'chef-dk/policyfile/source_uri'
+require "chef-dk/policyfile/source_uri"
 
 describe ChefDK::Policyfile::SourceURI do
   subject { described_class.parse(source_uri) }
 
   describe '#validate' do
-    context 'when the scheme is not https' do
-      let(:source_uri) { 'ftp://chef.example.com' }
+    context "when the scheme is not https" do
+      let(:source_uri) { "ftp://chef.example.com" }
 
-      it 'raises ChefDK::InvalidPolicyfileSourceURI' do
+      it "raises ChefDK::InvalidPolicyfileSourceURI" do
         expect do
           subject.validate
         end.to raise_error(ChefDK::InvalidPolicyfileSourceURI)

data/spec/unit/policyfile/storage_config_spec.rb

@@ -15,8 +15,8 @@
 # limitations under the License.
 #
 
-require 'spec_helper'
-require 'chef-dk/policyfile/storage_config'
+require "spec_helper"
+require "chef-dk/policyfile/storage_config"
 
 describe ChefDK::Policyfile::StorageConfig do
 
@@ -79,7 +79,7 @@ describe ChefDK::Policyfile::StorageConfig do
     end
 
     it "gives the expanded path to the policyfile" do
-      expect(storage_config.policyfile_expanded_path).to eq(File.expand_path('/path/to/Policyfile.rb'))
+      expect(storage_config.policyfile_expanded_path).to eq(File.expand_path("/path/to/Policyfile.rb"))
     end
 
     context "when the policyfile is given as a relative path" do
@@ -129,7 +129,6 @@ describe ChefDK::Policyfile::StorageConfig do
 
   end
 
-
   describe "updating storage config for policyfile lock location" do
 
     before do
@@ -174,11 +173,8 @@ describe ChefDK::Policyfile::StorageConfig do
         expect(storage_config.policyfile_lock_expanded_path).to eq(File.expand_path("Policyfile.lock.json", "."))
       end
 
-
     end
 
   end
 
-
 end
-

data/spec/unit/policyfile/undo_record_spec.rb

@@ -15,8 +15,8 @@
 # limitations under the License.
 #
 
-require 'spec_helper'
-require 'chef-dk/policyfile/undo_record'
+require "spec_helper"
+require "chef-dk/policyfile/undo_record"
 
 describe ChefDK::Policyfile::UndoRecord do
 
@@ -25,7 +25,7 @@ describe ChefDK::Policyfile::UndoRecord do
   let(:policy_revision) do
     {
       "name" => "appserver",
-      "revision_id" => "1111111111111111111111111111111111111111111111111111111111111111"
+      "revision_id" => "1111111111111111111111111111111111111111111111111111111111111111",
     }
   end
 
@@ -49,8 +49,8 @@ describe ChefDK::Policyfile::UndoRecord do
         "description" => "",
         "backup_data" => {
           "policy_groups" => [],
-          "policy_revisions" => []
-        }
+          "policy_revisions" => [],
+        },
       }
       expect(undo_record.for_serialization).to eq(expected)
     end
@@ -90,10 +90,10 @@ describe ChefDK::Policyfile::UndoRecord do
             {
               "policy_name" => "appserver",
               "policy_group" => "preprod",
-              "data" => policy_revision
-            }
-          ]
-        }
+              "data" => policy_revision,
+            },
+          ],
+        },
       }
       expect(undo_record.for_serialization).to eq(expected)
     end
@@ -130,7 +130,7 @@ describe ChefDK::Policyfile::UndoRecord do
           {
             "format_version" => 1,
             "description" => "delete-policy-group preprod",
-            "backup_data" => []
+            "backup_data" => [],
           }
         end
 
@@ -146,7 +146,7 @@ describe ChefDK::Policyfile::UndoRecord do
           {
             "format_version" => 1,
             "description" => "delete-policy-group preprod",
-            "backup_data" => {}
+            "backup_data" => {},
           }
         end
 
@@ -164,8 +164,8 @@ describe ChefDK::Policyfile::UndoRecord do
             "description" => "delete-policy-group preprod",
             "backup_data" => {
               "policy_groups" => nil,
-              "policy_revisions" => []
-            }
+              "policy_revisions" => [],
+            },
           }
         end
 
@@ -183,8 +183,8 @@ describe ChefDK::Policyfile::UndoRecord do
             "description" => "delete-policy-group preprod",
             "backup_data" => {
               "policy_groups" => [],
-              "policy_revisions" => nil
-            }
+              "policy_revisions" => nil,
+            },
           }
         end
 
@@ -202,8 +202,8 @@ describe ChefDK::Policyfile::UndoRecord do
             "description" => "delete-policy-group preprod",
             "backup_data" => {
               "policy_groups" => [],
-              "policy_revisions" => [ nil ]
-            }
+              "policy_revisions" => [ nil ],
+            },
           }
         end
 
@@ -225,10 +225,10 @@ describe ChefDK::Policyfile::UndoRecord do
               {
                 "policy_name" => "appserver",
                 "policy_group" => "preprod",
-                "data" => policy_revision
-              }
-            ]
-          }
+                "data" => policy_revision,
+              },
+            ],
+          },
         }
       end
 
@@ -252,9 +252,7 @@ describe ChefDK::Policyfile::UndoRecord do
         expect(undo_record.for_serialization).to eq(serialized_data)
       end
 
-
     end
   end
 
 end
-

data/spec/unit/policyfile/undo_stack_spec.rb

@@ -15,8 +15,8 @@
 # limitations under the License.
 #
 
-require 'spec_helper'
-require 'chef-dk/policyfile/undo_stack'
+require "spec_helper"
+require "chef-dk/policyfile/undo_stack"
 
 describe ChefDK::Policyfile::UndoStack do
 
@@ -25,7 +25,7 @@ describe ChefDK::Policyfile::UndoStack do
   let(:policy_revision) do
     {
       "name" => "appserver",
-      "revision_id" => "1111111111111111111111111111111111111111111111111111111111111111"
+      "revision_id" => "1111111111111111111111111111111111111111111111111111111111111111",
     }
   end
 
@@ -180,7 +180,6 @@ describe ChefDK::Policyfile::UndoStack do
 
   end
 
-
   context "when the stack is at the maximum configured size" do
     # `Time.new` is stubbed later on, need to force it to be evaluated before
     # then.
@@ -196,7 +195,7 @@ describe ChefDK::Policyfile::UndoStack do
     def incremented_undo_record(i)
       record = {
         "name" => "appserver",
-        "revision_id" => i.to_s * 64
+        "revision_id" => i.to_s * 64,
       }
 
       ChefDK::Policyfile::UndoRecord.new.tap do |undo_record|
@@ -241,7 +240,7 @@ describe ChefDK::Policyfile::UndoStack do
 
       before do
         FileUtils.touch(older_record_path)
-      expect(undo_stack_files.size).to eq(11)
+        expect(undo_stack_files.size).to eq(11)
       end
 
       describe "pushing a new undo record" do

data/spec/unit/policyfile/uploader_spec.rb

@@ -15,33 +15,33 @@
 # limitations under the License.
 #
 
-require 'spec_helper'
-require 'chef-dk/policyfile/uploader'
+require "spec_helper"
+require "chef-dk/policyfile/uploader"
 
 # We load this here to ensure we get the "verifying doubles" behavior from
 # RSpec. It's not used by Policyfile::Uploader, but it's a collaborator.
-require 'chef-dk/authenticated_http'
+require "chef-dk/authenticated_http"
 
 describe ChefDK::Policyfile::Uploader do
 
   let(:policyfile_lock_data) do
     {
-      "name"=> "example",
-      "run_list"=> [ "recipe[omnibus::default]" ],
-      "cookbook_locks"=> {
-        "omnibus"=> {
-          "version"=> "2.2.0",
-          "identifier"=> "64b3e64306cff223206348e46af545b19032b170",
-          "dotted_decimal_identifier"=> "28345299219435506.9887234981653237.76628930769264",
-          "cache_key"=> "omnibus-2cf98f9797cacce9c8688fc4e74858b858e2bc14",
-          "origin"=> "git@github.com:opscode-cookbooks/omnibus.git",
-          "source_options"=> {
-            "git"=> "git@github.com:opscode-cookbooks/omnibus.git",
-            "revision"=> "2cf98f9797cacce9c8688fc4e74858b858e2bc14",
-            "branch"=> "master"
-          }
-        }
-      }
+      "name" => "example",
+      "run_list" => [ "recipe[omnibus::default]" ],
+      "cookbook_locks" => {
+        "omnibus" => {
+          "version" => "2.2.0",
+          "identifier" => "64b3e64306cff223206348e46af545b19032b170",
+          "dotted_decimal_identifier" => "28345299219435506.9887234981653237.76628930769264",
+          "cache_key" => "omnibus-2cf98f9797cacce9c8688fc4e74858b858e2bc14",
+          "origin" => "git@github.com:opscode-cookbooks/omnibus.git",
+          "source_options" => {
+            "git" => "git@github.com:opscode-cookbooks/omnibus.git",
+            "revision" => "2cf98f9797cacce9c8688fc4e74858b858e2bc14",
+            "branch" => "master",
+          },
+        },
+      },
     }
   end
 
@@ -66,7 +66,7 @@ describe ChefDK::Policyfile::Uploader do
     policyfile_as_data_bag_item = {
       "id" => "example-unit-test",
       "name" => "data_bag_item_policyfiles_example-unit-test",
-      "data_bag" => "policyfiles"
+      "data_bag" => "policyfiles",
     }
     policyfile_as_data_bag_item["raw_data"] = policyfile_lock_data.dup
     policyfile_as_data_bag_item["raw_data"]["id"] = "example-unit-test"
@@ -210,7 +210,7 @@ describe ChefDK::Policyfile::Uploader do
           let(:expected_cookbooks_for_upload) do
             [
               cookbook_versions["my_apache2"],
-              cookbook_versions["my_jenkins"]
+              cookbook_versions["my_jenkins"],
             ]
           end
 
@@ -218,7 +218,6 @@ describe ChefDK::Policyfile::Uploader do
             expect(policyfile_lock).to receive(:validate_cookbooks!)
             expect(http_client).to receive(:get).with(list_cookbooks_url).and_return(existing_cookbook_on_remote)
 
-
             expect(uploader.cookbook_versions_to_upload).to eq(expected_cookbooks_for_upload)
           end
 
@@ -273,35 +272,35 @@ describe ChefDK::Policyfile::Uploader do
 
     context "when configured for policy document compat mode" do
 
-      let(:policyfiles_data_bag) { {"name" => "policyfiles" } }
+      let(:policyfiles_data_bag) { { "name" => "policyfiles" } }
 
-      let(:list_cookbooks_url) { 'cookbooks?num_versions=all' }
+      let(:list_cookbooks_url) { "cookbooks?num_versions=all" }
 
       let(:existing_cookbook_on_remote) do
-        {"apt"=>
-          {"url"=>"http://localhost:8889/cookbooks/apt",
-           "versions"=>
-            [{"url"=>
+        { "apt" =>
+          { "url" => "http://localhost:8889/cookbooks/apt",
+            "versions" =>
+            [{ "url" =>
                "http://localhost:8889/cookbooks/apt/46097674477573307.43471642740453733.243606720748315",
-              "version"=>"46097674477573307.43471642740453733.243606720748315"}]},
-         "build-essential"=>
-          {"url"=>"http://localhost:8889/cookbooks/build-essential",
-           "versions"=>
-            [{"url"=>
+               "version" => "46097674477573307.43471642740453733.243606720748315" }] },
+          "build-essential" =>
+          { "url" => "http://localhost:8889/cookbooks/build-essential",
+            "versions" =>
+            [{ "url" =>
                "http://localhost:8889/cookbooks/build-essential/67369247788170534.26353953100055918.55660493423796",
-              "version"=>"67369247788170534.26353953100055918.55660493423796"}]},
-         "java"=>
-          {"url"=>"http://localhost:8889/cookbooks/java",
-           "versions"=>
-            [{"url"=>
+               "version" => "67369247788170534.26353953100055918.55660493423796" }] },
+          "java" =>
+          { "url" => "http://localhost:8889/cookbooks/java",
+            "versions" =>
+            [{ "url" =>
                "http://localhost:8889/cookbooks/java/5664982062912610.52588194571203830.6215746262253",
-              "version"=>"5664982062912610.52588194571203830.6215746262253"}]},
-         "jenkins"=>
-          {"url"=>"http://localhost:8889/cookbooks/jenkins",
-           "versions"=>
-            [{"url"=>
+               "version" => "5664982062912610.52588194571203830.6215746262253" }] },
+          "jenkins" =>
+          { "url" => "http://localhost:8889/cookbooks/jenkins",
+            "versions" =>
+            [{ "url" =>
                "http://localhost:8889/cookbooks/jenkins/69194928762630300.30177357398946006.269829039948647",
-              "version"=>"69194928762630300.30177357398946006.269829039948647"}]}
+               "version" => "69194928762630300.30177357398946006.269829039948647" }] },
         }
       end
 
@@ -311,14 +310,14 @@ describe ChefDK::Policyfile::Uploader do
       end
 
       it "ensures a data bag named 'policyfiles' exists" do
-        expect(http_client).to receive(:post).with('data', policyfiles_data_bag)
+        expect(http_client).to receive(:post).with("data", policyfiles_data_bag)
         uploader.data_bag_create
       end
 
       it "does not error when the 'policyfiles' data bag exists" do
         response = double("Net::HTTP response", code: "409")
         error = Net::HTTPServerException.new("conflict", response)
-        expect(http_client).to receive(:post).with('data', {"name" => "policyfiles"}).and_raise(error)
+        expect(http_client).to receive(:post).with("data", { "name" => "policyfiles" }).and_raise(error)
         expect { uploader.data_bag_create }.to_not raise_error
       end
 
@@ -326,24 +325,24 @@ describe ChefDK::Policyfile::Uploader do
         response = double("Net::HTTP response", code: "404")
         error = Net::HTTPServerException.new("Not Found", response)
         expect(http_client).to receive(:put).
-          with('data/policyfiles/example-unit-test', policyfile_as_data_bag_item).
+          with("data/policyfiles/example-unit-test", policyfile_as_data_bag_item).
           and_raise(error)
         expect(http_client).to receive(:post).
-          with('data/policyfiles', policyfile_as_data_bag_item)
+          with("data/policyfiles", policyfile_as_data_bag_item)
 
         uploader.data_bag_item_create
       end
 
       it "replaces an existing policyfile on the server if it exists" do
         expect(http_client).to receive(:put).
-          with('data/policyfiles/example-unit-test', policyfile_as_data_bag_item)
+          with("data/policyfiles/example-unit-test", policyfile_as_data_bag_item)
         uploader.data_bag_item_create
       end
 
       it "creates the data bag and item to upload the policy" do
-        expect(http_client).to receive(:post).with('data', policyfiles_data_bag)
+        expect(http_client).to receive(:post).with("data", policyfiles_data_bag)
         expect(http_client).to receive(:put).
-          with('data/policyfiles/example-unit-test', policyfile_as_data_bag_item)
+          with("data/policyfiles/example-unit-test", policyfile_as_data_bag_item)
         uploader.upload_policy
       end
 
@@ -355,38 +354,38 @@ describe ChefDK::Policyfile::Uploader do
 
       let(:policy_document_native_api) { true }
 
-      let(:list_cookbooks_url) { 'cookbook_artifacts?num_versions=all' }
+      let(:list_cookbooks_url) { "cookbook_artifacts?num_versions=all" }
 
       let(:existing_cookbook_on_remote) do
-        {"apt"=>
-          {"url"=>"http://localhost:8889/cookbook_artifacts/apt",
-           "versions"=>
-            [{"url"=>
+        { "apt" =>
+          { "url" => "http://localhost:8889/cookbook_artifacts/apt",
+            "versions" =>
+            [{ "url" =>
                "http://localhost:8889/cookbook_artifacts/apt/5f7045a8aeaf6ccda3b3594258df9ee982b3a023",
-              "identifier"=>"5f7045a8aeaf6ccda3b3594258df9ee982b3a023"}]},
-         "build-essential"=>
-          {"url"=>"http://localhost:8889/cookbook_artifacts/build-essential",
-           "versions"=>
-            [{"url"=>
+               "identifier" => "5f7045a8aeaf6ccda3b3594258df9ee982b3a023" }] },
+          "build-essential" =>
+          { "url" => "http://localhost:8889/cookbook_artifacts/build-essential",
+            "versions" =>
+            [{ "url" =>
                "http://localhost:8889/cookbook_artifacts/build-essential/571d8ebd02b296fe90b2e4d68754af7e8e185f28",
-              "identifier"=>"571d8ebd02b296fe90b2e4d68754af7e8e185f28"}]},
-         "java"=>
-          {"url"=>"http://localhost:8889/cookbook_artifacts/java",
-           "versions"=>
-            [{"url"=>
+               "identifier" => "571d8ebd02b296fe90b2e4d68754af7e8e185f28" }] },
+          "java" =>
+          { "url" => "http://localhost:8889/cookbook_artifacts/java",
+            "versions" =>
+            [{ "url" =>
                "http://localhost:8889/cookbook_artifacts/java/9178a38ad3e3baa55b49c1b8d9f4bf6a43dbc358",
-              "identifier"=>"9178a38ad3e3baa55b49c1b8d9f4bf6a43dbc358"}]},
-         "jenkins"=>
-          {"url"=>"http://localhost:8889/cookbook_artifacts/jenkins",
-           "versions"=>
-            [{"url"=>
+               "identifier" => "9178a38ad3e3baa55b49c1b8d9f4bf6a43dbc358" }] },
+          "jenkins" =>
+          { "url" => "http://localhost:8889/cookbook_artifacts/jenkins",
+            "versions" =>
+            [{ "url" =>
                "http://localhost:8889/cookbook_artifacts/jenkins/0be380429add00d189b4431059ac967a60052323",
-              "identifier"=>"0be380429add00d189b4431059ac967a60052323"}]}
+               "identifier" => "0be380429add00d189b4431059ac967a60052323" }] },
         }
       end
       def expect_policyfile_upload
         expect(http_client).to receive(:put).
-          with('/policy_groups/unit-test/policies/example', policyfile_lock_data)
+          with("/policy_groups/unit-test/policies/example", policyfile_lock_data)
       end
 
       it "enables native document mode for policyfiles" do
@@ -395,7 +394,7 @@ describe ChefDK::Policyfile::Uploader do
 
       it "uploads the policyfile to the native API" do
         expect(http_client).to receive(:put).
-          with('/policy_groups/unit-test/policies/example', policyfile_lock_data)
+          with("/policy_groups/unit-test/policies/example", policyfile_lock_data)
 
         uploader.upload_policy
       end
@@ -407,4 +406,3 @@ describe ChefDK::Policyfile::Uploader do
   end
 
 end
-