chef 10.28.2-x86-mingw32

data/spec/unit/knife/client_delete_spec.rb

@@ -0,0 +1,40 @@
+#
+# Author:: Thomas Bishop (<bishop.thomas@gmail.com>)
+# Copyright:: Copyright (c) 2011 Thomas Bishop
+# 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::ClientDelete do
+  before(:each) do
+    @knife = Chef::Knife::ClientDelete.new
+    @knife.name_args = [ 'adam' ]
+  end
+
+  describe 'run' do
+    it 'should delete the client' do
+      @knife.should_receive(:delete_object).with(Chef::ApiClient, 'adam')
+      @knife.run
+    end
+
+    it 'should print usage and exit when a client 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
+end

data/spec/unit/knife/client_edit_spec.rb

@@ -0,0 +1,40 @@
+#
+# Author:: Thomas Bishop (<bishop.thomas@gmail.com>)
+# Copyright:: Copyright (c) 2011 Thomas Bishop
+# 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::ClientEdit do
+  before(:each) do
+    @knife = Chef::Knife::ClientEdit.new
+    @knife.name_args = [ 'adam' ]
+  end
+
+  describe 'run' do
+    it 'should edit the client' do
+      @knife.should_receive(:edit_object).with(Chef::ApiClient, 'adam')
+      @knife.run
+    end
+
+    it 'should print usage and exit when a client 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
+end

data/spec/unit/knife/client_list_spec.rb

@@ -0,0 +1,34 @@
+#
+# Author:: Thomas Bishop (<bishop.thomas@gmail.com>)
+# Copyright:: Copyright (c) 2011 Thomas Bishop
+# 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::ClientList do
+  before(:each) do
+    @knife = Chef::Knife::ClientList.new
+    @knife.name_args = [ 'adam' ]
+  end
+
+  describe 'run' do
+    it 'should list the clients' do
+      Chef::ApiClient.should_receive(:list)
+      @knife.should_receive(:format_list_for_display)
+      @knife.run
+    end
+  end
+end

data/spec/unit/knife/client_reregister_spec.rb

@@ -0,0 +1,62 @@
+#
+# Author:: Thomas Bishop (<bishop.thomas@gmail.com>)
+# Copyright:: Copyright (c) 2011 Thomas Bishop
+# 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::ClientReregister do
+  before(:each) do
+    @knife = Chef::Knife::ClientReregister.new
+    @knife.name_args = [ 'adam' ]
+    @client_mock = mock('client_mock', :private_key => "foo_key")
+    @stdout = StringIO.new
+    @knife.ui.stub!(:stdout).and_return(@stdout)
+  end
+
+  context "when no client name is given on the command line" do
+    before do
+      @knife.name_args = []
+    end
+
+    it 'should print usage and exit when a client name is not provided' do
+      @knife.should_receive(:show_usage)
+      @knife.ui.should_receive(:fatal)
+      lambda { @knife.run }.should raise_error(SystemExit)
+    end
+  end
+
+  context 'when not configured for file output' do
+    it 'reregisters the client and prints the key' do
+      Chef::ApiClient.should_receive(:reregister).with('adam').and_return(@client_mock)
+      @knife.run
+      @stdout.string.should match( /foo_key/ )
+    end
+  end
+
+  context 'when configured for file output' do
+    it 'should write the private key to a file' do
+      Chef::ApiClient.should_receive(:reregister).with('adam').and_return(@client_mock)
+
+      @knife.config[:file] = '/tmp/monkeypants'
+      filehandle = StringIO.new
+      File.should_receive(:open).with('/tmp/monkeypants', 'w').and_yield(filehandle)
+      @knife.run
+      filehandle.string.should == "foo_key"
+    end
+  end
+
+end

data/spec/unit/knife/client_show_spec.rb

@@ -0,0 +1,42 @@
+#
+# Author:: Thomas Bishop (<bishop.thomas@gmail.com>)
+# Copyright:: Copyright (c) 2011 Thomas Bishop
+# 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::ClientShow do
+  before(:each) do
+    @knife = Chef::Knife::ClientShow.new
+    @knife.name_args = [ 'adam' ]
+    @client_mock = mock('client_mock')
+  end
+
+  describe 'run' do
+    it 'should list the client' do
+      Chef::ApiClient.should_receive(:load).with('adam').and_return(@client_mock)
+      @knife.should_receive(:format_for_display).with(@client_mock)
+      @knife.run
+    end
+
+    it 'should print usage and exit when a client 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
+end

data/spec/unit/knife/config_file_selection_spec.rb

