microwave 1.0.4 → 11.400.2

data/spec/unit/knife/status_spec.rb

@@ -22,8 +22,8 @@ require 'highline'
 describe Chef::Knife::Status do
   before(:each) do
     node = Chef::Node.new.tap do |n|
-      n["fqdn"] = "foobar"
-      n["ohai_time"] = 1343845969
+      n.automatic_attrs["fqdn"] = "foobar"
+      n.automatic_attrs["ohai_time"] = 1343845969
     end
     query = mock("Chef::Search::Query")
     query.stub!(:search).and_yield(node)

data/spec/unit/knife/user_create_spec.rb

@@ -0,0 +1,86 @@
+#
+# Author:: Steven Danna (<steve@opscode.com>)
+# Copyright:: Copyright (c) 2012 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'
+
+Chef::Knife::UserCreate.load_deps
+
+describe Chef::Knife::UserCreate do
+  before(:each) do
+    @knife = Chef::Knife::UserCreate.new
+    @knife.name_args = [ 'a_user' ]
+    @knife.config[:user_password] = "foobar"
+    @user = Chef::User.new
+    @user.name "a_user"
+    @user_with_private_key = Chef::User.new
+    @user_with_private_key.name "a_user"
+    @user_with_private_key.private_key 'private_key'
+    @user.stub!(:create).and_return(@user_with_private_key)
+    Chef::User.stub!(:new).and_return(@user)
+    Chef::User.stub!(:from_hash).and_return(@user)
+    @knife.stub!(:edit_data).and_return(@user.to_hash)
+    @stdout = StringIO.new
+    @stderr = StringIO.new
+    @knife.ui.stub!(:stdout).and_return(@stdout)
+    @knife.ui.stub!(:stderr).and_return(@stderr)
+  end
+
+  it "creates a new user" do
+    Chef::User.should_receive(:new).and_return(@user)
+    @user.should_receive(:create)
+    @knife.run
+    @stdout.string.should match /created user.+a_user/i
+  end
+
+  it "sets the password" do
+    @knife.config[:user_password] = "a_password"
+    @user.should_receive(:password).with("a_password")
+    @knife.run
+  end
+
+  it "exits with an error if password is blank" do
+    @knife.config[:user_password] = ''
+    lambda { @knife.run }.should raise_error SystemExit
+    @stderr.string.should match /You must specify a non-blank password/
+  end
+
+  it "sets the user name" do
+    @user.should_receive(:name).with("a_user")
+    @knife.run
+  end
+
+  it "sets the public key if given" do
+    @knife.config[:user_key] = "/a/filename"
+    File.stub(:read).with(File.expand_path("/a/filename")).and_return("a_key")
+    @user.should_receive(:public_key).with("a_key")
+    @knife.run
+  end
+
+  it "allows you to edit the data" do
+    @knife.should_receive(:edit_data).with(@user)
+    @knife.run
+  end
+
+  it "writes the private key to a file when --file is specified" do
+    @knife.config[:file] = "/tmp/a_file"
+    filehandle = mock("filehandle")
+    filehandle.should_receive(:print).with('private_key')
+    File.should_receive(:open).with("/tmp/a_file", "w").and_yield(filehandle)
+    @knife.run
+  end
+end

data/spec/unit/knife/user_delete_spec.rb

@@ -0,0 +1,39 @@
+#
+# Author:: Steven Danna (<steve@opscode.com>)
+# Copyright:: Copyright (c) 2012 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::Knife::UserDelete do
+  before(:each) do
+    Chef::Knife::UserDelete.load_deps
+    @knife = Chef::Knife::UserDelete.new
+    @knife.name_args = [ 'my_user' ]
+  end
+
+  it 'deletes the user' do
+    @knife.should_receive(:delete_object).with(Chef::User, 'my_user')
+    @knife.run
+  end
+
+  it 'prints usage and exits when a user name is not provided' do
+    @knife.name_args = []
+    @knife.should_receive(:show_usage)
+    @knife.ui.should_receive(:fatal)
+    lambda { @knife.run }.should raise_error(SystemExit)
+  end
+end

data/spec/unit/knife/user_edit_spec.rb

