diff options
author | Seth Chisamore <schisamo@opscode.com> | 2012-10-30 10:39:35 -0400 |
---|---|---|
committer | Seth Chisamore <schisamo@opscode.com> | 2012-10-30 10:39:35 -0400 |
commit | 24dc69a9a97e82a6e4207de68d6dcc664178249b (patch) | |
tree | 19bb289c9f88b4bbab066bc56b95d6d222fd5c35 /spec/unit/application | |
parent | 9348c1c9c80ee757354d624b7dc1b78ebc7605c4 (diff) | |
download | chef-24dc69a9a97e82a6e4207de68d6dcc664178249b.tar.gz |
[OC-3564] move core Chef to the repo root \o/ \m/
The opscode/chef repository now only contains the core Chef library code
used by chef-client, knife and chef-solo!
Diffstat (limited to 'spec/unit/application')
-rw-r--r-- | spec/unit/application/agent_spec.rb | 0 | ||||
-rw-r--r-- | spec/unit/application/client_spec.rb | 136 | ||||
-rw-r--r-- | spec/unit/application/knife_spec.rb | 152 | ||||
-rw-r--r-- | spec/unit/application/server_spec.rb | 0 | ||||
-rw-r--r-- | spec/unit/application/solo_spec.rb | 167 |
5 files changed, 455 insertions, 0 deletions
diff --git a/spec/unit/application/agent_spec.rb b/spec/unit/application/agent_spec.rb new file mode 100644 index 0000000000..e69de29bb2 --- /dev/null +++ b/spec/unit/application/agent_spec.rb diff --git a/spec/unit/application/client_spec.rb b/spec/unit/application/client_spec.rb new file mode 100644 index 0000000000..c5480c4adc --- /dev/null +++ b/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 diff --git a/spec/unit/application/knife_spec.rb b/spec/unit/application/knife_spec.rb new file mode 100644 index 0000000000..78a65e7045 --- /dev/null +++ b/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 diff --git a/spec/unit/application/server_spec.rb b/spec/unit/application/server_spec.rb new file mode 100644 index 0000000000..e69de29bb2 --- /dev/null +++ b/spec/unit/application/server_spec.rb diff --git a/spec/unit/application/solo_spec.rb b/spec/unit/application/solo_spec.rb new file mode 100644 index 0000000000..148fb3cf87 --- /dev/null +++ b/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 + |