@@ -0,0 +1,118 @@
+#
+# Author:: Nicolas Vinot (<aeris@imirhil.fr>)
+# Copyright:: Copyright (c) 2010 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 File.expand_path(File.dirname(__FILE__) + '/../../spec_helper')
+require 'tmpdir'
+
+describe Chef::Knife do
+  before :each do
+    Chef::Config.stub!(:from_file).and_return(true)
+  end
+
+  it "configure knife from KNIFE_HOME env variable" do
+    env_config = File.expand_path(File.join(Dir.tmpdir, 'knife.rb'))
+    File.stub!(:exist?).and_return(false)
+    File.stub!(:exist?).with(env_config).and_return(true)
+
+    ENV['KNIFE_HOME'] = Dir.tmpdir
+    @knife = Chef::Knife.new
+    @knife.configure_chef
+    @knife.config[:config_file].should == env_config
+  end
+
+   it "configure knife from PWD" do
+    pwd_config = "#{Dir.pwd}/knife.rb"
+    File.stub!(:exist?).and_return do | arg |
+      [ pwd_config ].include? arg
+    end
+
+    @knife = Chef::Knife.new
+    @knife.configure_chef
+    @knife.config[:config_file].should == pwd_config
+  end
+  
+  it "configure knife from UPWARD" do
+    upward_dir = File.expand_path "#{Dir.pwd}/.chef"
+    upward_config = File.expand_path "#{upward_dir}/knife.rb"
+    File.stub!(:exist?).and_return do | arg |
+      [ upward_config ].include? arg
+    end
+    Chef::Knife.stub!(:chef_config_dir).and_return(upward_dir)
+    
+    @knife = Chef::Knife.new
+    @knife.configure_chef
+    @knife.config[:config_file].should == upward_config
+  end
+  
+  it "configure knife from HOME" do
+    home_config = File.expand_path(File.join("#{ENV['HOME']}", "/.chef/knife.rb"))
+    File.stub!(:exist?).and_return do | arg |
+      [ home_config ].include? arg
+    end
+    
+    @knife = Chef::Knife.new
+    @knife.configure_chef
+    @knife.config[:config_file].should == home_config
+  end
+  
+  it "configure knife from nothing" do
+    ::File.stub!(:exist?).and_return(false)
+    @knife = Chef::Knife.new
+    @knife.ui.should_receive(:warn).with("No knife configuration file found")
+    @knife.configure_chef
+    @knife.config[:config_file].should be_nil
+  end
+  
+  it "configure knife precedence" do
+    env_config = File.join(Dir.tmpdir, 'knife.rb')
+    pwd_config = "#{Dir.pwd}/knife.rb"
+    upward_dir = File.expand_path "#{Dir.pwd}/.chef"
+    upward_config = File.expand_path "#{upward_dir}/knife.rb"
+    home_config = File.expand_path(File.join("#{ENV['HOME']}", "/.chef/knife.rb"))
+    configs = [ env_config, pwd_config, upward_config, home_config ]
+    File.stub!(:exist?).and_return do | arg |
+      configs.include? arg
+    end
+    Chef::Knife.stub!(:chef_config_dir).and_return(upward_dir)
+    ENV['KNIFE_HOME'] = Dir.tmpdir
+   
+    @knife = Chef::Knife.new
+    @knife.configure_chef
+    @knife.config[:config_file].should == env_config
+    
+    configs.delete env_config
+    @knife.config.delete :config_file
+    @knife.configure_chef
+    @knife.config[:config_file].should == pwd_config
+
+    configs.delete pwd_config
+    @knife.config.delete :config_file
+    @knife.configure_chef
+    @knife.config[:config_file].should == upward_config
+
+    configs.delete upward_config
+    @knife.config.delete :config_file
+    @knife.configure_chef
+    @knife.config[:config_file].should == home_config
+
+    configs.delete home_config
+    @knife.config.delete :config_file
+    @knife.configure_chef
+    @knife.config[:config_file].should be_nil
+  end
+end

data/spec/unit/knife/configure_client_spec.rb

