chef 10.12.0 → 10.14.0.beta.1

data/spec/unit/application/client_spec.rb

@@ -0,0 +1,136 @@
+#
+# Author:: AJ Christensen (<aj@junglist.gen.nz>)
+# Copyright:: Copyright (c) 2008 Opscode, 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'
+
+describe Chef::Application::Client, "reconfigure" do
+  before do
+    @original_config = Chef::Config.configuration
+
+    @app = Chef::Application::Client.new
+    @app.stub!(:configure_opt_parser).and_return(true)
+    @app.stub!(:configure_chef).and_return(true)
+    @app.stub!(:configure_logging).and_return(true)
+    Chef::Config[:json_attribs] = nil
+    Chef::Config[:interval] = 10
+    Chef::Config[:splay] = nil
+
+    Chef::Config[:once] = false
+  end
+
+  after do
+    Chef::Config.configuration.replace(@original_config)
+  end
+
+  describe "when in daemonized mode and no interval has been set" do
+    before do
+      Chef::Config[:daemonize] = true
+      Chef::Config[:interval] = nil
+    end
+
+    it "should set the interval to 1800" do
+      @app.reconfigure
+      Chef::Config.interval.should == 1800
+    end
+  end
+
+  describe "when configured to run once" do
+    before do
+      Chef::Config[:once] = true
+      Chef::Config[:daemonize] = false
+      Chef::Config[:splay] = 60
+      Chef::Config[:interval] = 1800
+    end
+
+    it "ignores the splay" do
+      @app.reconfigure
+      Chef::Config.splay.should be_nil
+    end
+
+    it "forces the interval to nil" do
+      @app.reconfigure
+      Chef::Config.interval.should be_nil
+    end
+
+  end
+
+  describe "when the json_attribs configuration option is specified" do
+
+    describe "and the json_attribs matches a HTTP regex" do
+      before do
+        @json = StringIO.new({:a=>"b"}.to_json)
+        @json_tempfile = mock("Tempfile for remote JSON", :open => @json)
+        @rest = mock("Chef::REST", :get_rest => @json_tempfile)
+
+        Chef::Config[:json_attribs] = "https://foo.com/foo.json"
+        Chef::REST.stub!(:new).with("https://foo.com/foo.json", nil, nil).and_return(@rest)
+        @app.stub!(:open).with("/etc/chef/dna.json").and_return(@json)
+      end
+
+      it "should perform a RESTful GET on the supplied URL" do
+        @app.reconfigure
+        @app.chef_client_json.should == {"a" => "b"}
+      end
+    end
+
+    describe "and the json_attribs does not match the HTTP regex" do
+      before do
+        Chef::Config[:json_attribs] = "/etc/chef/dna.json"
+        @json = StringIO.new({:a=>"b"}.to_json)
+        @app.stub!(:open).with("/etc/chef/dna.json").and_return(@json)
+      end
+
+      it "should parse the json out of the file" do
+        @app.reconfigure
+        @app.chef_client_json.should == {"a" => "b"}
+      end
+    end
+
+    describe "when parsing fails" do
+      before do
+        Chef::Config[:json_attribs] = "/etc/chef/dna.json"
+        @json = mock("Tempfile", :read => {:a=>"b"}.to_json)
+        @app.stub!(:open).with("/etc/chef/dna.json").and_return(@json)
+        Chef::JSONCompat.stub!(:from_json).with(@json.read).and_raise(JSON::ParserError)
+        Chef::Application.stub!(:fatal!).and_return(true)
+      end
+
+      it "should hard fail the application" do
+        Chef::Application.should_receive(:fatal!).with("Could not parse the provided JSON file (/etc/chef/dna.json)!: JSON::ParserError", 2).and_return(true)
+        @app.reconfigure
+      end
+    end
+  end
+end
+
+describe Chef::Application::Client, "setup_application" do
+  before do
+    @app = Chef::Application::Client.new
+    # this is all stuff the reconfigure method needs
+    @app.stub!(:configure_opt_parser).and_return(true)
+    @app.stub!(:configure_chef).and_return(true)
+    @app.stub!(:configure_logging).and_return(true)
+  end
+
+  it "should change privileges" do
+    Chef::Daemon.should_receive(:change_privilege).and_return(true)
+    @app.setup_application
+  end
+  after do
+    Chef::Config[:solo] = false
+  end
+end

data/spec/unit/application/knife_spec.rb