@@ -0,0 +1,42 @@
+#
+# Author:: Steven Danna (<steve@opscode.com>)
+# Copyright:: Copyright (c) 2012 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::Knife::UserEdit do
+  before(:each) do
+    Chef::Knife::UserEdit.load_deps
+    @knife = Chef::Knife::UserEdit.new
+    @knife.name_args = [ 'my_user' ]
+    @knife.config[:disable_editing] = true
+  end
+
+  it 'loads and edits the user' do
+    data = { :name => "my_user" }
+    Chef::User.stub(:load).with("my_user").and_return(data)
+    @knife.should_receive(:edit_data).with(data).and_return(data)
+    @knife.run
+  end
+
+  it 'prints usage and exits when a user name is not provided' do
+    @knife.name_args = []
+    @knife.should_receive(:show_usage)
+    @knife.ui.should_receive(:fatal)
+    lambda { @knife.run }.should raise_error(SystemExit)
+  end
+end

data/spec/unit/knife/user_list_spec.rb

@@ -0,0 +1,32 @@
+#
+# Author:: Steven Danna
+# Copyright:: Copyright (c) 2012 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::Knife::UserList do
+  before(:each) do
+    Chef::Knife::UserList.load_deps
+    @knife = Chef::Knife::UserList.new
+  end
+
+  it 'lists the users' do
+    Chef::User.should_receive(:list)
+    @knife.should_receive(:format_list_for_display)
+    @knife.run
+  end
+end

data/spec/unit/knife/user_reregister_spec.rb

@@ -0,0 +1,53 @@
+#
+# Author:: Steven Danna (<steve@opscode.com>)
+# Copyright:: Copyright (c) 2012 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::Knife::UserReregister do
+  before(:each) do
+    Chef::Knife::UserReregister.load_deps
+    @knife = Chef::Knife::UserReregister.new
+    @knife.name_args = [ 'a_user' ]
+    @user_mock = mock('user_mock', :private_key => "private_key")
+    Chef::User.stub!(:load).and_return(@user_mock)
+    @stdout = StringIO.new
+    @knife.ui.stub!(:stdout).and_return(@stdout)
+  end
+
+  it 'prints usage and exits when a user name is not provided' do
+    @knife.name_args = []
+    @knife.should_receive(:show_usage)
+    @knife.ui.should_receive(:fatal)
+    lambda { @knife.run }.should raise_error(SystemExit)
+  end
+
+  it 'reregisters the user and prints the key' do
+    @user_mock.should_receive(:reregister).and_return(@user_mock)
+    @knife.run
+    @stdout.string.should match( /private_key/ )
+  end
+
+  it 'writes the private key to a file when --file is specified' do
+    @user_mock.should_receive(:reregister).and_return(@user_mock)
+    @knife.config[:file] = '/tmp/a_file'
+    filehandle = StringIO.new
+    File.should_receive(:open).with('/tmp/a_file', 'w').and_yield(filehandle)
+    @knife.run
+    filehandle.string.should == "private_key"
+  end
+end

data/spec/unit/knife/user_show_spec.rb

@@ -0,0 +1,41 @@
+#
+# Author:: Steven Danna (<steve@opscode.com>)
+# Copyright:: Copyright (c) 2012 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::Knife::UserShow do
+  before(:each) do
+    Chef::Knife::UserShow.load_deps
+    @knife = Chef::Knife::UserShow.new
+    @knife.name_args = [ 'my_user' ]
+    @user_mock = mock('user_mock')
+  end
+
+  it 'loads and displays the user' do
+    Chef::User.should_receive(:load).with('my_user').and_return(@user_mock)
+    @knife.should_receive(:format_for_display).with(@user_mock)
+    @knife.run
+  end
+
+  it 'prints usage and exits when a user name is not provided' do
+    @knife.name_args = []
+    @knife.should_receive(:show_usage)
+    @knife.ui.should_receive(:fatal)
+    lambda { @knife.run }.should raise_error(SystemExit)
+  end
+end

data/spec/unit/knife_spec.rb

@@ -40,6 +40,29 @@ describe Chef::Knife do
     @stdout = StringIO.new
   end
 