@@ -0,0 +1,83 @@
+#
+# Author:: Thomas Bishop (<bishop.thomas@gmail.com>)
+# Copyright:: Copyright (c) 2011 Thomas Bishop
+# 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::ConfigureClient do
+  before do
+    @knife = Chef::Knife::ConfigureClient.new
+    Chef::Config[:chef_server_url] = 'https://chef.example.com'
+    Chef::Config[:validation_client_name] = 'chef-validator'
+    Chef::Config[:validation_key] = '/etc/chef/validation.pem'
+
+    @stdout = StringIO.new
+    @knife.ui.stub!(:stdout).and_return(@stdout)
+  end
+
+  describe 'run' do
+    it 'should print usage and exit when a directory is not provided' do
+      @knife.should_receive(:show_usage)
+      @knife.ui.should_receive(:fatal).with(/must provide the directory/)
+      lambda {
+        @knife.run
+      }.should raise_error SystemExit
+    end
+
+    describe 'when specifing a directory' do
+      before do
+        @knife.name_args = ['/home/bob/.chef']
+        @client_file = StringIO.new
+        @validation_file = StringIO.new
+        File.should_receive(:open).with('/home/bob/.chef/client.rb', 'w').
+                                   and_yield(@client_file)
+        File.should_receive(:open).with('/home/bob/.chef/validation.pem', 'w').
+                                   and_yield(@validation_file)
+        IO.should_receive(:read).and_return('foo_bar_baz')
+      end
+
+      it 'should recursively create the directory' do
+        FileUtils.should_receive(:mkdir_p).with('/home/bob/.chef')
+        @knife.run
+      end
+
+      it 'should write out the config file' do
+        FileUtils.stub!(:mkdir_p)
+        @knife.run
+        @client_file.string.should match /log_level\s+\:info/
+        @client_file.string.should match /log_location\s+STDOUT/
+        @client_file.string.should match /chef_server_url\s+'https\:\/\/chef\.example\.com'/
+        @client_file.string.should match /validation_client_name\s+'chef-validator'/
+      end
+
+      it 'should write out the validation.pem file' do
+        FileUtils.stub!(:mkdir_p)
+        @knife.run
+        @validation_file.string.should match /foo_bar_baz/
+      end
+
+      it 'should print information on what is being configured' do
+        FileUtils.stub!(:mkdir_p)
+        @knife.run
+        @stdout.string.should match /creating client configuration/i
+        @stdout.string.should match /writing client\.rb/i
+        @stdout.string.should match /writing validation\.pem/i
+      end
+    end
+  end
+
+end

data/spec/unit/knife/configure_spec.rb

