summaryrefslogtreecommitdiff
path: root/spec/unit/application
diff options
context:
space:
mode:
authorSeth Chisamore <schisamo@opscode.com>2012-10-30 10:39:35 -0400
committerSeth Chisamore <schisamo@opscode.com>2012-10-30 10:39:35 -0400
commit24dc69a9a97e82a6e4207de68d6dcc664178249b (patch)
tree19bb289c9f88b4bbab066bc56b95d6d222fd5c35 /spec/unit/application
parent9348c1c9c80ee757354d624b7dc1b78ebc7605c4 (diff)
downloadchef-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.rb0
-rw-r--r--spec/unit/application/client_spec.rb136
-rw-r--r--spec/unit/application/knife_spec.rb152
-rw-r--r--spec/unit/application/server_spec.rb0
-rw-r--r--spec/unit/application/solo_spec.rb167
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
+