chef-dk 0.11.2 → 0.12.0

data/spec/spec_helper.rb

@@ -1,50 +1,51 @@
-#
-# 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 'rubygems'
-require 'rspec/mocks'
-require 'test_helpers'
-
-# needed since we stub it for every test
-require 'chef/workstation_config_loader'
-
-RSpec.configure do |c|
-  running_on_windows = (RUBY_PLATFORM =~ /mswin|mingw|windows/)
-
-  c.include ChefDK
-  c.include TestHelpers
-
-  # Avoid loading config.rb/knife.rb unintentionally
-  c.before(:each) do
-    Chef::Config.reset
-    Chef::Config.treat_deprecation_warnings_as_errors(true)
-    allow_any_instance_of(Chef::WorkstationConfigLoader).to receive(:load)
-  end
-
-  c.after(:all) { clear_tempdir }
-
-  c.filter_run :focus => true
-  c.run_all_when_everything_filtered = true
-  # Tests that randomly fail, but may have value.
-  c.filter_run_excluding :volatile => true
-  c.filter_run_excluding :skip_on_windows => true if running_on_windows
-
-  c.mock_with(:rspec) do |mocks|
-    mocks.verify_partial_doubles = true
-  end
-
-end
+#
+# 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 'rubygems'
+require 'rspec/mocks'
+require 'test_helpers'
+
+# needed since we stub it for every test
+require 'chef/workstation_config_loader'
+require 'chef/config'
+
+RSpec.configure do |c|
+  running_on_windows = (RUBY_PLATFORM =~ /mswin|mingw|windows/)
+
+  c.include ChefDK
+  c.include TestHelpers
+
+  # Avoid loading config.rb/knife.rb unintentionally
+  c.before(:each) do
+    Chef::Config.reset
+    Chef::Config.treat_deprecation_warnings_as_errors(true)
+    allow_any_instance_of(Chef::WorkstationConfigLoader).to receive(:load)
+  end
+
+  c.after(:all) { clear_tempdir }
+
+  c.filter_run :focus => true
+  c.run_all_when_everything_filtered = true
+  # Tests that randomly fail, but may have value.
+  c.filter_run_excluding :volatile => true
+  c.filter_run_excluding :skip_on_windows => true if running_on_windows
+
+  c.mock_with(:rspec) do |mocks|
+    mocks.verify_partial_doubles = true
+  end
+
+end

data/spec/test_helpers.rb

@@ -1,84 +1,84 @@
-#
-# 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 'fileutils'
-require 'tmpdir'
-
-module TestHelpers
-
-  # A globally accessible place where we can put some state to verify that a
-  # test performed a certain operation.
-  def self.test_state
-    @test_state ||= {}
-  end
-
-  def self.reset!
-    @test_state = nil
-  end
-
-  def test_state
-    TestHelpers.test_state
-  end
-
-  def fixtures_path
-    File.expand_path(File.dirname(__FILE__) + "/unit/fixtures/")
-  end
-
-  def project_root
-    File.expand_path("../..", __FILE__)
-  end
-
-  def reset_tempdir
-    clear_tempdir
-    FileUtils.mkdir_p(tempdir)
-  end
-
-  def clear_tempdir
-    FileUtils.rm_rf(tempdir)
-    @tmpdir = nil
-  end
-
-  def tempdir
-    @tmpdir ||= Dir.mktmpdir("chef-dk")
-    File.realpath(@tmpdir)
-  end
-
-  class TestUI
-
-    attr_reader :output_stream
-
-    def initialize
-      @output_stream = StringIO.new
-    end
-
-    def err(message)
-      @output_stream.puts(message)
-    end
-
-    def msg(message)
-      @output_stream.puts(message)
-    end
-
-    def print(message)
-      @output_stream.print(message)
-    end
-
-    def output
-      @output_stream.string
-    end
-  end
-end
+#
+# 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 'fileutils'
+require 'tmpdir'
+
+module TestHelpers
+
+  # A globally accessible place where we can put some state to verify that a
+  # test performed a certain operation.
+  def self.test_state
+    @test_state ||= {}
+  end
+
+  def self.reset!
+    @test_state = nil
+  end
+
+  def test_state
+    TestHelpers.test_state
+  end
+
+  def fixtures_path
+    File.expand_path(File.dirname(__FILE__) + "/unit/fixtures/")
+  end
+
+  def project_root
+    File.expand_path("../..", __FILE__)
+  end
+
+  def reset_tempdir
+    clear_tempdir
+    FileUtils.mkdir_p(tempdir)
+  end
+
+  def clear_tempdir
+    FileUtils.rm_rf(tempdir)
+    @tmpdir = nil
+  end
+
+  def tempdir
+    @tmpdir ||= Dir.mktmpdir("chef-dk")
+    File.realpath(@tmpdir)
+  end
+
+  class TestUI
+
+    attr_reader :output_stream
+
+    def initialize
+      @output_stream = StringIO.new
+    end
+
+    def err(message)
+      @output_stream.puts(message)
+    end
+
+    def msg(message)
+      @output_stream.puts(message)
+    end
+
+    def print(message)
+      @output_stream.print(message)
+    end
+
+    def output
+      @output_stream.string
+    end
+  end
+end

