microwave 0.1006.33 → 1.0.0

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.instance_variable_get(:@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.instance_variable_get(:@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
+
+
+
+    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
+

data/spec/unit/application_spec.rb

@@ -0,0 +1,239 @@
+#
+# Author:: AJ Christensen (<aj@junglist.gen.nz>)
+# Author:: Mark Mzyk (mmzyk@opscode.com)
+# 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 do
+  before do
+    Chef::Log.logger = Logger.new(StringIO.new)
+    @app = Chef::Application.new
+    Dir.stub!(:chdir).and_return(0)
+    @app.stub!(:reconfigure)
+  end
+
+  describe "reconfigure" do
+    before do
+      @app = Chef::Application.new
+      @app.stub!(:configure_chef).and_return(true)
+      @app.stub!(:configure_logging).and_return(true)
+    end
+
+    it "should configure chef" do
+      @app.should_receive(:configure_chef).and_return(true)
+      @app.reconfigure
+    end
+
+    it "should configure logging" do
+      @app.should_receive(:configure_logging).and_return(true)
+      @app.reconfigure
+    end
+
+  end
+
+  describe Chef::Application do
+    before do
+      @app = Chef::Application.new
+    end
+
+    describe "run" do
+      before do
+        @app.stub!(:setup_application).and_return(true)
+        @app.stub!(:run_application).and_return(true)
+        @app.stub!(:configure_chef).and_return(true)
+        @app.stub!(:configure_logging).and_return(true)
+      end
+
+      it "should reconfigure the application before running" do
+        @app.should_receive(:reconfigure).and_return(true)
+        @app.run
+      end
+
+      it "should setup the application before running it" do
+        @app.should_receive(:setup_application).and_return(true)
+        @app.run
+      end
+
+      it "should run the actual application" do
+        @app.should_receive(:run_application).and_return(true)
+        @app.run
+      end
+    end
+  end
+
+  describe "configure_chef" do
+    before do
+      @app = Chef::Application.new
+      #Chef::Config.stub!(:merge!).and_return(true)
+      @app.stub!(:parse_options).and_return(true)
+    end
+
+    it "should parse the commandline options" do
+      @app.should_receive(:parse_options).and_return(true)
+      @app.config[:config_file] = "/etc/chef/default.rb" #have a config file set, to prevent triggering error block
+      @app.configure_chef
+    end
+
+    describe "when a config_file is present" do
+      before do
+        Chef::Config.configuration.delete('rspec_ran')
+
+        @config_file = Tempfile.new("rspec-chef-config")
+        @config_file.puts("rspec_ran('true')")
+        @config_file.close
+
+        @app.config[:config_file] = "/etc/chef/default.rb"
+      end
+
+      after do
+        @config_file.unlink
+      end
+
+      it "should configure chef::config from a file" do
+        File.should_receive(:open).with("/etc/chef/default.rb").and_yield(@config_file)
+        Chef::Config.should_receive(:from_file).with(@config_file.path)
+        @app.configure_chef
+      end
+
+      it "should merge the local config hash into chef::config" do
+        File.should_receive(:open).with("/etc/chef/default.rb").and_yield(@config_file)
+        @app.configure_chef
+        Chef::Config.rspec_ran.should == "true"
+      end
+
+    end
+
+    describe "when there is no config_file defined" do
+      before do
+        @app.config[:config_file] = nil
+      end
+
+      it "should raise a fatal" do
+        Chef::Config.should_not_receive(:from_file).with("/etc/chef/default.rb")
+        Chef::Application.should_receive(:fatal!)
+        @app.configure_chef
+      end
+    end
+
+    describe "when the config file is set and not found" do
+      before do
+        @app.config[:config_file] = "/etc/chef/notfound"
+      end
+      it "should use the passed in command line options and defaults" do
+        Chef::Config.should_receive(:merge!)
+        @app.configure_chef
+      end
+    end
+
+    describe "when the config_file is an URL" do
+      before do
+        Chef::Config.configuration.delete('rspec_ran')
+
+        @app.config[:config_file] = "http://example.com/foo.rb"
+
+        @config_file = Tempfile.new("rspec-chef-config")
+        @config_file.puts("rspec_ran('true')")
+        @config_file.close
+
+
+        @cf = mock("cf")
+        #@cf.stub!(:path).and_return("/tmp/some/path")
+        #@cf.stub!(:nil?).and_return(false)
+        @rest = mock("rest")
+        #@rest.stub!(:get_rest).and_return(@rest)
+        #@rest.stub!(:open).and_yield(@cf)
+        Chef::REST.stub!(:new).and_return(@rest)
+      end
+
+      after {@config_file.unlink}
+
+      it "should configure chef::config from an URL" do
+        Chef::REST.should_receive(:new).with("", nil, nil).at_least(1).times.and_return(@rest)
+        @rest.should_receive(:fetch).with("http://example.com/foo.rb").and_yield(@config_file)
+        @app.configure_chef
+        Chef::Config.rspec_ran.should == "true"
+      end
+    end
+  end
+
+  describe "configure_logging" do
+    before do
+      @app = Chef::Application.new
+      Chef::Log.stub!(:init)
+      Chef::Log.stub!(:level=)
+    end
+
+    it "should initialise the chef logger" do
+      Chef::Log.should_receive(:init).with(Chef::Config[:log_location]).and_return(true)
+      @app.configure_logging
+    end
+
+    it "should initialise the chef logger level" do
+      Chef::Log.should_receive(:level=).with(Chef::Config[:log_level]).and_return(true)
+      @app.configure_logging
+    end
+
+  end
+
+  describe "class method: fatal!" do
+    before do
+      STDERR.stub!(:puts).with("FATAL: blah").and_return(true)
+      Chef::Log.stub!(:fatal).with("blah").and_return(true)
+      Process.stub!(:exit).and_return(true)
+    end
+
+    it "should log an error message to the logger" do
+      Chef::Log.should_receive(:fatal).with("blah").and_return(true)
+      Chef::Application.fatal! "blah"
+    end
+
+    describe "when an exit code is supplied" do
+      it "should exit with the given exit code" do
+        Process.should_receive(:exit).with(-100).and_return(true)
+        Chef::Application.fatal! "blah", -100
+      end
+    end
+
+    describe "when an exit code is not supplied" do
+      it "should exit with the default exit code" do
+        Process.should_receive(:exit).with(-1).and_return(true)
+        Chef::Application.fatal! "blah"
+      end
+    end
+
+  end
+
+  describe "setup_application" do
+    before do
+      @app = Chef::Application.new
+    end
+
+    it "should raise an error" do
+      lambda { @app.setup_application }.should raise_error(Chef::Exceptions::Application)
+    end
+  end
+
+  describe "run_application" do
+    before do
+      @app = Chef::Application.new
+    end
+
+    it "should raise an error" do
+      lambda { @app.run_application }.should raise_error(Chef::Exceptions::Application)
+    end
+  end
+end