microwave 0.1006.33 → 1.0.0

data/spec/unit/mixin/securable_spec.rb

@@ -0,0 +1,254 @@
+#
+# Author:: Mark Mzyk (<mmzyk@opscode.com>)
+# Copyright:: Copyright (c) 2011 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::Mixin::Securable do
+
+  before(:each) do
+    @securable = Object.new
+    @securable.send(:extend, Chef::Mixin::Securable)
+    @securable.send(:extend, Chef::Mixin::ParamsValidate)
+  end
+
+  it "should accept a group name or id for group" do
+    lambda { @securable.group "root" }.should_not raise_error(ArgumentError)
+    lambda { @securable.group 123 }.should_not raise_error(ArgumentError)
+    lambda { @securable.group "root*goo" }.should raise_error(ArgumentError)
+  end
+
+  it "should accept a user name or id for owner" do
+    lambda { @securable.owner "root" }.should_not raise_error(ArgumentError)
+    lambda { @securable.owner 123 }.should_not raise_error(ArgumentError)
+    lambda { @securable.owner "root*goo" }.should raise_error(ArgumentError)
+  end
+
+  it "allows the owner to be specified as #user" do
+    @securable.should respond_to(:user)
+  end
+
+  describe "unix-specific behavior" do
+    before(:each) do
+      platform_mock :unix do
+        @original_config = Chef::Config.hash_dup
+        load File.join(File.dirname(__FILE__), "..", "..", "..", "lib", "chef", "config.rb")
+        load File.join(File.dirname(__FILE__), "..", "..", "..", "lib", "chef", "mixin", "securable.rb")
+        @securable = Object.new
+        @securable.send(:extend, Chef::Mixin::Securable)
+        @securable.send(:extend, Chef::Mixin::ParamsValidate)
+      end
+    end
+
+    after(:each) do
+      Chef::Config.configuration = @original_config
+    end
+
+    it "should accept a group name or id for group with spaces and backslashes" do
+      lambda { @securable.group 'test\ group' }.should_not raise_error(ArgumentError)
+    end
+
+    it "should accept a unix file mode in string form as an octal number" do
+      lambda { @securable.mode "0" }.should_not raise_error(ArgumentError)
+      lambda { @securable.mode "0000" }.should_not raise_error(ArgumentError)
+      lambda { @securable.mode "0111" }.should_not raise_error(ArgumentError)
+      lambda { @securable.mode "0444" }.should_not raise_error(ArgumentError)
+
+      lambda { @securable.mode "111" }.should_not raise_error(ArgumentError)
+      lambda { @securable.mode "444" }.should_not raise_error(ArgumentError)
+      lambda { @securable.mode "7777" }.should_not raise_error(ArgumentError)
+      lambda { @securable.mode "07777" }.should_not raise_error(ArgumentError)
+
+      lambda { @securable.mode "-01" }.should raise_error(ArgumentError)
+      lambda { @securable.mode "010000" }.should raise_error(ArgumentError)
+      lambda { @securable.mode "-1" }.should raise_error(ArgumentError)
+      lambda { @securable.mode "10000" }.should raise_error(ArgumentError)
+
+      lambda { @securable.mode "07778" }.should raise_error(ArgumentError)
+      lambda { @securable.mode "7778" }.should raise_error(ArgumentError)
+      lambda { @securable.mode "4095" }.should raise_error(ArgumentError)
+
+      lambda { @securable.mode "0foo1234" }.should raise_error(ArgumentError)
+      lambda { @securable.mode "foo1234" }.should raise_error(ArgumentError)
+    end
+
+    it "should accept a unix file mode in numeric form as a ruby-interpreted integer" do
+      lambda { @securable.mode 0 }.should_not raise_error(ArgumentError)
+      lambda { @securable.mode 0000 }.should_not raise_error(ArgumentError)
+      lambda { @securable.mode 444 }.should_not raise_error(ArgumentError)
+      lambda { @securable.mode 0444 }.should_not raise_error(ArgumentError)
+      lambda { @securable.mode 07777 }.should_not raise_error(ArgumentError)
+
+      lambda { @securable.mode 292 }.should_not raise_error(ArgumentError)
+      lambda { @securable.mode 4095 }.should_not raise_error(ArgumentError)
+
+      lambda { @securable.mode 0111 }.should_not raise_error(ArgumentError)
+      lambda { @securable.mode 73 }.should_not raise_error(ArgumentError)
+
+      lambda { @securable.mode -01 }.should raise_error(ArgumentError)
+      lambda { @securable.mode 010000 }.should raise_error(ArgumentError)
+      lambda { @securable.mode -1 }.should raise_error(ArgumentError)
+      lambda { @securable.mode 4096 }.should raise_error(ArgumentError)
+    end
+  end
+
+  describe "windows-specific behavior" do
+    before(:each) do
+      platform_mock :windows do
+        @original_config = Chef::Config.hash_dup
+        load File.join(File.dirname(__FILE__), "..", "..", "..", "lib", "chef", "config.rb")
+        load File.join(File.dirname(__FILE__), "..", "..", "..", "lib", "chef", "mixin", "securable.rb")
+        @securable = Object.new
+        @securable.send(:extend, Chef::Mixin::Securable)
+        @securable.send(:extend, Chef::Mixin::ParamsValidate)
+      end
+    end
+
+    after(:all) do
+      Chef::Config.configuration = @original_config if @original_config
+    end
+
+    after(:each) do
+      Chef::Config.configuration = @original_config if @original_config
+    end
+
+    it "should not accept a group name or id for group with spaces and multiple backslashes" do
+      lambda { @securable.group 'test\ \group' }.should raise_error(ArgumentError)
+    end
+
+    it "should accept a unix file mode in string form as an octal number" do
+      lambda { @securable.mode "0" }.should_not raise_error(ArgumentError)
+      lambda { @securable.mode "0000" }.should_not raise_error(ArgumentError)
+      lambda { @securable.mode "0111" }.should_not raise_error(ArgumentError)
+      lambda { @securable.mode "0444" }.should_not raise_error(ArgumentError)
+
+      lambda { @securable.mode "111" }.should_not raise_error(ArgumentError)
+      lambda { @securable.mode "444" }.should_not raise_error(ArgumentError)
+      lambda { @securable.mode "7777" }.should raise_error(ArgumentError)
+      lambda { @securable.mode "07777" }.should raise_error(ArgumentError)
+
+      lambda { @securable.mode "-01" }.should raise_error(ArgumentError)
+      lambda { @securable.mode "010000" }.should raise_error(ArgumentError)
+      lambda { @securable.mode "-1" }.should raise_error(ArgumentError)
+      lambda { @securable.mode "10000" }.should raise_error(ArgumentError)
+
+      lambda { @securable.mode "07778" }.should raise_error(ArgumentError)
+      lambda { @securable.mode "7778" }.should raise_error(ArgumentError)
+      lambda { @securable.mode "4095" }.should raise_error(ArgumentError)
+
+      lambda { @securable.mode "0foo1234" }.should raise_error(ArgumentError)
+      lambda { @securable.mode "foo1234" }.should raise_error(ArgumentError)
+    end
+
+    it "should accept a unix file mode in numeric form as a ruby-interpreted integer" do
+      lambda { @securable.mode 0 }.should_not raise_error(ArgumentError)
+      lambda { @securable.mode 0000 }.should_not raise_error(ArgumentError)
+      lambda { @securable.mode 444 }.should_not raise_error(ArgumentError)
+      lambda { @securable.mode 0444 }.should_not raise_error(ArgumentError)
+      lambda { @securable.mode 07777 }.should raise_error(ArgumentError)
+
+      lambda { @securable.mode 292 }.should_not raise_error(ArgumentError)
+      lambda { @securable.mode 4095 }.should raise_error(ArgumentError)
+
+      lambda { @securable.mode 0111 }.should_not raise_error(ArgumentError)
+      lambda { @securable.mode 73 }.should_not raise_error(ArgumentError)
+
+      lambda { @securable.mode -01 }.should raise_error(ArgumentError)
+      lambda { @securable.mode 010000 }.should raise_error(ArgumentError)
+      lambda { @securable.mode -1 }.should raise_error(ArgumentError)
+      lambda { @securable.mode 4096 }.should raise_error(ArgumentError)
+    end
+
+    it "should allow you to specify :full_control, :modify, :read_execute, :read, and :write rights" do
+      lambda { @securable.rights :full_control, "The Dude" }.should_not raise_error(ArgumentError)
+      lambda { @securable.rights :modify, "The Dude" }.should_not raise_error(ArgumentError)
+      lambda { @securable.rights :read_execute, "The Dude" }.should_not raise_error(ArgumentError)
+      lambda { @securable.rights :read, "The Dude" }.should_not raise_error(ArgumentError)
+      lambda { @securable.rights :write, "The Dude" }.should_not raise_error(ArgumentError)
+      lambda { @securable.rights :to_party, "The Dude" }.should raise_error(ArgumentError)
+    end
+
+    it "should allow you to specify :full_control, :modify, :read_execute, :read, and :write deny_rights" do
+      lambda { @securable.deny_rights :full_control, "The Dude" }.should_not raise_error(ArgumentError)
+      lambda { @securable.deny_rights :modify, "The Dude" }.should_not raise_error(ArgumentError)
+      lambda { @securable.deny_rights :read_execute, "The Dude" }.should_not raise_error(ArgumentError)
+      lambda { @securable.deny_rights :read, "The Dude" }.should_not raise_error(ArgumentError)
+      lambda { @securable.deny_rights :write, "The Dude" }.should_not raise_error(ArgumentError)
+      lambda { @securable.deny_rights :to_party, "The Dude" }.should raise_error(ArgumentError)
+    end
+
+    it "should accept a principal as a string or an array" do
+      lambda { @securable.rights :read, "The Dude" }.should_not raise_error(ArgumentError)
+      lambda { @securable.rights :read, ["The Dude","Donny"] }.should_not raise_error(ArgumentError)
+      lambda { @securable.rights :read, 3 }.should raise_error(ArgumentError)
+    end
+
+    it "should allow you to specify whether the permissions applies_to_children with true/false/:containers_only/:objects_only" do
+      lambda { @securable.rights :read, "The Dude", :applies_to_children => false }.should_not raise_error(ArgumentError)
+      lambda { @securable.rights :read, "The Dude", :applies_to_children => true }.should_not raise_error(ArgumentError)
+      lambda { @securable.rights :read, "The Dude", :applies_to_children => :containers_only }.should_not raise_error(ArgumentError)
+      lambda { @securable.rights :read, "The Dude", :applies_to_children => :objects_only }.should_not raise_error(ArgumentError)
+      lambda { @securable.rights :read, "The Dude", :applies_to_children => 'poop' }.should raise_error(ArgumentError)
+    end
+
+    it "should allow you to specify whether the permissions applies_to_self with true/false" do
+      lambda { @securable.rights :read, "The Dude", :applies_to_children => true, :applies_to_self => false }.should_not raise_error(ArgumentError)
+      lambda { @securable.rights :read, "The Dude", :applies_to_self => true }.should_not raise_error(ArgumentError)
+      lambda { @securable.rights :read, "The Dude", :applies_to_self => 'poop' }.should raise_error(ArgumentError)
+    end
+
+    it "should allow you to specify whether the permissions applies one_level_deep with true/false" do
+      lambda { @securable.rights :read, "The Dude", :applies_to_children => true, :one_level_deep => false }.should_not raise_error(ArgumentError)
+      lambda { @securable.rights :read, "The Dude", :applies_to_children => true, :one_level_deep => true }.should_not raise_error(ArgumentError)
+      lambda { @securable.rights :read, "The Dude", :applies_to_children => true, :one_level_deep => 'poop' }.should raise_error(ArgumentError)
+    end
+
+    it "should allow multiple rights and deny_rights declarations" do
+      @securable.rights :read, "The Dude"
+      @securable.deny_rights :full_control, "The Dude"
+      @securable.rights :full_control, "The Dude"
+      @securable.rights :write, "The Dude"
+      @securable.deny_rights :read, "The Dude"
+      @securable.rights.size.should == 3
+      @securable.deny_rights.size.should == 2
+    end
+
+    it "should allow you to specify whether the permission applies_to_self only if you specified applies_to_children" do
+      lambda { @securable.rights :read, "The Dude", :applies_to_children => true, :applies_to_self => true }.should_not raise_error(ArgumentError)
+      lambda { @securable.rights :read, "The Dude", :applies_to_children => true, :applies_to_self => false }.should_not raise_error(ArgumentError)
+      lambda { @securable.rights :read, "The Dude", :applies_to_children => false, :applies_to_self => true }.should_not raise_error(ArgumentError)
+      lambda { @securable.rights :read, "The Dude", :applies_to_children => false, :applies_to_self => false }.should raise_error(ArgumentError)
+      lambda { @securable.rights :read, "The Dude", :applies_to_self => true }.should_not raise_error(ArgumentError)
+      lambda { @securable.rights :read, "The Dude", :applies_to_self => false }.should_not raise_error(ArgumentError)
+    end
+
+    it "should allow you to specify whether the permission applies one_level_deep only if you specified applies_to_children" do
+      lambda { @securable.rights :read, "The Dude", :applies_to_children => true, :one_level_deep => true }.should_not raise_error(ArgumentError)
+      lambda { @securable.rights :read, "The Dude", :applies_to_children => true, :one_level_deep => false }.should_not raise_error(ArgumentError)
+      lambda { @securable.rights :read, "The Dude", :applies_to_children => false, :one_level_deep => true }.should raise_error(ArgumentError)
+      lambda { @securable.rights :read, "The Dude", :applies_to_children => false, :one_level_deep => false }.should_not raise_error(ArgumentError)
+      lambda { @securable.rights :read, "The Dude", :one_level_deep => true }.should_not raise_error(ArgumentError)
+      lambda { @securable.rights :read, "The Dude", :one_level_deep => false }.should_not raise_error(ArgumentError)
+    end
+
+    it "should allow you to specify whether the permissions inherit with true/false" do
+      lambda { @securable.inherits true }.should_not raise_error(ArgumentError)
+      lambda { @securable.inherits false }.should_not raise_error(ArgumentError)
+      lambda { @securable.inherits "monkey" }.should raise_error(ArgumentError)
+    end
+  end
+end