data/spec/unit/chef_runner_spec.rb

@@ -1,139 +1,139 @@
-#
-# 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 'stringio'
-require 'chef-dk/chef_runner'
-
-describe ChefDK::ChefRunner do
-
-  let(:stdout_io) { StringIO.new }
-
-  let(:stderr_io) { StringIO.new }
-
-  let(:default_cookbook_path) do
-    File.expand_path("chef-runner-cookbooks", fixtures_path)
-  end
-
-  let(:run_list) { [ "recipe[test_cookbook::recipe_one]", "recipe[test_cookbook::recipe_two]" ] }
-
-  subject(:chef_runner) do
-    r = ChefDK::ChefRunner.new(default_cookbook_path, run_list)
-    allow(r).to receive(:stdout).and_return(stdout_io)
-    allow(r).to receive(:stderr).and_return(stderr_io)
-    r
-  end
-
-  it "sets up Chef::Config" do
-    chef_runner.configure
-    expect(Chef::Config.solo).to be true
-    expect(Chef::Config.cookbook_path).to eq(default_cookbook_path)
-    expect(Chef::Config.color).to be true
-    expect(Chef::Config.diff_disabled).to be true
-    expect(Chef::Config.file_staging_uses_destdir).to be true
-  end
-
-  it "disables atomic file updates on Windows" do
-    allow(Chef::Platform).to receive(:windows?) { true }
-    chef_runner.configure
-    expect(Chef::Config.file_atomic_update).to be false
-  end
-
-  it "configures a formatter for the chef run" do
-    expect(chef_runner.formatter).to be_a(Chef::EventDispatch::Dispatcher)
-
-    # TODO: Once https://github.com/chef/chef/pull/3340 is merged/released,
-    # just use `formatter.subscribers`
-    subscribers = chef_runner.formatter.instance_variable_get(:@subscribers)
-
-    expect(subscribers.size).to eq(1)
-    expect(subscribers.first).to be_a(Chef::Formatters::Doc)
-  end
-
-  it "detects the platform with ohai" do
-    expect(chef_runner.ohai).to be_a(Ohai::System)
-    expect(chef_runner.ohai.data["platform"]).to_not be_nil
-    expect(chef_runner.ohai.data["platform_version"]).to_not be_nil
-  end
-
-  it "sets up chef policy" do
-    chef_runner.configure
-    expect(chef_runner.policy.node.run_list).to eq(run_list)
-  end
-
-  it "runs a chef converge" do
-    chef_runner.converge
-    expect(test_state[:loaded_recipes]).to eq([ "recipe_one", "recipe_two" ])
-    expect(test_state[:converged_recipes]).to eq([ "recipe_one", "recipe_two" ])
-  end
-
-  context "when policyfile options are set in the workstation config" do
-
-    before do
-      Chef::Config.use_policyfile true
-      Chef::Config.policy_name "workstation"
-      Chef::Config.policy_group "test"
-
-      # chef-client ignores `deployment_group` unless
-      # `policy_document_native_api` is set to false
-      Chef::Config.deployment_group "workstation-test"
-      Chef::Config.policy_document_native_api false
-    end
-
-    it "unsets the options" do
-      chef_runner.configure
-
-      expect(Chef::Config.use_policyfile).to be(false)
-      expect(Chef::Config.policy_name).to be_nil
-      expect(Chef::Config.policy_group).to be_nil
-      expect(Chef::Config.deployment_group).to be_nil
-    end
-
-    it "converges successfully" do
-      expect { chef_runner.converge }.to_not raise_error
-    end
-
-  end
-
-  context "when the embedded chef run fails" do
-
-    let(:embedded_runner) { instance_double("Chef::Runner") }
-
-    before do
-      allow(Chef::Runner).to receive(:new).and_return(embedded_runner)
-      allow(embedded_runner).to receive(:converge).and_raise("oops")
-    end
-
-    it "wraps the exception in a ChefConvergeError" do
-      expect { chef_runner.converge }.to raise_error(ChefDK::ChefConvergeError)
-    end
-
-  end
-
-  context "when cookbook_path is relative" do
-
-    let(:default_cookbook_path) { "~/heres_some_cookbooks" }
-
-    it "expands the path" do
-      expect(chef_runner.cookbook_path).to eq(File.expand_path(default_cookbook_path))
-    end
-
-  end
-
-end
-
-
+#
+# 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 'stringio'
+require 'chef-dk/chef_runner'
+
+describe ChefDK::ChefRunner do
+
+  let(:stdout_io) { StringIO.new }
+
+  let(:stderr_io) { StringIO.new }
+
+  let(:default_cookbook_path) do
+    File.expand_path("chef-runner-cookbooks", fixtures_path)
+  end
+
+  let(:run_list) { [ "recipe[test_cookbook::recipe_one]", "recipe[test_cookbook::recipe_two]" ] }
+
+  subject(:chef_runner) do
+    r = ChefDK::ChefRunner.new(default_cookbook_path, run_list)
+    allow(r).to receive(:stdout).and_return(stdout_io)
+    allow(r).to receive(:stderr).and_return(stderr_io)
+    r
+  end
+
+  it "sets up Chef::Config" do
+    chef_runner.configure
+    expect(Chef::Config.solo).to be true
+    expect(Chef::Config.cookbook_path).to eq(default_cookbook_path)
+    expect(Chef::Config.color).to be true
+    expect(Chef::Config.diff_disabled).to be true
+    expect(Chef::Config.file_staging_uses_destdir).to be true
+  end
+
+  it "disables atomic file updates on Windows" do
+    allow(Chef::Platform).to receive(:windows?) { true }
+    chef_runner.configure
+    expect(Chef::Config.file_atomic_update).to be false
+  end
+
+  it "configures a formatter for the chef run" do
+    expect(chef_runner.formatter).to be_a(Chef::EventDispatch::Dispatcher)
+
+    # TODO: Once https://github.com/chef/chef/pull/3340 is merged/released,
+    # just use `formatter.subscribers`
+    subscribers = chef_runner.formatter.instance_variable_get(:@subscribers)
+
+    expect(subscribers.size).to eq(1)
+    expect(subscribers.first).to be_a(Chef::Formatters::Doc)
+  end
+
+  it "detects the platform with ohai" do
+    expect(chef_runner.ohai).to be_a(Ohai::System)
+    expect(chef_runner.ohai.data["platform"]).to_not be_nil
+    expect(chef_runner.ohai.data["platform_version"]).to_not be_nil
+  end
+
+  it "sets up chef policy" do
+    chef_runner.configure
+    expect(chef_runner.policy.node.run_list).to eq(run_list)
+  end
+
+  it "runs a chef converge" do
+    chef_runner.converge
+    expect(test_state[:loaded_recipes]).to eq([ "recipe_one", "recipe_two" ])
+    expect(test_state[:converged_recipes]).to eq([ "recipe_one", "recipe_two" ])
+  end
+
+  context "when policyfile options are set in the workstation config" do
+
+    before do
+      Chef::Config.use_policyfile true
+      Chef::Config.policy_name "workstation"
+      Chef::Config.policy_group "test"
+
+      # chef-client ignores `deployment_group` unless
+      # `policy_document_native_api` is set to false
+      Chef::Config.deployment_group "workstation-test"
+      Chef::Config.policy_document_native_api false
+    end
+
+    it "unsets the options" do
+      chef_runner.configure
+
+      expect(Chef::Config.use_policyfile).to be(false)
+      expect(Chef::Config.policy_name).to be_nil
+      expect(Chef::Config.policy_group).to be_nil
+      expect(Chef::Config.deployment_group).to be_nil
+    end
+
+    it "converges successfully" do
+      expect { chef_runner.converge }.to_not raise_error
+    end
+
+  end
+
+  context "when the embedded chef run fails" do
+
+    let(:embedded_runner) { instance_double("Chef::Runner") }
+
+    before do
+      allow(Chef::Runner).to receive(:new).and_return(embedded_runner)
+      allow(embedded_runner).to receive(:converge).and_raise("oops")
+    end
+
+    it "wraps the exception in a ChefConvergeError" do
+      expect { chef_runner.converge }.to raise_error(ChefDK::ChefConvergeError)
+    end
+
+  end
+
+  context "when cookbook_path is relative" do
+
+    let(:default_cookbook_path) { "~/heres_some_cookbooks" }
+
+    it "expands the path" do
+      expect(chef_runner.cookbook_path).to eq(File.expand_path(default_cookbook_path))
+    end
+
+  end
+
+end
+
+