chef-dk 0.2.1 → 0.3.0

data/spec/unit/policyfile_services/push_spec.rb

@@ -0,0 +1,202 @@
+#
+# Copyright:: Copyright (c) 2014 Chef Software Inc.
+# License:: Apache License, Version 2.0
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+
+require 'spec_helper'
+require 'chef-dk/policyfile_services/push'
+
+describe ChefDK::PolicyfileServices::Push do
+
+  let(:working_dir) do
+    path = File.join(tempdir, "policyfile_services_test_working_dir")
+    Dir.mkdir(path)
+    path
+  end
+
+  let(:policyfile_rb_explicit_name) { nil }
+
+  let(:policyfile_rb_name) { policyfile_rb_explicit_name || "Policyfile.rb" }
+
+  let(:policyfile_lock_name) { "Policyfile.lock.json" }
+
+  let(:policyfile_rb_path) { File.join(working_dir, policyfile_rb_name) }
+
+  let(:policyfile_lock_path) { File.join(working_dir, policyfile_lock_name) }
+
+  let(:policy_group) { "staging-cluster-1" }
+
+  let(:local_cookbooks_root) do
+    File.join(fixtures_path, "local_path_cookbooks")
+  end
+
+  let(:config) do
+    double("Chef::Config", chef_server_url: "https://localhost:10443", client_key: "/path/to/client/key.pem", node_name: "deuce")
+  end
+
+  let(:ui) { TestHelpers::TestUI.new }
+
+  let(:push_service) { described_class.new(policyfile: policyfile_rb_name, policy_group: policy_group, ui: ui, config: config, root_dir: working_dir) }
+
+  it "configures an HTTP client" do
+    expect(ChefDK::AuthenticatedHTTP).to receive(:new).with("https://localhost:10443",
+                                                       signing_key_filename: "/path/to/client/key.pem",
+                                                       client_name: "deuce")
+    push_service.http_client
+  end
+
+  it "infers the path to Policyfile.lock.json" do
+    expect(push_service.lockfile_path).to eq(policyfile_lock_path)
+  end
+
+  it "has a storage configuration" do
+    storage_config = push_service.storage_config
+    expect(storage_config.policyfile_lock_filename).to eq(policyfile_lock_path)
+    expect(storage_config.relative_paths_root).to eq(working_dir)
+  end
+
+  context "when given an explicit path to the policyfile" do
+
+    let(:policyfile_rb_name) { "MyPolicy.rb" }
+
+    let(:policyfile_lock_name) { "MyPolicy.lock.json" }
+
+    it "infers the path to the lockfile from the policyfile location" do
+      expect(push_service.lockfile_path).to eq(policyfile_lock_path)
+    end
+
+  end
+
+  context "when no lockfile is present" do
+
+    it "errors out" do
+      expect { push_service.run }.to raise_error(ChefDK::LockfileNotFound)
+    end
+
+  end
+
+  context "when a lockfile is present" do
+
+    before do
+      File.open(policyfile_lock_path, "w+") { |f| f.print(lockfile_content) }
+    end
+
+    context "and the lockfile has invalid JSON" do
+
+      let(:lockfile_content) { ":::" }
+
+      it "errors out" do
+        expect { push_service.run }.to raise_error(ChefDK::PolicyfilePushError)
+      end
+
+    end
+
+    context "and the lockfile is semantically invalid" do
+
+      let(:lockfile_content) { '{ }' }
+
+      it "errors out" do
+        expect { push_service.run }.to raise_error(ChefDK::PolicyfilePushError)
+      end
+
+    end
+
+    context "and the lockfile is valid" do
+
+      let(:local_cookbook_path) { File.join(fixtures_path, "local_path_cookbooks/local-cookbook") }
+
+      let(:lockfile_content) do
+        <<-E
+{
+  "name": "install-example",
+  "run_list": [
+    "recipe[local-cookbook::default]"
+  ],
+  "cookbook_locks": {
+    "local-cookbook": {
+      "version": "2.3.4",
+      "identifier": "fab501cfaf747901bd82c1bc706beae7dc3a350c",
+      "dotted_decimal_identifier": "70567763561641081.489844270461035.258281553147148",
+      "source": "#{local_cookbook_path}",
+      "cache_key": null,
+      "scm_info": null,
+      "source_options": {
+        "path": "#{local_cookbook_path}"
+      }
+    }
+  },
+  "solution_dependencies": {
+    "Policyfile": [
+      [
+        "local-cookbook",
+        ">= 0.0.0"
+      ]
+    ],
+    "dependencies": {
+      "local-cookbook (2.3.4)": [
+
+      ]
+    }
+  }
+}
+E
+      end
+
+      let(:http_client) { instance_double(ChefDK::AuthenticatedHTTP) }
+
+      let(:updated_lockfile_io) { StringIO.new }
+
+      let(:uploader) { instance_double(ChefDK::Policyfile::Uploader) }
+
+      before do
+        expect(push_service).to receive(:http_client).and_return(http_client)
+
+        expect(ChefDK::Policyfile::Uploader).to receive(:new).
+               with(push_service.policyfile_lock, policy_group, http_client: http_client, ui: ui).
+               and_return(uploader)
+      end
+
+
+      it "configures a Policyfile Uploader" do
+        push_service.uploader
+      end
+
+      it "validates the lockfile, writes any updates, and uploads the cookbooks" do
+        allow(File).to receive(:open).and_call_original
+        expect(File).to receive(:open).with(policyfile_lock_path, "w+").and_yield(updated_lockfile_io)
+        expect(uploader).to receive(:upload)
+
+        push_service.run
+      end
+
+      describe "when an error occurs in upload" do
+
+        before do
+          allow(File).to receive(:open).and_call_original
+          expect(File).to receive(:open).with(policyfile_lock_path, "w+").and_yield(updated_lockfile_io)
+          expect(uploader).to receive(:upload).and_raise("an error")
+        end
+
+        it "raises an error" do
+          expect { push_service.run }.to raise_error(ChefDK::PolicyfilePushError)
+        end
+
+      end
+
+    end
+
+  end
+
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: chef-dk
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.3.0
 platform: ruby
 authors:
 - Daniel DeLeo
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-08-28 00:00:00.000000000 Z
+date: 2014-10-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: mixlib-cli
@@ -28,32 +28,52 @@ dependencies:
         version: '1.5'
 - !ruby/object:Gem::Dependency
   name: mixlib-shellout
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 2.0.0.rc.0
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: 3.0.0
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 2.0.0.rc.0
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: 3.0.0
+- !ruby/object:Gem::Dependency
+  name: ffi-yajl
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.4'
+        version: '1.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.4'
+        version: '1.0'
 - !ruby/object:Gem::Dependency
   name: chef
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 12.0.0.alpha.1
+        version: 12.0.0.alpha.2
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 12.0.0.alpha.1
+        version: 12.0.0.alpha.2
 - !ruby/object:Gem::Dependency
   name: solve
   requirement: !ruby/object:Gem::Requirement