@@ -0,0 +1,229 @@
+require 'spec_helper'
+
+describe Chef::Knife::Configure do
+  before do
+    @original_config = Chef::Config.configuration.dup
+
+    Chef::Log.logger = Logger.new(StringIO.new)
+
+    Chef::Config[:node_name]  = "webmonkey.example.com"
+    @knife = Chef::Knife::Configure.new
+    @rest_client = mock("null rest client", :post_rest => { :result => :true })
+    @knife.stub!(:rest).and_return(@rest_client)
+
+    @out = StringIO.new
+    @knife.ui.stub!(:stdout).and_return(@out)
+    @knife.config[:config_file] = '/home/you/.chef/knife.rb'
+
+    @in = StringIO.new("\n" * 7)
+    @knife.ui.stub!(:stdin).and_return(@in)
+
+    @err = StringIO.new
+    @knife.ui.stub!(:stderr).and_return(@err)
+
+    @ohai = Ohai::System.new
+    @ohai.stub(:require_plugin)
+    @ohai[:fqdn] = "foo.example.org"
+    Ohai::System.stub!(:new).and_return(@ohai)
+  end
+
+  after do
+    Chef::Config.configuration.replace(@original_config)
+  end
+
+  it "asks the user for the URL of the chef server" do
+    @knife.ask_user_for_config
+    @out.string.should match(Regexp.escape('Please enter the chef server URL: [http://foo.example.org:4000]'))
+    @knife.chef_server.should == 'http://foo.example.org:4000'
+  end
+
+  it "asks the user for the clientname they want for the new client if -i is specified" do
+    @knife.config[:initial] = true
+    Etc.stub!(:getlogin).and_return("a-new-user")
+    @knife.ask_user_for_config
+    @out.string.should match(Regexp.escape("Please enter a clientname for the new client: [a-new-user]"))
+    @knife.new_client_name.should == Etc.getlogin
+  end
+
+  it "should not ask the user for the clientname they want for the new client if -i and --node_name are specified" do
+    @knife.config[:initial] = true
+    @knife.config[:node_name] = 'testnode'
+    Etc.stub!(:getlogin).and_return("a-new-user")
+    @knife.ask_user_for_config
+    @out.string.should_not match(Regexp.escape("Please enter a clientname for the new client"))
+    @knife.new_client_name.should == 'testnode'
+  end
+
+  it "asks the user for the existing API username or clientname if -i is not specified" do
+    Etc.stub!(:getlogin).and_return("a-new-user")
+    @knife.ask_user_for_config
+    @out.string.should match(Regexp.escape("Please enter an existing username or clientname for the API: [a-new-user]"))
+    @knife.new_client_name.should == Etc.getlogin
+  end
+
+  it "asks the user for the existing admin client's name if -i is specified" do
+    @knife.config[:initial] = true
+    @knife.ask_user_for_config
+    @out.string.should match(Regexp.escape("Please enter the existing admin clientname: [chef-webui]"))
+    @knife.admin_client_name.should == 'chef-webui'
+  end
+
+  it "should not ask the user for the existing admin client's name if -i and --admin-client_name are specified" do
+    @knife.config[:initial] = true
+    @knife.config[:admin_client_name] = 'my-webui'
+    @knife.ask_user_for_config
+    @out.string.should_not match(Regexp.escape("Please enter the existing admin clientname:"))
+    @knife.admin_client_name.should == 'my-webui'
+  end
+
+  it "should not ask the user for the existing admin client's name if -i is not specified" do
+    @knife.ask_user_for_config
+    @out.string.should_not match(Regexp.escape("Please enter the existing admin clientname: [chef-webui]"))
+    @knife.admin_client_name.should_not == 'chef-webui'
+  end
+
+  it "asks the user for the location of the existing admin key if -i is specified" do
+    @knife.config[:initial] = true
+    @knife.ask_user_for_config
+    @out.string.should match(Regexp.escape("Please enter the location of the existing admin client's private key: [/etc/chef/webui.pem]"))
+    if windows?
+      @knife.admin_client_key.capitalize.should == 'C:/etc/chef/webui.pem'
+    else
+      @knife.admin_client_key.should == '/etc/chef/webui.pem'
+    end
+  end
+
+  it "should not ask the user for the location of the existing admin key if -i and --admin_client_key are specified" do
+    @knife.config[:initial] = true
+    @knife.config[:admin_client_key] = '/home/you/.chef/my-webui.pem'
+    @knife.ask_user_for_config
+    @out.string.should_not match(Regexp.escape("Please enter the location of the existing admin client's private key:"))
+    if windows?
+      @knife.admin_client_key.capitalize.should == 'C:/home/you/.chef/my-webui.pem'
+    else
+      @knife.admin_client_key.should == '/home/you/.chef/my-webui.pem'
+    end
+  end
+
+  it "should not ask the user for the location of the existing admin key if -i is not specified" do
+    @knife.ask_user_for_config
+    @out.string.should_not match(Regexp.escape("Please enter the location of the existing admin client's private key: [/etc/chef/webui.pem]"))
+    if windows?
+      @knife.admin_client_key.should_not == 'C:/etc//chef/webui.pem'
+    else
+      @knife.admin_client_key.should_not == '/etc/chef/webui.pem' 
+    end
+  end
+
+  it "asks the user for the location of a chef repo" do
+    @knife.ask_user_for_config
+    @out.string.should match(Regexp.escape("Please enter the path to a chef repository (or leave blank):"))
+    @knife.chef_repo.should == ''
+  end
+
+  it "asks the users for the name of the validation client" do
+    @knife.ask_user_for_config
+    @out.string.should match(Regexp.escape("Please enter the validation clientname: [chef-validator]"))
+    @knife.validation_client_name.should == 'chef-validator'
+  end
+
+  it "should not ask the users for the name of the validation client if --validation_client_name is specified" do
+    @knife.config[:validation_client_name] = 'my-validator'
+    @knife.ask_user_for_config
+    @out.string.should_not match(Regexp.escape("Please enter the validation clientname:"))
+    @knife.validation_client_name.should == 'my-validator'
+  end
+
+  it "asks the users for the location of the validation key" do
+    @knife.ask_user_for_config
+    @out.string.should match(Regexp.escape("Please enter the location of the validation key: [/etc/chef/validation.pem]"))
+    if windows?
+      @knife.validation_key.capitalize.should == 'C:/etc/chef/validation.pem'
+    else
+      @knife.validation_key.should == '/etc/chef/validation.pem' 
+    end
+  end
+
+  it "should not ask the users for the location of the validation key if --validation_key is specified" do
+    @knife.config[:validation_key] = '/home/you/.chef/my-validation.pem'
+    @knife.ask_user_for_config
+    @out.string.should_not match(Regexp.escape("Please enter the location of the validation key:"))
+    if windows?
+      @knife.validation_key.capitalize.should == 'C:/home/you/.chef/my-validation.pem'
+    else
+      @knife.validation_key.should == '/home/you/.chef/my-validation.pem'
+    end
+  end
+
+  it "should not ask the user for anything if -i and all other properties are specified" do
+    @knife.config[:initial] = true
+    @knife.config[:chef_server_url] = 'http://localhost:5000'
+    @knife.config[:node_name] = 'testnode'
+    @knife.config[:admin_client_name] = 'my-webui'
+    @knife.config[:admin_client_key] = '/home/you/.chef/my-webui.pem'
+    @knife.config[:validation_client_name] = 'my-validator'
+    @knife.config[:validation_key] = '/home/you/.chef/my-validation.pem'
+    @knife.config[:repository] = ''
+    @knife.config[:client_key] = '/home/you/a-new-user.pem'
+    Etc.stub!(:getlogin).and_return('a-new-user')
+
+    @knife.ask_user_for_config
+    @out.string.should match(/\s*/)
+
+    @knife.new_client_name.should == 'testnode'
+    @knife.chef_server.should == 'http://localhost:5000'
+    @knife.admin_client_name.should == 'my-webui'
+    if windows?
+      @knife.admin_client_key.capitalize.should == 'C:/home/you/.chef/my-webui.pem'
+      @knife.validation_key.capitalize.should == 'C:/home/you/.chef/my-validation.pem'
+      @knife.new_client_key.capitalize.should == 'C:/home/you/a-new-user.pem'
+    else
+      @knife.admin_client_key.should == '/home/you/.chef/my-webui.pem'
+      @knife.validation_key.should == '/home/you/.chef/my-validation.pem'
+      @knife.new_client_key.should == '/home/you/a-new-user.pem'
+    end
+    @knife.validation_client_name.should == 'my-validator'
+    @knife.chef_repo.should == ''
+  end
+
+  it "writes the new data to a config file" do
+    File.stub!(:expand_path).with("/home/you/.chef/knife.rb").and_return("/home/you/.chef/knife.rb")
+    File.stub!(:expand_path).with("/home/you/.chef/#{Etc.getlogin}.pem").and_return("/home/you/.chef/#{Etc.getlogin}.pem")
+    File.stub!(:expand_path).with("/etc/chef/validation.pem").and_return("/etc/chef/validation.pem")
+    File.stub!(:expand_path).with("/etc/chef/webui.pem").and_return("/etc/chef/webui.pem")
+    FileUtils.should_receive(:mkdir_p).with("/home/you/.chef")
+    config_file = StringIO.new
+    ::File.should_receive(:open).with("/home/you/.chef/knife.rb", "w").and_yield config_file
+    @knife.config[:repository] = '/home/you/chef-repo'
+    @knife.run
+    config_file.string.should match(/^node_name[\s]+'#{Etc.getlogin}'$/)
+    config_file.string.should match(%r{^client_key[\s]+'/home/you/.chef/#{Etc.getlogin}.pem'$})
+    config_file.string.should match(/^validation_client_name\s+'chef-validator'$/)
+    config_file.string.should match(%r{^validation_key\s+'/etc/chef/validation.pem'$})
+    config_file.string.should match(%r{^chef_server_url\s+'http://foo.example.org:4000'$})
+    config_file.string.should match(%r{cookbook_path\s+\[ '/home/you/chef-repo/cookbooks' \]})
+  end
+
+  it "creates a new client when given the --initial option" do
+    File.stub!(:expand_path).with("/home/you/.chef/knife.rb").and_return("/home/you/.chef/knife.rb")
+    File.stub!(:expand_path).with("/home/you/.chef/a-new-user.pem").and_return("/home/you/.chef/a-new-user.pem")
+    File.stub!(:expand_path).with("/etc/chef/validation.pem").and_return("/etc/chef/validation.pem")
+    File.stub!(:expand_path).with("/etc/chef/webui.pem").and_return("/etc/chef/webui.pem")
+    Chef::Config[:node_name]  = "webmonkey.example.com"
+    client_command = Chef::Knife::ClientCreate.new
+    client_command.should_receive(:run)
+
+    Etc.stub!(:getlogin).and_return("a-new-user")
+
+    Chef::Knife::ClientCreate.stub!(:new).and_return(client_command)
+    FileUtils.should_receive(:mkdir_p).with("/home/you/.chef")
+    ::File.should_receive(:open).with("/home/you/.chef/knife.rb", "w")
+    @knife.config[:initial] = true
+    @knife.run
+    client_command.name_args.should == Array("a-new-user")
+    client_command.config[:admin].should be_true
+    client_command.config[:file].should == "/home/you/.chef/a-new-user.pem"
+    client_command.config[:yes].should be_true
+    client_command.config[:disable_editing].should be_true
+  end
+end