+  describe "selecting a config file" do
+    context "when the current working dir is inside a symlinked directory" do
+      before do
+        Chef::Knife.reset_config_path!
+        # pwd according to your shell is /home/someuser/prod/chef-repo, but
+        # chef-repo is a symlink to /home/someuser/codes/chef-repo
+        ENV.stub!(:[]).with("PWD").and_return("/home/someuser/prod/chef-repo")
+        Dir.stub!(:pwd).and_return("/home/someuser/codes/chef-repo")
+      end
+
+      after do
+        Chef::Knife.reset_config_path!
+      end
+
+      it "loads the config from the non-dereferenced directory path" do
+        File.should_receive(:exist?).with("/home/someuser/prod/chef-repo/.chef").and_return(false)
+        File.should_receive(:exist?).with("/home/someuser/prod/.chef").and_return(true)
+        File.should_receive(:directory?).with("/home/someuser/prod/.chef").and_return(true)
+        Chef::Knife.chef_config_dir.should == "/home/someuser/prod/.chef"
+      end
+    end
+  end
+
   describe "after loading a subcommand" do
     before do
       Chef::Knife.reset_subcommands!
@@ -157,6 +180,36 @@ describe Chef::Knife do
       lambda {Chef::Knife.run(%w{fuuu uuuu fuuuu})}.should raise_error(SystemExit) { |e| e.status.should_not == 0 }
     end
 
+    describe "merging configuration options" do
+      before do
+        KnifeSpecs::TestYourself.option(:opt_with_default,
+                                        :short => "-D VALUE",
+                                        :default => "default-value")
+        Chef::Config[:knife] = {}
+      end
+
+      it "prefers the default value if no config or command line value is present" do
+        knife_command = KnifeSpecs::TestYourself.new([]) #empty argv
+        knife_command.configure_chef
+        knife_command.config[:opt_with_default].should == "default-value"
+      end
+
+      it "prefers a value in Chef::Config[:knife] to the default" do
+        Chef::Config[:knife][:opt_with_default] = "from-knife-config"
+        knife_command = KnifeSpecs::TestYourself.new([]) #empty argv
+        knife_command.configure_chef
+        knife_command.config[:opt_with_default].should == "from-knife-config"
+      end
+
+      it "prefers a value from command line over Chef::Config and the default" do
+        Chef::Config[:knife][:opt_with_default] = "from-knife-config"
+        knife_command = KnifeSpecs::TestYourself.new(["-D", "from-cli"])
+        knife_command.configure_chef
+        knife_command.config[:opt_with_default].should == "from-cli"
+      end
+    end
+
+
   end
 
   describe "when first created" do

data/spec/unit/lwrp_spec.rb

@@ -6,9 +6,9 @@
 # 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.
@@ -25,23 +25,23 @@ describe "override logging" do
 
   it "should log if attempting to load resource of same name" do
     Dir[File.expand_path(File.join(File.dirname(__FILE__), "..", "data", "lwrp", "resources", "*"))].each do |file|
-      Chef::Resource.build_from_file("lwrp", file, nil)
+      Chef::Resource::LWRPBase.build_from_file("lwrp", file, nil)
     end
 
     Dir[File.expand_path(File.join(File.dirname(__FILE__), "..", "data", "lwrp_override", "resources", "*"))].each do |file|
       Chef::Log.should_receive(:info).with(/overriding/)
-      Chef::Resource.build_from_file("lwrp", file, nil)
+      Chef::Resource::LWRPBase.build_from_file("lwrp", file, nil)
     end
   end
 
   it "should log if attempting to load provider of same name" do
     Dir[File.expand_path(File.join(File.dirname(__FILE__), "..", "data", "lwrp", "providers", "*"))].each do |file|
-      Chef::Provider.build_from_file("lwrp", file, nil)
+      Chef::Provider::LWRPBase.build_from_file("lwrp", file, nil)
     end
 
     Dir[File.expand_path(File.join(File.dirname(__FILE__), "..", "data", "lwrp_override", "providers", "*"))].each do |file|
       Chef::Log.should_receive(:info).with(/overriding/)
-      Chef::Provider.build_from_file("lwrp", file, nil)
+      Chef::Provider::LWRPBase.build_from_file("lwrp", file, nil)
     end
   end
 
@@ -61,11 +61,11 @@ describe "LWRP" do
 
     before do
       Dir[File.expand_path(File.join(File.dirname(__FILE__), "..", "data", "lwrp", "resources", "*"))].each do |file|
-        Chef::Resource.build_from_file("lwrp", file, nil)
+        Chef::Resource::LWRPBase.build_from_file("lwrp", file, nil)
       end
 
       Dir[File.expand_path(File.join(File.dirname(__FILE__), "..", "data", "lwrp_override", "resources", "*"))].each do |file|
-        Chef::Resource.build_from_file("lwrp", file, nil)
+        Chef::Resource::LWRPBase.build_from_file("lwrp", file, nil)
       end
     end
 
@@ -94,12 +94,12 @@ describe "LWRP" do
     end
 
     it "should have access to the run context and node during class definition" do
-      node = Chef::Node.new(nil)
-      node[:penguin_name] = "jackass"
+      node = Chef::Node.new
+      node.normal[:penguin_name] = "jackass"
       run_context = Chef::RunContext.new(node, Chef::CookbookCollection.new, @events)
 
       Dir[File.expand_path(File.join(File.dirname(__FILE__), "..", "data", "lwrp", "resources_with_default_attributes", "*"))].each do |file|
-        Chef::Resource.build_from_file("lwrp", file, run_context)
+        Chef::Resource::LWRPBase.build_from_file("lwrp", file, run_context)
       end
 
       cls = Chef::Resource.const_get("LwrpNodeattr")
@@ -113,8 +113,8 @@ describe "LWRP" do
   describe "Lightweight Chef::Provider" do
     before do
       @node = Chef::Node.new
-      @node.platform(:ubuntu)
-      @node.platform_version('8.10')
+      @node.automatic[:platform] = :ubuntu
+      @node.automatic[:platform_version] = '8.10'
       @events = Chef::EventDispatch::Dispatcher.new
       @run_context = Chef::RunContext.new(@node, Chef::CookbookCollection.new({}), @events)
       @runner = Chef::Runner.new(@run_context)
@@ -122,19 +122,19 @@ describe "LWRP" do
 
     before(:each) do
       Dir[File.expand_path(File.join(File.dirname(__FILE__), "..", "data", "lwrp", "resources", "*"))].each do |file|
-        Chef::Resource.build_from_file("lwrp", file, @run_context)
+        Chef::Resource::LWRPBase.build_from_file("lwrp", file, @run_context)
       end
 
       Dir[File.expand_path(File.join(File.dirname(__FILE__), "..", "data", "lwrp_override", "resources", "*"))].each do |file|
-        Chef::Resource.build_from_file("lwrp", file, @run_context)
+        Chef::Resource::LWRPBase.build_from_file("lwrp", file, @run_context)
       end
 
       Dir[File.expand_path(File.join(File.dirname(__FILE__), "..", "data", "lwrp", "providers", "*"))].each do |file|
-        Chef::Provider.build_from_file("lwrp", file, @run_context)
+        Chef::Provider::LWRPBase.build_from_file("lwrp", file, @run_context)
       end
 
       Dir[File.expand_path(File.join(File.dirname(__FILE__), "..", "data", "lwrp_override", "providers", "*"))].each do |file|
-        Chef::Provider.build_from_file("lwrp", file, @run_context)
+        Chef::Provider::LWRPBase.build_from_file("lwrp", file, @run_context)
       end
 
     end
@@ -226,6 +226,48 @@ describe "LWRP" do
       provider.enclosed_resource.monkey.should == 'bob, the monkey'
     end
 
+    describe "when using inline compilation" do
+      before do
+        # Behavior in these examples depends on implementation of fixture provider.
+        # See spec/data/lwrp/providers/inline_compiler
+
+        # Side effect of lwrp_inline_compiler provider for testing notifications.
+        $interior_ruby_block_2 = nil
+        # resource type doesn't matter, so make an existing resource type work with provider.
+        @resource = Chef::Resource::LwrpFoo.new("morpheus", @run_context)
+        @resource.allowed_actions << :test
+        @resource.action(:test)
+        @resource.provider(:lwrp_inline_compiler)
+      end
+
+      it "does not add interior resources to the exterior resource collection" do
+        @resource.run_action(:test)
+        @run_context.resource_collection.should be_empty
+      end
+
+      context "when interior resources are updated" do
+        it "processes notifications within the LWRP provider's action" do
+          @resource.run_action(:test)
+          $interior_ruby_block_2.should == "executed"
+        end
+
+        it "marks the parent resource updated" do
+          @resource.run_action(:test)
+          @resource.should be_updated
+          @resource.should be_updated_by_last_action
+        end
+      end
+
+      context "when interior resources are not updated" do
+        it "does not mark the parent resource updated" do
+          @resource.run_action(:no_updates)
+          @resource.should_not be_updated
+          @resource.should_not be_updated_by_last_action
+        end
+      end
+
+    end
+
   end
 
 end