@@ -159,6 +179,8 @@ files:
 - lib/chef-dk/command/generator_commands/recipe.rb
 - lib/chef-dk/command/generator_commands/repo.rb
 - lib/chef-dk/command/generator_commands/template.rb
+- lib/chef-dk/command/install.rb
+- lib/chef-dk/command/push.rb
 - lib/chef-dk/command/shell_init.rb
 - lib/chef-dk/command/verify.rb
 - lib/chef-dk/commands_map.rb
@@ -179,11 +201,16 @@ files:
 - lib/chef-dk/policyfile/dsl.rb
 - lib/chef-dk/policyfile/null_cookbook_source.rb
 - lib/chef-dk/policyfile/read_cookbook_for_compat_mode_upload.rb
+- lib/chef-dk/policyfile/reports/install.rb
+- lib/chef-dk/policyfile/reports/table_printer.rb
+- lib/chef-dk/policyfile/reports/upload.rb
 - lib/chef-dk/policyfile/solution_dependencies.rb
 - lib/chef-dk/policyfile/storage_config.rb
 - lib/chef-dk/policyfile/uploader.rb
 - lib/chef-dk/policyfile_compiler.rb
 - lib/chef-dk/policyfile_lock.rb
+- lib/chef-dk/policyfile_services/install.rb
+- lib/chef-dk/policyfile_services/push.rb
 - lib/chef-dk/shell_out.rb
 - lib/chef-dk/skeletons/code_generator/files/default/Berksfile
 - lib/chef-dk/skeletons/code_generator/files/default/chefignore