data/spec/unit/mixin/shell_out_spec.rb

@@ -0,0 +1,109 @@
+#
+# Author:: Ho-Sheng Hsiao (hosh@opscode.com)
+# Code derived from spec/unit/mixin/command_spec.rb
+#
+# Original header:
+# Author:: Hongli Lai (hongli@phusion.nl)
+# Copyright:: Copyright (c) 2009 Phusion
+# 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::Mixin::ShellOut do
+  include Chef::Mixin::ShellOut
+
+  describe '#run_command_compatible_options' do
+    subject { run_command_compatible_options(command_args) }
+    let(:command_args) { [ cmd, options ] }
+    let(:cmd) { "echo '#{rand(1000)}'" }
+
+    let(:output) { StringIO.new }
+    let(:capture_log_output) { Chef::Log.logger = Logger.new(output)  }
+    let(:assume_deprecation_log_level) { Chef::Log.stub!(:level).and_return(:warn) }
+
+    context 'without options' do
+      let(:command_args) { [ cmd ] }
+
+      it 'should not edit command args' do
+        should eql(command_args)
+      end
+    end
+
+    context 'without deprecated options' do
+      let(:options) { { :environment => environment } }
+      let(:environment) { { 'LC_ALL' => 'C' } }
+
+      it 'should not edit command args' do
+        should eql(command_args)
+      end
+    end
+
+    def self.should_emit_deprecation_warning_about(old_option, new_option)
+      it 'should emit a deprecation warning' do
+        assume_deprecation_log_level and capture_log_output
+        subject
+        output.string.should match /DEPRECATION:/
+        output.string.should match Regexp.escape(old_option.to_s)
+        output.string.should match Regexp.escape(new_option.to_s)
+      end
+    end
+
+    context 'with :command_log_level option' do
+      let(:options) { { :command_log_level => command_log_level } }
+      let(:command_log_level) { :warn }
+
+      it 'should convert :command_log_level to :log_level' do
+        should eql [ cmd, { :log_level => command_log_level } ]
+      end
+
+      should_emit_deprecation_warning_about :command_log_level, :log_level
+    end
+
+    context 'with :command_log_prepend option' do
+      let(:options) { { :command_log_prepend => command_log_prepend } }
+      let(:command_log_prepend) { 'PROVIDER:' }
+
+      it 'should convert :command_log_prepend to :log_tag' do
+        should eql [ cmd, { :log_tag => command_log_prepend } ]
+      end
+
+      should_emit_deprecation_warning_about :command_log_prepend, :log_tag
+    end
+
+    context "with 'command_log_level' option" do
+      let(:options) { { 'command_log_level' => command_log_level } }
+      let(:command_log_level) { :warn }
+
+      it "should convert 'command_log_level' to :log_level" do
+        should eql [ cmd, { :log_level => command_log_level } ]
+      end
+
+      should_emit_deprecation_warning_about :command_log_level, :log_level
+    end
+
+    context "with 'command_log_prepend' option" do
+      let(:options) { { 'command_log_prepend' => command_log_prepend } }
+      let(:command_log_prepend) { 'PROVIDER:' }
+
+      it "should convert 'command_log_prepend' to :log_tag" do
+        should eql [ cmd, { :log_tag => command_log_prepend } ]
+      end
+
+      should_emit_deprecation_warning_about :command_log_prepend, :log_tag
+    end
+
+  end
+end

data/spec/unit/mixin/template_spec.rb

@@ -0,0 +1,104 @@
+#
+# Author:: Adam Jacob (<adam@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'
+
+class TinyTemplateClass; include Chef::Mixin::Template; end
+require 'cgi'
+describe Chef::Mixin::Template, "render_template" do
+
+  before :each do
+    @template = TinyTemplateClass.new
+  end
+
+  it "should render the template evaluated in the given context" do
+    @template.render_template("<%= @foo %>", { :foo => "bar" }) do |tmp|
+      tmp.open.read.should == "bar"
+    end
+  end
+
+  it "should provide a node method to access @node" do
+    @template.render_template("<%= node %>",{:node => "tehShizzle"}) do |tmp|
+      tmp.open.read.should == "tehShizzle"
+    end
+  end
+
+  it "should yield the tempfile it renders the template to" do
+    @template.render_template("abcdef", {}) do |tempfile|
+      tempfile.should be_kind_of(Tempfile)
+    end
+  end
+
+  describe "when an exception is raised in the template" do
+    def do_raise
+      @context = {:chef => "cool"}
+      @template.render_template("foo\nbar\nbaz\n<%= this_is_not_defined %>\nquin\nqunx\ndunno", @context) {|r| r}
+    end
+
+    it "should catch and re-raise the exception as a TemplateError" do
+      lambda { do_raise }.should raise_error(Chef::Mixin::Template::TemplateError)
+    end
+
+    it "should raise an error if an attempt is made to access node but it is nil" do
+      lambda {@template.render_template("<%= node %>",{}) {|r| r}}.should raise_error(Chef::Mixin::Template::TemplateError)
+    end
+
+    describe "the raised TemplateError" do
+      before :each do
+        begin
+          do_raise
+        rescue Chef::Mixin::Template::TemplateError => e
+          @exception = e
+        end
+      end
+
+      it "should have the original exception" do
+        @exception.original_exception.should be
+        @exception.original_exception.message.should =~ /undefined local variable or method `this_is_not_defined'/
+      end
+
+      it "should determine the line number of the exception" do
+        @exception.line_number.should == 4
+      end
+
+      it "should provide a source listing of the template around the exception" do
+        @exception.source_listing.should == "  2: bar\n  3: baz\n  4: <%= this_is_not_defined %>\n  5: quin\n  6: qunx"
+      end
+
+      it "should provide the evaluation context of the template" do
+        @exception.context.should == @context
+      end
+
+      it "should defer the message to the original exception" do
+        @exception.message.should =~ /undefined local variable or method `this_is_not_defined'/
+      end
+
+      it "should provide a nice source location" do
+        @exception.source_location.should == "on line #4"
+      end
+
+      it "should create a pretty output for the terminal" do
+        @exception.to_s.should =~ /Chef::Mixin::Template::TemplateError/
+        @exception.to_s.should =~ /undefined local variable or method `this_is_not_defined'/
+        @exception.to_s.should include("  2: bar\n  3: baz\n  4: <%= this_is_not_defined %>\n  5: quin\n  6: qunx")
+        @exception.to_s.should include(@exception.original_exception.backtrace.first)
+      end
+    end
+  end
+end
+