@@ -0,0 +1,152 @@
+#
+# Author:: AJ Christensen (<aj@junglist.gen.nz>)
+# Copyright:: Copyright (c) 2008 Opscode, 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_SPEC_DATA}/knife_subcommand/test_yourself"
+
+describe Chef::Application::Knife do
+  include SpecHelpers::Knife
+
+  before(:all) do
+    class NoopKnifeCommand < Chef::Knife
+      def run
+      end
+    end
+  end
+
+  before(:each) do
+    @knife = Chef::Application::Knife.new
+    @knife.stub!(:puts)
+    Chef::Knife.stub!(:list_commands)
+  end
+
+  it "should exit 1 and print the options if no arguments are given at all" do
+    with_argv([]) do
+      lambda { @knife.run }.should raise_error(SystemExit) { |e| e.status.should == 1 }
+    end
+  end
+
+  it "should exit 2 if run without a sub command" do
+    with_argv("--user", "adam") do
+      Chef::Log.should_receive(:error).with(/you need to pass a sub\-command/i)
+      lambda { @knife.run }.should raise_error(SystemExit) { |e| e.status.should == 2 }
+    end
+  end
+
+  it "should run a sub command with the applications command line option prototype" do
+    with_argv(*%w{noop knife command with some args}) do
+      knife = mock(Chef::Knife)
+      Chef::Knife.should_receive(:run).with(ARGV, @knife.options).and_return(knife)
+      @knife.should_receive(:exit).with(0)
+      @knife.run
+    end
+  end
+
+  describe "when given a path to the client key" do
+    it "expands a relative path relative to the CWD" do
+      relative_path = '.chef/client.pem'
+      Dir.stub!(:pwd).and_return(CHEF_SPEC_DATA)
+      with_argv(*%W{noop knife command -k #{relative_path}}) do
+        @knife.should_receive(:exit).with(0)
+        @knife.run
+      end
+      Chef::Config[:client_key].should == File.join(CHEF_SPEC_DATA, relative_path)
+    end
+
+    it "expands a ~/home/path to the correct full path" do
+      home_path = '~/.chef/client.pem'
+      with_argv(*%W{noop knife command -k #{home_path}}) do
+        @knife.should_receive(:exit).with(0)
+        @knife.run
+      end
+      Chef::Config[:client_key].should == File.join(ENV['HOME'], '.chef/client.pem').gsub((File::ALT_SEPARATOR || '\\'), File::SEPARATOR)
+    end
+
+    it "does not expand a full path" do
+      full_path = if windows?
+        'C:/chef/client.pem'
+      else
+        '/etc/chef/client.pem'
+      end
+      with_argv(*%W{noop knife command -k #{full_path}}) do
+        @knife.should_receive(:exit).with(0)
+        @knife.run
+      end
+      Chef::Config[:client_key].should == full_path
+    end
+
+  end
+
+  describe "with environment configuration" do
+    before do
+      Chef::Config[:environment] = nil
+    end
+
+    it "should default to no environment" do
+      with_argv(*%w{noop knife command}) do
+        @knife.should_receive(:exit).with(0)
+        @knife.run
+      end
+      Chef::Config[:environment].should == nil
+    end
+
+    it "should load the environment from the config file" do
+      config_file = File.join(CHEF_SPEC_DATA,"environment-config.rb")
+      with_argv(*%W{noop knife command -c #{config_file}}) do
+        @knife.should_receive(:exit).with(0)
+        @knife.run
+      end
+      Chef::Config[:environment].should == 'production'
+    end
+
+    it "should load the environment from the CLI options" do
+      with_argv(*%W{noop knife command -E development}) do
+        @knife.should_receive(:exit).with(0)
+        @knife.run
+      end
+      Chef::Config[:environment].should == 'development'
+    end
+
+    it "should override the config file environment with the CLI environment" do
+      config_file = File.join(CHEF_SPEC_DATA,"environment-config.rb")
+      with_argv(*%W{noop knife command -c #{config_file} -E override}) do
+        @knife.should_receive(:exit).with(0)
+        @knife.run
+      end
+      Chef::Config[:environment].should == 'override'
+    end
+
+    it "should override the config file environment with the CLI environment regardless of order" do
+      config_file = File.join(CHEF_SPEC_DATA,"environment-config.rb")
+      with_argv(*%W{noop knife command -E override -c #{config_file}}) do
+        @knife.should_receive(:exit).with(0)
+        @knife.run
+      end
+      Chef::Config[:environment].should == 'override'
+    end
+
+    it "should run a sub command with the applications command line option prototype" do
+      with_argv(*%w{noop knife command with some args}) do
+        knife = mock(Chef::Knife)
+        Chef::Knife.should_receive(:run).with(ARGV, @knife.options).and_return(knife)
+        @knife.should_receive(:exit).with(0)
+        @knife.run
+      end
+    end
+
+  end
+end

data/spec/unit/application/solo_spec.rb

@@ -0,0 +1,167 @@
+#
+# Author:: AJ Christensen (<aj@junglist.gen.nz>)
+# Copyright:: Copyright (c) 2008 Opscode, 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'
+
+describe Chef::Application::Solo do
+  before do
+    @original_config = Chef::Config.configuration
+
+
+    @app = Chef::Application::Solo.new
+    @app.stub!(:configure_opt_parser).and_return(true)
+    @app.stub!(:configure_chef).and_return(true)
+    @app.stub!(:configure_logging).and_return(true)
+    Chef::Config[:recipe_url] = false
+    Chef::Config[:json_attribs] = false
+    Chef::Config[:splay] = nil
+    Chef::Config[:solo] = true
+  end
+
+  after do
+    Chef::Config[:solo] = nil
+    Chef::Config.configuration.replace(@original_config)
+    Chef::Config[:solo] = false
+  end
+
+  describe "configuring the application" do
+    it "should set solo mode to true" do
+      @app.reconfigure
+      Chef::Config[:solo].should be_true
+    end
+
+    describe "when in daemonized mode and no interval has been set" do
+      before do
+        Chef::Config[:daemonize] = true
+      end
+
+      it "should set the interval to 1800" do
+        Chef::Config[:interval] = nil
+        @app.reconfigure
+        Chef::Config[:interval].should == 1800
+      end
+    end
+
+    describe "when the json_attribs configuration option is specified" do
+
+      describe "and the json_attribs matches a HTTP regex" do
+        before do
+          @json = StringIO.new({:a=>"b"}.to_json)
+          @json_tempfile = mock("Tempfile (mock)", :open => @json)
+          @rest = mock("Chef::REST", :get_rest => @json_tempfile)
+
+          Chef::Config[:json_attribs] = "https://foo.com/foo.json"
+          Chef::REST.stub!(:new).with("https://foo.com/foo.json", nil, nil).and_return(@rest)
+          @app.stub!(:open).with("/etc/chef/dna.json").and_return(@json)
+        end
+
+        it "should perform a RESTful GET on the supplied URL" do
+          @app.reconfigure
+          @app.chef_solo_json.should == {"a" => "b"}
+        end
+      end
+
+      describe "and the json_attribs does not match the HTTP regex" do
+        before do
+          Chef::Config[:json_attribs] = "/etc/chef/dna.json"
+          @json = StringIO.new({:a=>"b"}.to_json)
+          @app.stub!(:open).with("/etc/chef/dna.json").and_return(@json)
+        end
+
+        it "should parse the json out of the file" do
+          @app.reconfigure
+          @app.chef_solo_json.should == {"a" => "b"}
+        end
+      end
+
+      describe "when parsing fails" do
+        before do
+          Chef::Config[:json_attribs] = "/etc/chef/dna.json"
+          @json = mock("Tempfile", :read => {:a=>"b"}.to_json)
+          @app.stub!(:open).with("/etc/chef/dna.json").and_return(@json)
+          Chef::JSONCompat.stub!(:from_json).with(@json.read).and_raise(JSON::ParserError)
+          Chef::Application.stub!(:fatal!).and_return(true)
+        end
+
+        it "should hard fail the application" do
+          Chef::Application.should_receive(:fatal!).with("Could not parse the provided JSON file (/etc/chef/dna.json)!: JSON::ParserError", 2).and_return(true)
+          @app.reconfigure
+        end
+      end
+    end
+
+
+
+    describe "when the recipe_url configuration option is specified" do
+      before do
+        Chef::Config[:cookbook_path] = "#{Dir.tmpdir}/chef-solo/cookbooks"
+        Chef::Config[:recipe_url] = "http://junglist.gen.nz/recipes.tgz"
+        FileUtils.stub!(:mkdir_p).and_return(true)
+        @tarfile = StringIO.new("remote_tarball_content")
+        @app.stub!(:open).with("http://junglist.gen.nz/recipes.tgz").and_yield(@tarfile)
+
+        @target_file = StringIO.new
+        File.stub!(:open).with("#{Dir.tmpdir}/chef-solo/recipes.tgz", "wb").and_yield(@target_file)
+
+        Chef::Mixin::Command.stub!(:run_command).and_return(true)
+      end
+
+      it "should create the recipes path based on the parent of the cookbook path" do
+        FileUtils.should_receive(:mkdir_p).with("#{Dir.tmpdir}/chef-solo").and_return(true)
+        @app.reconfigure
+      end
+
+      it "should download the recipes" do
+        @app.should_receive(:open).with("http://junglist.gen.nz/recipes.tgz").and_yield(@tarfile)
+        @app.reconfigure
+      end
+
+      it "should write the recipes to the target path" do
+        @app.reconfigure
+        @target_file.string.should == "remote_tarball_content"
+      end
+
+      it "should untar the target file to the parent of the cookbook path" do
+        Chef::Mixin::Command.should_receive(:run_command).with({:command => "tar zxvfC #{Dir.tmpdir}/chef-solo/recipes.tgz #{Dir.tmpdir}/chef-solo"}).and_return(true)
+        @app.reconfigure
+      end
+    end
+  end
+
+
+  describe "after the application has been configured" do
+    before do
+      Chef::Config[:solo] = true
+
+      Chef::Daemon.stub!(:change_privilege)
+      @chef_client = mock("Chef::Client")
+      Chef::Client.stub!(:new).and_return(@chef_client)
+      @app = Chef::Application::Solo.new
+      # this is all stuff the reconfigure method needs
+      @app.stub!(:configure_opt_parser).and_return(true)
+      @app.stub!(:configure_chef).and_return(true)
+      @app.stub!(:configure_logging).and_return(true)
+    end
+
+    it "should change privileges" do
+      Chef::Daemon.should_receive(:change_privilege).and_return(true)
+      @app.setup_application
+    end
+  end
+
+end
+