@@ -224,6 +251,7 @@ files:
 - lib/chef-dk/skeletons/code_generator/templates/default/repo/gitignore.erb
 - lib/chef-dk/skeletons/code_generator/templates/default/resource.rb.erb
 - lib/chef-dk/skeletons/code_generator/templates/default/template.erb
+- lib/chef-dk/ui.rb
 - lib/chef-dk/version.rb
 - spec/shared/a_file_generator.rb
 - spec/shared/a_generated_file.rb
@@ -244,6 +272,8 @@ files:
 - spec/unit/command/generator_commands/recipe_spec.rb
 - spec/unit/command/generator_commands/repo_spec.rb
 - spec/unit/command/generator_commands/template_spec.rb
+- spec/unit/command/install_spec.rb
+- spec/unit/command/push_spec.rb
 - spec/unit/command/shell_init_spec.rb
 - spec/unit/command/verify_spec.rb
 - spec/unit/commands_map_spec.rb
@@ -323,6 +353,8 @@ files:
 - spec/unit/policyfile/cookbook_locks_spec.rb
 - spec/unit/policyfile/null_cookbook_source_spec.rb
 - spec/unit/policyfile/read_cookbook_for_compat_mode_upload_spec.rb
+- spec/unit/policyfile/reports/install_spec.rb
+- spec/unit/policyfile/reports/upload_spec.rb
 - spec/unit/policyfile/solution_dependencies_spec.rb
 - spec/unit/policyfile/storage_config_spec.rb
 - spec/unit/policyfile/uploader_spec.rb
@@ -330,7 +362,10 @@ files:
 - spec/unit/policyfile_evaluation_spec.rb
 - spec/unit/policyfile_lock_build_spec.rb
 - spec/unit/policyfile_lock_install_spec.rb
+- spec/unit/policyfile_lock_serialization_spec.rb
 - spec/unit/policyfile_lock_validation_spec.rb
+- spec/unit/policyfile_services/install_spec.rb
+- spec/unit/policyfile_services/push_spec.rb
 - spec/unit/shell_out_spec.rb
 homepage: http://www.getchef.com/
 licenses: []
@@ -375,6 +410,8 @@ test_files:
 - spec/unit/command/generator_commands/recipe_spec.rb
 - spec/unit/command/generator_commands/repo_spec.rb
 - spec/unit/command/generator_commands/template_spec.rb
+- spec/unit/command/install_spec.rb
+- spec/unit/command/push_spec.rb
 - spec/unit/command/shell_init_spec.rb
 - spec/unit/command/verify_spec.rb
 - spec/unit/commands_map_spec.rb
@@ -454,6 +491,8 @@ test_files:
 - spec/unit/policyfile/cookbook_locks_spec.rb
 - spec/unit/policyfile/null_cookbook_source_spec.rb
 - spec/unit/policyfile/read_cookbook_for_compat_mode_upload_spec.rb
+- spec/unit/policyfile/reports/install_spec.rb
+- spec/unit/policyfile/reports/upload_spec.rb
 - spec/unit/policyfile/solution_dependencies_spec.rb
 - spec/unit/policyfile/storage_config_spec.rb
 - spec/unit/policyfile/uploader_spec.rb
@@ -461,6 +500,9 @@ test_files:
 - spec/unit/policyfile_evaluation_spec.rb
 - spec/unit/policyfile_lock_build_spec.rb
 - spec/unit/policyfile_lock_install_spec.rb
+- spec/unit/policyfile_lock_serialization_spec.rb
 - spec/unit/policyfile_lock_validation_spec.rb
+- spec/unit/policyfile_services/install_spec.rb
+- spec/unit/policyfile_services/push_spec.rb
 - spec/unit/shell_out_spec.rb
 has_rdoc: